zzag added inline comments. INLINE COMMENTS
> xcbeventlistener.cpp:135 > > - xcb_generic_event_t* e = static_cast<xcb_generic_event_t *>(message); > + auto* e = static_cast<xcb_generic_event_t *>(message); > const uint8_t xEventType = e->response_type & ~0x80; `auto *e` > broulik wrote in xcbeventlistener.cpp:151 > I prefer to keep `*` and `&` even for when using `auto` Also, keeping `*` in some cases matter. For example, int x = 42; const auto foo = &x; // foo has type of int* const *foo = 43; // totally fine const auto *bar = &x; // bar has type of const int* *bar = 43; // compilation error REPOSITORY R110 KScreen Library REVISION DETAIL https://phabricator.kde.org/D14081 To: gladhorn, #plasma Cc: zzag, broulik, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart