Looks good. A few questions: This looks like a typo: > QAboutDialog::~QAboutDialog() > -{ > -} > +{} > +y >
?????: Index: src/frontends/qt2/QRefDialog.h - void refHighlighted(QString const &); - void refSelected(QString const &); + void refHighlighted(const QString &); + void refSelected(const QString &); Why?: Index: src/frontends/xforms/FormPrint.C +#include "support/lstrings.h" I don't understand... When should \param be used and when \c? Index: src/frontends/xforms/combox.h /** The browser will be displayed either below or above the button, - * dependent upon \param position. + * dependent upon \c position. -- Angus