anthonyfieroni added a comment.

  Why not
  
    static const auto flags = qGuiApp->platformName() == 
QLatin1String("wayland") ? Qt::WindowFlags(Qt::ToolTip | 
Qt::FramelessWindowHint | Qt::BypassWindowManagerHint) :
                                                                                
    Qt::WindowFlags(Qt::Window | Qt::FramelessWindowHint | 
Qt::BypassWindowManagerHint);
    q->setWindowFlags(flags);
  
  It's not need to check in every completion construction.

REPOSITORY
  R284 KCompletion

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D12448

To: apol, #frameworks, davidedmundson
Cc: anthonyfieroni, michaelh, bruns

Reply via email to