<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/2/3 Angelo Naselli <span dir="ltr"><<a href="mailto:anaselli@linux.it" target="_blank">anaselli@linux.it</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
</div>Il 03/02/2013 23:05, Angelo Naselli ha scritto:<br>
<div class="im">> Il 03/02/2013 18:33, Gilles Caulier ha scritto:<br>
>> - PrintImages ==> (Angelo Naselli ?)<br>
</div>I should have fixed the UNINIT ones.<br></blockquote><div><br></div><div style>ok.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I left printimagesconfig.cpp, because i haven't undesrstood<br>
what's wrong, and because it's generated from .kcfg and .kcfgc<br>
<br></blockquote><div><br></div><div style>Right it's a auto-generated files. We cannot fix it as well in kcfg source...</div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
And file layouttree.cpp, because it's not my code and i should<br>
learn it better first, I've just protected a call in case of null<br>
pointer, at the moment.<br></blockquote><div><br></div><div style>Yes, this is the better way. In fact, parser look if m_root is null when you assign it with bestTree which can be null by processing previous loop. I think the current code is safe...</div>
<div style><br></div><div style>cropframe .cpp fixes will be very simple. Some members are not initialized in constructor.</div><div style><br></div><div style>Gilles</div></div></div></div>