1. This is a pointless statement:
+ if (((w->priv->state & MAXIMIZE_STATE) == MAXIMIZE_STATE) ||
+ (w->priv->state & CompWindowStateMaximizedHorzMask) ||
+ (w->priv->state & CompWindowStateMaximizedVertMask))
and is equivalent to:
if (w->priv->state & MAXIMIZE_STATE)
2. If design approved this I think they made a mistake. Compiz already has key combinations unmaximize_window_key and minimize_window_key. It's redundant to add a new combo that overlaps those.
1. This is a pointless statement: MaximizedHorzMa sk) || MaximizedVertMa sk))
+ if (((w->priv->state & MAXIMIZE_STATE) == MAXIMIZE_STATE) ||
+ (w->priv->state & CompWindowState
+ (w->priv->state & CompWindowState
and is equivalent to:
if (w->priv->state & MAXIMIZE_STATE)
2. If design approved this I think they made a mistake. Compiz already has key combinations unmaximize_ window_ key and minimize_ window_ key. It's redundant to add a new combo that overlaps those.