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

| > 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 really dislike ifdef as well. Other solutions should be sought
after.

| I would also get rid of
| the one around menubar_ at the end:
| | #ifdef Q_WS_MACX
|       boost::scoped_ptr<QMenuBar> menubar_;
| #endif

Why is menubar_ needed on mac and not otherwise?

It has to do with the way Qt rearrange automatically the Menus under Mac. There is maybe a cleaner solution with Qt4 but I did not investigate that much.

I have tried to minimise '#ifdef' use inside QLMenuBar.C.

Abdel.


Reply via email to