koffice/krita/core/tiles

Casper Boemann cbr at boemann.dk
Sun Mar 27 15:29:21 CEST 2005


On Sunday 27 March 2005 14:38, Adrian Page wrote:
> Casper Boemann wrote:
> > CVS commit by boemann:
> >
> > --- koffice/krita/core/tiles/kis_memento.h  #1.4:1.5
> > @@ -45,4 +45,6 @@ private:
> >          Q_UINT32 m_numTiles;
> >          KisTile **m_redoHashTable;
> > +        struct DeletedTile {Q_INT32 col; Q_INT32 row; DeletedTile
> > *next;}; +        DeletedTile *m_delTilesTable;
> >  };
>
> A quick tip: you could use QValueVector<struct DeletedTile> here, which
> would mean you wouldn't need to write all the linked list manipulation
> code. The Qt collection classes save you time and are less prone to bugs
> - worth considering in the future.

Thanks

didn't think of that because I don't like to have qt this deep into krita, but 
I guess you're right.

-- 
best regards / venlig hilsen
Casper Boemann


More information about the kimageshop mailing list