On Mon, Feb 05, 2007 at 04:36:01PM +0100, Abdelrazak Younes wrote:
> Jean-Marc Lasgouttes wrote:
> >
> >+////////////////////////////////////////////////////////////////////////
> >+// Mac specific stuff goes here...
> >+#ifdef Q_WS_MACX
> >+
> >+class MenuTranslator : public QTranslator {
> >+ virtual QString translate(const char * context,
> >+ const char * sourceText,
> >+ const char * comment = 0) const;
>
> I don't see where in your patch MenuTranslator::translate() is called.
> If this is called internally by Qt, then I suggest to add a virtual
> destructor to MenuTranslator.
Which would not be needed if the base class already has one.
Andre'
