Shared pointers and visitors

Casper Boemann cbr at boemann.dk
Fri Apr 22 09:56:02 CEST 2005


On Thursday 21 April 2005 20:45, Boudewijn Rempt wrote:
> On Thursday 21 April 2005 20:29, Casper Boemann wrote:
> > On Thursday 21 April 2005 20:27, Boudewijn Rempt wrote:
> > > On Thursday 21 April 2005 20:18, Casper Boemann wrote:
> > > > On Thursday 21 April 2005 19:46, Boudewijn Rempt wrote:
> > > > > Was there a good reason for not using the KisPaintDeviceSP but
> > > > > instead a plain pointer in the visitors? I truly don't remember
> > > > > anything about it, but if there's no good reason, I might convert
> > > > > the visitors to use it.
> > > >
> > > > well yes - it crashes
> > > >
> > > > Or rather the kisBackground crashes. The kisbackground constructor
> > > > uses an iterator to set the pattern, and when this pointer is
> > > > destroyed the background is destroyed, even before the constructor is
> > > > done.
> > >
> > > But the background is never zoomed, rotated or transformed, is it?
> >
> > No - what are you hinting at?
>
> If I remove the visitor tag from the rotate/scale/transform visitors, I'd
> like to make them use the KSharedPtr arguments instead of the plain
> pointers.
Oh sorry you wrote in the visitors not iterators

I don't know, but I don't think there were any specific problems.

However, and this is more general, I don't like the shared pointers. They lead 
to all sorts of problems like the iterator/kisbackground I talked about. Or 
that automatic casting is disabled (because it now has to go through two 
castings and C++ doesn't do that)

I really feel it's a thing from hell. Either you have them built in like in 
java or you don't. This half way of doing it is not good enough imho

-- 
best regards / venlig hilsen
Casper Boemann


More information about the kimageshop mailing list