c2: make override_redirect only match top-level windows without a client

Fixes #625

Signed-off-by: Yuxuan Shui <yshuiv7@gmail.com>

CHANGELOG.md

@@ -28,6 +28,7 @@
 
 ## Notable changes
 
+* `override_redirect` in rules now only matches top-level windows that doesn't have a client window. Some window managers (e.g. awesome) set override_redirect for all window manager frame windows, causing this rule to match against everything (#625).
 * Marginally improve performance when resizing/opening/closing windows. (#1190)
 * Type and format specifiers are no longer used in rules. These specifiers are what you put after the colon (':') in rules, e.g. the `:32c` in `"_GTK_FRAME_EXTENTS@:32c"`. Now this information is ignored and the property is matched regardless of format or type.
 

src/c2.c

@@ -1588,7 +1588,9 @@ static bool c2_match_once_leaf_int(const struct managed_win *w, const c2_l_t *le
 		case C2_L_PHEIGHTB: predef_target = w->heightb; break;
 		case C2_L_PBDW: predef_target = w->g.border_width; break;
 		case C2_L_PFULLSCREEN: predef_target = w->is_fullscreen; break;
-		case C2_L_POVREDIR: predef_target = w->a.override_redirect; break;
+		case C2_L_POVREDIR:
+			predef_target = w->a.override_redirect && w->client_win == w->base.id;
+			break;
 		case C2_L_PARGB: predef_target = win_has_alpha(w); break;
 		case C2_L_PFOCUSED: predef_target = win_is_focused_raw(w); break;
 		case C2_L_PWMWIN: predef_target = win_is_wmwin(w); break;