<div dir="ltr">Done. I review all code and fixed coding style for better readability. It's not easy. Some part of code and very difficult to parse. Coding style is very important to share code with other developer.<div>

<br></div><div style>I don't blame nobody here. But it's a strict rule that we need to follow, to be sure that code can be maintained by all developers...</div><div style><br></div><div style>I think code is more easy to handle now. I take a care to don't break logic everywhere. Also, when it's possible to constify pointers is always more safe, when it's possible.</div>

<div style><br></div><div style>Gilles</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/2/4 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 04/02/2013 12:34, Angelo Naselli ha scritto:<br>
<div class="im">> Il 04/02/2013 11:21, Gilles Caulier ha scritto:<br>
>> cropframe .cpp fixes will be very simple. Some members are not<br>
>> initialized in constructor.<br>
> I should have missed that last night...<br>
</div>hmm Gilles can you check it better? i should have committed<br>
it last night as well...<br>
<br>
Thanks<br>
<div class="im">        Angelo<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v1.4.12 (GNU/Linux)<br>
Comment: Using GnuPG with Mozilla - <a href="http://enigmail.mozdev.org/" target="_blank">http://enigmail.mozdev.org/</a><br>
<br>
</div>iEYEARECAAYFAlEQFH8ACgkQqEs9DA4DquAGxgCffCpiF4FxLMAsYsZx4du4mQcK<br>
ueMAoJeVK99uhDp0IzNomc66ScYEOjSx<br>
=CFZ5<br>
<div class="HOEnZb"><div class="h5">-----END PGP SIGNATURE-----<br>
_______________________________________________<br>
Kde-imaging mailing list<br>
<a href="mailto:Kde-imaging@kde.org">Kde-imaging@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-imaging" target="_blank">https://mail.kde.org/mailman/listinfo/kde-imaging</a><br>
</div></div></blockquote></div><br></div>