>Oh, your patch is the wrong way round.<br><br>Sorry, my mistake.<br><br>>Please submit a patch the other way round, and clean up the code (you can remove the comments like<br>>"NOTE: Now this class are using QTableWidgetItem to stored and retrieve the data( color )", this is obvious from the code,<br>
>you can also remove all the "old" code that you commented out).<br><br>Done ; )<br><br>Greetings <br><br><div><span class="gmail_quote">2008/2/12, David Faure <<a href="mailto:faure@kde.org">faure@kde.org</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Monday 11 February 2008, Percy Camilo Triveņo Aucahuasi wrote:<br>> ; ) ... ok , I also fix a bug ... when you drop a color in a void cell (cell<br>> without color) the application crasshed<br><br>Oh, your patch is the wrong way round.<br>
<br>-    //QColor *colors; // WARNING NOT NESCESARY ... erase this member, dont alow dynamic alloc..<br>+    QColor *colors;<br><br>I guess you commented it out, since it's not commented out in the svn version.<br><br>
Please submit a patch the other way round, and clean up the code (you can remove the comments like<br>"NOTE: Now this class are using QTableWidgetItem to stored and retrieve the data( color )", this is obvious from the code,<br>
you can also remove all the "old" code that you commented out).<br><br>--<br>David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, sponsored by Trolltech to work on KDE,<br>Konqueror (<a href="http://www.konqueror.org">http://www.konqueror.org</a>), and KOffice (<a href="http://www.koffice.org">http://www.koffice.org</a>).<br>
</blockquote></div><br>