[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kopete-devel
Subject:    Re: [kopete-devel] [PATCH] New class Kopete::Picture
From:       Olivier Goffart <ogoffart () kde ! org>
Date:       2005-12-10 21:15:37
Message-ID: 200512102215.41968.ogoffart () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Le Samedi 10 D=E9cembre 2005 21:46, Micha=EBl Larouche a =E9crit=A0:
> Le December 10, 2005 15:31, Olivier Goffart a =E9crit=A0:
> > Le Samedi 10 D=E9cembre 2005 21:06, Micha=EBl Larouche a =E9crit=A0:
> > > Yet another version which do the conversion on demand. See the new
> > > attached patch.
> >
> > Cool.
> >
> > i think d->update* variable are useless, and you can use
> > d->image.isNull() or something instead.
>
> No, they are not.
>
> I was using isNull or isEmpty on a getter method. But what if the content
> is changed with setPicture ?

Then you can safelly set the image or the path to null again.
no ?

> purpose of this class is to cache the QImage, path and base64, processing
> those each time would kill the purpose of my class.

of course.

[Attachment #5 (application/pgp-signature)]

_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel


[prev in list] [next in list] [prev in thread] [next in thread]