Lars Gullik Bjønnes a écrit :
Abdelrazak Younes <[EMAIL PROTECTED]> writes:

| Index: D:/msys/home/yns/src/lyx-svn/trunk/src/frontends/qt4/QLMenubar.h
| ===================================================================
| --- D:/msys/home/yns/src/lyx-svn/trunk/src/frontends/qt4/QLMenubar.h  
(revision 13312)
| +++ D:/msys/home/yns/src/lyx-svn/trunk/src/frontends/qt4/QLMenubar.h  
(working copy)
| @@ -15,7 +15,11 @@
| | #include "frontends/Menubar.h" | | -#include <map>
| +#include <map>
| +
| +#ifdef Q_WS_MACX
| +#include <boost/scoped_ptr.hpp>
| +#endif

Wh is the ifdef needed? Does something bad happens if it is left out?

No it's just that the include is not needed otherwise. I thought that it
was a general rule for LyX code but if you don't mind, I prefer to
remove them... I am not really fond of #ifdef. I would also get rid of
the one around menubar_ at the end:

#ifdef Q_WS_MACX
        boost::scoped_ptr<QMenuBar> menubar_;
#endif

I 'll send a new patch that gets rid of the #ifdef.

Abdel.

Reply via email to