On Thursday 17 October 2002 9:10 am, Andre Poenitz wrote:
> Mostly whitespace,
> _foo -> foo_,
> i++ -> ++i,
> remove unneeded header,
> make sure that operator=() is not used.
>
> Ok?

While you're at it, why not wrap 
        QString * texts_;
        QPixmap* pixmaps_;
inside a boost::scoped_array and get rid of ~QBrowseBox() ?

Also, since you're playing with formatting, what's the "standard" 
way to declare Q_OBJECT?

I find
class BulletsModule : public BulletsModuleBase
{
        Q_OBJECT

class QAboutDialog : public QAboutDialogBase
{
Q_OBJECT

class QBibitemDialog : public QBibitemDialogBase
{ Q_OBJECT

Angus

Reply via email to