zzag added inline comments. INLINE COMMENTS
> netwm.cpp:4697-4707 > if (dirty2 & WM2GTKFrameExtents) { > p->gtk_frame_extents = NETStrut(); > > QVector<uint32_t> data = get_array_reply<uint32_t>(p->conn, > cookies[c++], XCB_ATOM_CARDINAL); > if (data.count() == 4) { > p->gtk_frame_extents.left = data[0]; > p->gtk_frame_extents.right = data[1]; Please move WM2GTKFrameExtents above WM2AppMenuObjectPath to match the order in which `cookies` is filled. REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D28112 To: cblack, #plasma, broulik, zzag, #kwin Cc: zzag, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns