-
Notifications
You must be signed in to change notification settings - Fork 342
Conversation
b64bb4f
to
e628045
Compare
xwayland/xwm.c
Outdated
@@ -365,6 +368,36 @@ static void read_surface_class(struct wlr_xwm *xwm, | |||
wlr_signal_emit_safe(&surface->events.set_class, surface); | |||
} | |||
|
|||
#ifdef WLR_HAS_XCB_ICCCM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does this require ICCCM?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It probably doesn't. I was reading about WM_WINDOW_ROLE
at https://tronche.com/gui/x/icccm/sec-5.html and due to the page title and the fact that it has icccm in the URL, I drew the conclusion that it required ICCCM.
Should I remove the #ifdef
?.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
xcb-icccm is an optional library that makes it easier to parse and format ICCCM messages. If we can parse some simple ICCCM messages without using xcb-icccm, I think we should go for it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the explanation. I've removed the #ifdef
.
I'm fine with |
e628045
to
b8cc4a4
Compare
Thanks! |
Do we want to name the event
set_role
orset_window_role
?