Hm. About code separation.

[Boudewijn Rempt]

On Tuesday 27 July 2004 22:15, Casper Boemann wrote:
> Hi
>
> your ideas sounds like good OO thinking.

I'm up to my elbows in the code, comparing the tile manager to what it was, 
but even though I get everything to compile, linking gives tons of errors 
that point to something being wrong with the tile manager:

.libs/kis_strategy_colorspace_cmyk.o(.text+0x99c): In function `KisStrategyColorSpaceCMYK::convertToImage(KSharedPtr<KisTileMgr>, unsigned, int, int, int, int) const':
: warning: undefined reference to `vtable for KisPixelData'
.libs/kis_strategy_colorspace_cmyk.o(.text+0xaa2): In function `KisStrategyColorSpaceCMYK::convertToImage(KSharedPtr<KisTileMgr>, unsigned, int, int, int, int) const':
: warning: undefined reference to `KisTileMgr::readPixelData(KSharedPtr<KisPixelData>)'
.libs/kis_strategy_colorspace_cmyk.o(.text+0x11d1): In function `KisStrategyColorSpaceCMYK::~KisStrategyColorSpaceCMYK [in-charge deleting]()':
: warning: undefined reference to `KisStrategyColorSpace::~KisStrategyColorSpace [not-in-charge]()'
.libs/kis_strategy_colorspace_cmyk.o(.text+0x1241): In function `KisStrategyColorSpaceCMYK::~KisStrategyColorSpaceCMYK [in-charge]()':
: warning: undefined reference to `KisStrategyColorSpace::~KisStrategyColorSpace [not-in-charge]()'
.libs/kis_strategy_colorspace_cmyk.o(.text+0x12a1): In function `KisStrategyColorSpaceCMYK::~KisStrategyColorSpaceCMYK [not-in-charge]()':
: warning: undefined reference to `KisStrategyColorSpace::~KisStrategyColorSpace [not-in-charge]()'
.libs/kis_strategy_colorspace_cmyk.o(.text+0x12ed): In function `KisStrategyColorSpaceCMYK::KisStrategyColorSpaceCMYK[in-charge]()':
: warning: undefined reference to `KisStrategyColorSpace::KisStrategyColorSpace[not-in-charge](QString const&)'
.libs/kis_strategy_colorspace_cmyk.o(.text+0x139d): In function `KisStrategyColorSpaceCMYK::KisStrategyColorSpaceCMYK[not-in-charge]()':
: warning: undefined reference to `KisStrategyColorSpace::KisStrategyColorSpace[not-in-charge](QString const&)'
.libs/kis_strategy_colorspace_cmyk.o(.gnu.linkonce.d._ZTV25KisStrategyColorSpaceCMYK+0x38): warning: undefined reference to `KisStrategyColorSpace::convertTo(KisPixelRepresentation&, KisPixelRepresentation&, KSharedPtr<KisStrategyColorSpace>)'
.libs/kis_strategy_colorspace_cmyk.o(.gnu.linkonce.d._ZTI25KisStrategyColorSpaceCMYK+0x8): warning: undefined reference to `typeinfo for KisStrategyColorSpace'
.libs/kis_strategy_colorspace_cmyk.o(.gnu.linkonce.t._ZN15KisIteratorUnitcvPhEv+0x48): In function `KisIteratorUnit::operator unsigned char*()':
: warning: undefined reference to `KisTileMgr::tile(int, int)'
.libs/kis_strategy_colorspace_cmyk.o(.gnu.linkonce.t._ZN15KisIteratorUnitcvPhEv+0xbc): In function `KisIteratorUnit::operator unsigned char*()':
: warning: undefined reference to `KisTile::data(int, int)'
.libs/kis_strategy_colorspace_cmyk.o(.gnu.linkonce.t._ZN15KisIteratorUnitcvPhEv+0xf4): In function `KisIteratorUnit::operator unsigned char*()':
: warning: undefined reference to `KisTileMgr::tile(int, int)'

And more stuff like that... Oh, well, I'll probably find out what's wrong,
fix it, and then immediately forget it and be just as baffled the next time.

>
> btw I have seen the DAB video now - cool - very cool - lets have somthing
> like that in the future.
>
And impasto is even cooler :-).

-- 
Boudewijn Rempt | http://www.valdyas.org/fading/index.cgi