On Sep 19, 2010, at 8:25 AM, Pavel Sanda wrote: > Jens Nöckel wrote: >> + connect(dontswapCB, SIGNAL(toggled(bool)), >> + this, SIGNAL(changed())); >> + dontswapCB->setVisible(false); >> +#ifdef Q_WS_MACX >> +#if QT_VERSION > 0x040600 >> + dontswapCB->setVisible(true); >> +#endif >> >> But then the actual code on Mac would execute two calls to setVisible >> instead of one, which just seems slightly more inelegant. > > i see. what you feel better with > > bool swapcb=false; > > #ifdef Q_WS_MACX > #if QT_VERSION > 0x040600 > swapcb=true; > #endif > > dontswapCB->setVisible(swapcb); > > pavel
That's perfect - please feel free to modify my patch this way. Jens
