<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Hi Lubomir,<br></div><div class="gmail_quote"><br></div><div class="gmail_quote"><div>It's not about a dangerous lack of initialization - it's more about code consistence. So, the question should be why not initialize all members as we should do?</div><div><br></div><div>In addition, most of these are not following the new coding style (which we discussed in another thread), so if you want to apply them, I could send new patches fixing it (and join all changes in a single patch if you like)...</div></div><div class="gmail_quote"><br></div><div class="gmail_quote">2015-03-23 12:31 GMT-03:00 Lubomir I. Ivanov <span dir="ltr"><<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On 23 March 2015 at 06:43, Marcos Cardinot <span dir="ltr"><<a href="mailto:mcardinot@gmail.com" target="_blank">mcardinot@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>The attached patches initialize some members in several classes of the qt-ui package...</div><div><br></div></div></blockquote><div> <br></div></span></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><div></div></span><div>hello,</div><div><br></div><div>do these patches have the idea to solve compiler warnings (what GCC version and -W level) or potentially dangerous lack of initialization (according to you)?</div><div>if no to both, perhaps we should leave these changes out.<br></div><div><br></div><div>for instance about 008, the setup() is called in the constructor which will surely *not* leave the variable uninitialized.</div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><div><br></div><div>lubomir<br></div><div>--</div><div><br></div></div></div></div>
</blockquote></div><br></div></div>