I guess you can even remove this now
const std::string REDRAW_IDLE = "redraw-idle";
in: if (launcher_controller_.get()) no need for .get().
Overall looks good...
PS: pay attention to style, i.e.: bool UnityScreen::shellIsHidden(const CompOutput &output) -> bool UnityScreen::ShellIsHidden(CompOutput const& output) const std::vector<Window> &nuxwins(nux::XInputWindow::NativeHandleList()); -> std::vector<Window> const& nuxwins(nux::XInputWindow::NativeHandleList());
« Back to merge proposal
I guess you can even remove this now
const std::string REDRAW_IDLE = "redraw-idle";
in: if (launcher_ controller_ .get()) no need for .get().
Overall looks good...
PS: pay attention to style, i.e.: :shellIsHidden( const CompOutput &output) -> bool UnityScreen: :ShellIsHidden( CompOutput const& output) nux::XInputWind ow::NativeHandl eList() ); -> std::vector<Window> const& nuxwins( nux::XInputWind ow::NativeHandl eList() );
bool UnityScreen:
const std::vector<Window> &nuxwins(