KXStitch moved to KDE Review

Albert Astals Cid aacid at kde.org
Tue Jun 17 18:59:08 BST 2014


El Dilluns, 16 de juny de 2014, a les 23:21:41, Steve Allewell va escriure:
> On 16/06/14 22:46, Albert Astals Cid wrote:
> > You may want to have a look at Palette.cpp:284
> 
> Nothing jumps out as being wrong, I'll dig deeper into it.

Well, m_width and m_height are only initialized after this line, so it looks 
wrong to me. Maybe initialize them in the constructor to 0? Or something?

Cheers,
  Albert




More information about the kde-core-devel mailing list