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

List:       koffice-devel
Subject:    Re: kpresenter patch
From:       Thorsten Zachmann <t.zachmann () zagge ! de>
Date:       2004-03-18 5:19:18
Message-ID: 200403180619.18663.t.zachmann () zagge ! de
[Download RAW message or body]

Hello Nicolas,

> > here is another patch for kpresenter. I cleaned up the part which is used
> > for painting in the canvas.
>
> Thank you for adding some Doxygen comments.
>
> Just a few comments on the comments in the class defintions:
> - one-line Doxygen comments start with /// (// is not enough)
> - the class PresStep's class comment starts only with /* so it is not
> Doxygen compatible (however I do not know if Doxygen support classes in a
> class.) - in KDE, the texts in Doxygen comments start normally with an
> upper case letter.

I'm new to doxygen commands. I'll update the one you proposed and than have a 
deeper look into doxygen, to make the comments look better.

> Also unrelated, what is this PicCache class? Nothing should use a QPicture
> directly in KPresenter. (Well, it is not in your code anyway...)

The PicCache class is nowhere used in the source. I will remove it in a 
different commit.

Thorsten 
_______________________________________________
koffice-devel mailing list
koffice-devel@mail.kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]