KColorCells fails on resizing rows/columns

David Faure faure at kde.org
Tue Feb 12 18:00:50 GMT 2008


On Monday 11 February 2008, Percy Camilo TriveƱo Aucahuasi wrote:
> ; ) ... ok , I also fix a bug ... when you drop a color in a void cell (cell
> without color) the application crasshed

Oh, your patch is the wrong way round.

-    //QColor *colors; // WARNING NOT NESCESARY ... erase this member, dont alow dynamic alloc..
+    QColor *colors;

I guess you commented it out, since it's not commented out in the svn version.

Please submit a patch the other way round, and clean up the code (you can remove the comments like
"NOTE: Now this class are using QTableWidgetItem to stored and retrieve the data( color )", this is obvious from the code,
you can also remove all the "old" code that you commented out).

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).




More information about the kde-core-devel mailing list