> On Sept. 6, 2015, 10:52 p.m., Albert Astals Cid wrote: > > include config-kwindowsystem.h and use KWINDOWSYSTEM_HAVE_X11?
And move all X11 related includes together ? - Samuel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125081/#review84924 ----------------------------------------------------------- On Sept. 6, 2015, 10:46 p.m., Samuel Gaist wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125081/ > ----------------------------------------------------------- > > (Updated Sept. 6, 2015, 10:46 p.m.) > > > Review request for KDE Base Apps. > > > Repository: kde-baseapps > > > Description > ------- > > Move netwm.h include under Q_WS_X11 since that header is made available with > the xcb plugin. > > > added .reviewboardrc > > > Diffs > ----- > > .reviewboardrc PRE-CREATION > konqueror/src/konqmainwindow.cpp c7a81c8 > > Diff: https://git.reviewboard.kde.org/r/125081/diff/ > > > Testing > ------- > > Build on OS X 10.8 > > > Thanks, > > Samuel Gaist > >