]> code.delx.au - spectrwm/blobdiff - spectrwm.c
oops, killed a d on the way out
[spectrwm] / spectrwm.c
index f3ac3c23e1bdf2b30384dab5faff7f28554d9c27..e202f9c11fb131598a827e99f636dbb87d6639c3 100644 (file)
@@ -81,6 +81,7 @@
 #include <locale.h>
 #include <paths.h>
 #include <pwd.h>
+#include <regex.h>
 #include <signal.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -284,8 +285,8 @@ xcb_atom_t          a_takefocus;
 xcb_atom_t             a_utf8_string;
 xcb_atom_t             a_swm_iconic;
 xcb_atom_t             a_swm_ws;
-volatile sig_atomic_t   running = 1;
-volatile sig_atomic_t   restart_wm = 0;
+volatile sig_atomic_t  running = 1;
+volatile sig_atomic_t  restart_wm = 0;
 xcb_timestamp_t                last_event_time = 0;
 int                    outputs = 0;
 int                    other_wm;
@@ -344,10 +345,10 @@ double                    dialog_ratio = 0.6;
 #define SWM_BAR_JUSTIFY_CENTER (1)
 #define SWM_BAR_JUSTIFY_RIGHT  (2)
 #define SWM_BAR_OFFSET         (4)
-#define SWM_BAR_FONTS          "-*-terminus-medium-*-*-*-12-*-*-*-*-*-*-*," \
-                               "-*-profont-*-*-*-*-12-*-*-*-*-*-*-*,"      \
-                               "-*-times-medium-r-*-*-12-*-*-*-*-*-*-*,"    \
-                               "-misc-fixed-medium-r-*-*-12-*-*-*-*-*-*-*,"  \
+#define SWM_BAR_FONTS          "-*-terminus-medium-*-*-*-12-*-*-*-*-*-*-*,"\
+                               "-*-profont-*-*-*-*-12-*-*-*-*-*-*-*,"\
+                               "-*-times-medium-r-*-*-12-*-*-*-*-*-*-*,"\
+                               "-misc-fixed-medium-r-*-*-12-*-*-*-*-*-*-*,"\
                                "-*-*-*-r-*-*-*-*-*-*-*-*-*-*"
 
 #ifdef X_HAVE_UTF8_STRING
@@ -372,10 +373,11 @@ int                bar_justify = SWM_BAR_JUSTIFY_LEFT;
 char            *bar_format = NULL;
 int             stack_enabled = 1;
 int             clock_enabled = 1;
+int             iconic_enabled = 0;
 int             urgent_enabled = 0;
 char           *clock_format = NULL;
-int             title_name_enabled = 0;
-int             title_class_enabled = 0;
+int             window_class_enabled = 0;
+int             window_instance_enabled = 0;
 int             window_name_enabled = 0;
 int             focus_mode = SWM_FOCUS_DEFAULT;
 int             focus_close = SWM_STACK_BELOW;
@@ -613,8 +615,12 @@ union arg {
 /* quirks */
 struct quirk {
        TAILQ_ENTRY(quirk)      entry;
-       char                    *class;
-       char                    *name;
+       char                    *class;         /* WM_CLASS:class */
+       char                    *instance;      /* WM_CLASS:instance */
+       char                    *name;          /* WM_NAME */
+       regex_t                 regex_class;
+       regex_t                 regex_instance;
+       regex_t                 regex_name;
        unsigned long           quirk;
 #define SWM_Q_FLOAT            (1<<0)  /* float this window */
 #define SWM_Q_TRANSSZ          (1<<1)  /* transiend window size too small */
@@ -666,33 +672,33 @@ struct ewmh_hint {
        char            *name;
        xcb_atom_t      atom;
 } ewmh[SWM_EWMH_HINT_MAX] =    {
-    /* must be in same order as in the enum */
-    {"_NET_ACTIVE_WINDOW", XCB_ATOM_NONE},
-    {"_NET_CLOSE_WINDOW", XCB_ATOM_NONE},
-    {"_NET_MOVERESIZE_WINDOW", XCB_ATOM_NONE},
-    {"_NET_WM_ACTION_ABOVE", XCB_ATOM_NONE},
-    {"_NET_WM_ACTION_CLOSE", XCB_ATOM_NONE},
-    {"_NET_WM_ACTION_FULLSCREEN", XCB_ATOM_NONE},
-    {"_NET_WM_ACTION_MOVE", XCB_ATOM_NONE},
-    {"_NET_WM_ACTION_RESIZE", XCB_ATOM_NONE},
-    {"_NET_WM_ALLOWED_ACTIONS", XCB_ATOM_NONE},
-    {"_NET_WM_NAME", XCB_ATOM_NONE},
-    {"_NET_WM_STATE", XCB_ATOM_NONE},
-    {"_NET_WM_STATE_ABOVE", XCB_ATOM_NONE},
-    {"_NET_WM_STATE_FULLSCREEN", XCB_ATOM_NONE},
-    {"_NET_WM_STATE_HIDDEN", XCB_ATOM_NONE},
-    {"_NET_WM_STATE_MAXIMIZED_HORZ", XCB_ATOM_NONE},
-    {"_NET_WM_STATE_MAXIMIZED_VERT", XCB_ATOM_NONE},
-    {"_NET_WM_STATE_SKIP_PAGER", XCB_ATOM_NONE},
-    {"_NET_WM_STATE_SKIP_TASKBAR", XCB_ATOM_NONE},
-    {"_NET_WM_WINDOW_TYPE", XCB_ATOM_NONE},
-    {"_NET_WM_WINDOW_TYPE_DIALOG", XCB_ATOM_NONE},
-    {"_NET_WM_WINDOW_TYPE_DOCK", XCB_ATOM_NONE},
-    {"_NET_WM_WINDOW_TYPE_NORMAL", XCB_ATOM_NONE},
-    {"_NET_WM_WINDOW_TYPE_SPLASH", XCB_ATOM_NONE},
-    {"_NET_WM_WINDOW_TYPE_TOOLBAR", XCB_ATOM_NONE},
-    {"_NET_WM_WINDOW_TYPE_UTILITY", XCB_ATOM_NONE},
-    {"_SWM_WM_STATE_MANUAL", XCB_ATOM_NONE},
+       /* must be in same order as in the enum */
+       {"_NET_ACTIVE_WINDOW", XCB_ATOM_NONE},
+       {"_NET_CLOSE_WINDOW", XCB_ATOM_NONE},
+       {"_NET_MOVERESIZE_WINDOW", XCB_ATOM_NONE},
+       {"_NET_WM_ACTION_ABOVE", XCB_ATOM_NONE},
+       {"_NET_WM_ACTION_CLOSE", XCB_ATOM_NONE},
+       {"_NET_WM_ACTION_FULLSCREEN", XCB_ATOM_NONE},
+       {"_NET_WM_ACTION_MOVE", XCB_ATOM_NONE},
+       {"_NET_WM_ACTION_RESIZE", XCB_ATOM_NONE},
+       {"_NET_WM_ALLOWED_ACTIONS", XCB_ATOM_NONE},
+       {"_NET_WM_NAME", XCB_ATOM_NONE},
+       {"_NET_WM_STATE", XCB_ATOM_NONE},
+       {"_NET_WM_STATE_ABOVE", XCB_ATOM_NONE},
+       {"_NET_WM_STATE_FULLSCREEN", XCB_ATOM_NONE},
+       {"_NET_WM_STATE_HIDDEN", XCB_ATOM_NONE},
+       {"_NET_WM_STATE_MAXIMIZED_HORZ", XCB_ATOM_NONE},
+       {"_NET_WM_STATE_MAXIMIZED_VERT", XCB_ATOM_NONE},
+       {"_NET_WM_STATE_SKIP_PAGER", XCB_ATOM_NONE},
+       {"_NET_WM_STATE_SKIP_TASKBAR", XCB_ATOM_NONE},
+       {"_NET_WM_WINDOW_TYPE", XCB_ATOM_NONE},
+       {"_NET_WM_WINDOW_TYPE_DIALOG", XCB_ATOM_NONE},
+       {"_NET_WM_WINDOW_TYPE_DOCK", XCB_ATOM_NONE},
+       {"_NET_WM_WINDOW_TYPE_NORMAL", XCB_ATOM_NONE},
+       {"_NET_WM_WINDOW_TYPE_SPLASH", XCB_ATOM_NONE},
+       {"_NET_WM_WINDOW_TYPE_TOOLBAR", XCB_ATOM_NONE},
+       {"_NET_WM_WINDOW_TYPE_UTILITY", XCB_ATOM_NONE},
+       {"_SWM_WM_STATE_MANUAL", XCB_ATOM_NONE},
 };
 
 /* Cursors */
@@ -854,240 +860,237 @@ enum keyfuncid {
 };
 
 struct key {
-        RB_ENTRY(key)           entry;
-        unsigned int            mod;
-        KeySym                  keysym;
-        enum keyfuncid          funcid;
-        char                    *spawn_name;
+       RB_ENTRY(key)   entry;
+       unsigned int    mod;
+       KeySym          keysym;
+       enum keyfuncid  funcid;
+       char            *spawn_name;
 };
 RB_HEAD(key_tree, key);
 
 /* function prototypes */
-void    adjust_font(struct ws_win *);
-void    bar_class_name(char *, size_t, struct swm_region *);
-void    bar_class_title_name(char *, size_t, struct swm_region *);
-void    bar_cleanup(struct swm_region *);
-void    bar_extra_setup(void);
-void    bar_extra_stop(void);
-int     bar_extra_update(void);
-void    bar_fmt(const char *, char *, struct swm_region *, size_t);
-void    bar_fmt_expand(char *, size_t);
-void    bar_draw(void);
-void    bar_print(struct swm_region *, const char *);
-void    bar_print_legacy(struct swm_region *, const char *);
-void    bar_replace(char *, char *, struct swm_region *, size_t);
-void    bar_replace_pad(char *, int *, size_t);
+void   adjust_font(struct ws_win *);
+char   *argsep(char **);
+void   bar_cleanup(struct swm_region *);
+void   bar_extra_setup(void);
+void   bar_extra_stop(void);
+int    bar_extra_update(void);
+void   bar_fmt(const char *, char *, struct swm_region *, size_t);
+void   bar_fmt_expand(char *, size_t);
+void   bar_draw(void);
+void   bar_print(struct swm_region *, const char *);
+void   bar_print_legacy(struct swm_region *, const char *);
+void   bar_replace(char *, char *, struct swm_region *, size_t);
+void   bar_replace_pad(char *, int *, size_t);
 char   *bar_replace_seq(char *, char *, struct swm_region *, size_t *, size_t);
-void    bar_setup(struct swm_region *);
-void    bar_title_name(char *, size_t, struct swm_region *);
-void    bar_toggle(struct swm_region *, union arg *);
-void    bar_urgent(char *, size_t);
-void    bar_window_float(char *, size_t, struct swm_region *);
-void    bar_window_name(char *, size_t, struct swm_region *);
-void    bar_workspace_name(char *, size_t, struct swm_region *);
-void    buttonpress(xcb_button_press_event_t *);
-void    check_conn(void);
-void    clear_keys(void);
-void    clientmessage(xcb_client_message_event_t *);
-void    client_msg(struct ws_win *, xcb_atom_t, xcb_timestamp_t);
-int     conf_load(const char *, int);
-void    configurenotify(xcb_configure_notify_event_t *);
-void    configurerequest(xcb_configure_request_event_t *);
-void    config_win(struct ws_win *, xcb_configure_request_event_t *);
-void    constrain_window(struct ws_win *, struct swm_geometry *, int *);
-int     count_win(struct workspace *, int);
-void    cursors_cleanup(void);
-void    cursors_load(void);
-void    custom_region(char *);
-void    cyclerg(struct swm_region *, union arg *);
-void    cyclews(struct swm_region *, union arg *);
-void    cycle_layout(struct swm_region *, union arg *);
-void    destroynotify(xcb_destroy_notify_event_t *);
-void    dumpwins(struct swm_region *, union arg *);
-int     enable_wm(void);
-void    enternotify(xcb_enter_notify_event_t *);
-void    event_drain(uint8_t);
-void    event_error(xcb_generic_error_t *);
-void    event_handle(xcb_generic_event_t *);
-void    ewmh_autoquirk(struct ws_win *);
-void    ewmh_get_win_state(struct ws_win *);
-int     ewmh_set_win_fullscreen(struct ws_win *, int);
-void    ewmh_update_actions(struct ws_win *);
-void    ewmh_update_win_state(struct ws_win *, xcb_atom_t, long);
+void   bar_setup(struct swm_region *);
+void   bar_toggle(struct swm_region *, union arg *);
+void   bar_urgent(char *, size_t);
+void   bar_window_class(char *, size_t, struct swm_region *);
+void   bar_window_class_instance(char *, size_t, struct swm_region *);
+void   bar_window_float(char *, size_t, struct swm_region *);
+void   bar_window_instance(char *, size_t, struct swm_region *);
+void   bar_window_name(char *, size_t, struct swm_region *);
+void   bar_workspace_name(char *, size_t, struct swm_region *);
+void   buttonpress(xcb_button_press_event_t *);
+void   check_conn(void);
+void   clear_keys(void);
+void   clientmessage(xcb_client_message_event_t *);
+void   client_msg(struct ws_win *, xcb_atom_t, xcb_timestamp_t);
+int    conf_load(const char *, int);
+void   configurenotify(xcb_configure_notify_event_t *);
+void   configurerequest(xcb_configure_request_event_t *);
+void   config_win(struct ws_win *, xcb_configure_request_event_t *);
+void   constrain_window(struct ws_win *, struct swm_geometry *, int *);
+int    count_win(struct workspace *, int);
+void   cursors_cleanup(void);
+void   cursors_load(void);
+void   custom_region(const char *);
+void   cyclerg(struct swm_region *, union arg *);
+void   cyclews(struct swm_region *, union arg *);
+void   cycle_layout(struct swm_region *, union arg *);
+void   destroynotify(xcb_destroy_notify_event_t *);
+void   dumpwins(struct swm_region *, union arg *);
+int    enable_wm(void);
+void   enternotify(xcb_enter_notify_event_t *);
+void   event_drain(uint8_t);
+void   event_error(xcb_generic_error_t *);
+void   event_handle(xcb_generic_event_t *);
+void   ewmh_autoquirk(struct ws_win *);
+void   ewmh_get_win_state(struct ws_win *);
+int    ewmh_set_win_fullscreen(struct ws_win *, int);
+void   ewmh_update_actions(struct ws_win *);
+void   ewmh_update_win_state(struct ws_win *, xcb_atom_t, long);
 char   *expand_tilde(const char *);
-void    expose(xcb_expose_event_t *);
-void    fake_keypress(struct ws_win *, xcb_keysym_t, uint16_t);
+void   expose(xcb_expose_event_t *);
+void   fake_keypress(struct ws_win *, xcb_keysym_t, uint16_t);
+void   floating_toggle(struct swm_region *, union arg *);
+int    floating_toggle_win(struct ws_win *);
+void   focus(struct swm_region *, union arg *);
+void   focus_flush(void);
+void   focus_region(struct swm_region *);
+void   focusrg(struct swm_region *, union arg *);
+void   focus_win(struct ws_win *);
+void   fontset_init(void);
+void   free_window(struct ws_win *);
+void   search_do_resp(void);
+void   search_resp_name_workspace(const char *, size_t);
+void   search_resp_search_window(const char *);
+void   search_resp_search_workspace(const char *);
+void   search_resp_uniconify(const char *, size_t);
+void   search_win(struct swm_region *, union arg *);
+void   search_win_cleanup(void);
+void   search_workspace(struct swm_region *, union arg *);
+void   send_to_rg(struct swm_region *, union arg *);
+void   send_to_ws(struct swm_region *, union arg *);
+void   set_region(struct swm_region *);
+int    setautorun(const char *, const char *, int);
+int    setconfbinding(const char *, const char *, int);
+int    setconfcolor(const char *, const char *, int);
+int    setconfmodkey(const char *, const char *, int);
+int    setconfquirk(const char *, const char *, int);
+int    setconfregion(const char *, const char *, int);
+int    setconfspawn(const char *, const char *, int);
+int    setconfvalue(const char *, const char *, int);
+void   setkeybinding(unsigned int, KeySym, enum keyfuncid, const char *);
+int    setkeymapping(const char *, const char *, int);
+int    setlayout(const char *, const char *, int);
+void   setquirk(const char *, const char *, const char *,unsigned long);
+void   setscreencolor(const char *, int, int);
+void   setspawn(const char *, const char *, int);
+void   setup_ewmh(void);
+void   setup_globals(void);
+void   setup_keys(void);
+void   setup_quirks(void);
+void   setup_screens(void);
+void   setup_spawn(void);
+void   set_child_transient(struct ws_win *, xcb_window_t *);
+void   set_swm_iconic(struct ws_win *, int);
+void   set_win_state(struct ws_win *, uint16_t);
+void   shutdown_cleanup(void);
+void   sighdlr(int);
+void   socket_setnonblock(int);
+void   sort_windows(struct ws_win_list *);
+void   spawn(int, union arg *, int);
+void   spawn_custom(struct swm_region *, union arg *, const char *);
+int    spawn_expand(struct swm_region *, union arg *, const char *, char ***);
+void   spawn_insert(const char *, const char *, int);
+int32_t        get_swm_iconic(struct ws_win *);
+char   *get_win_name(xcb_window_t);
+void   get_wm_protocols(struct ws_win *);
+int    get_ws_idx(xcb_window_t);
+void   grabbuttons(struct ws_win *);
+void   grabkeys(void);
+void   grab_windows(void);
+void   iconify(struct swm_region *, union arg *);
+int    isxlfd(char *);
+void   keypress(xcb_key_press_event_t *);
+int    key_cmp(struct key *, struct key *);
+void   key_insert(unsigned int, KeySym, enum keyfuncid, const char *);
+void   key_remove(struct key *);
+void   key_replace(struct key *, unsigned int, KeySym, enum keyfuncid,
+           const char *);
+void   kill_bar_extra_atexit(void);
+void   kill_refs(struct ws_win *);
+void   load_float_geom(struct ws_win *);
+void   map_window(struct ws_win *, xcb_window_t);
+void   mapnotify(xcb_map_notify_event_t *);
+void   mappingnotify(xcb_mapping_notify_event_t *);
+void   maprequest(xcb_map_request_event_t *);
+void   motionnotify(xcb_motion_notify_event_t *);
+void   move(struct ws_win *, union arg *);
+void   move_step(struct swm_region *, union arg *);
+uint32_t name_to_pixel(int, const char *);
+void   name_workspace(struct swm_region *, union arg *);
+void   new_region(struct swm_screen *, int, int, int, int);
+int    parsekeys(const char *, unsigned int, unsigned int *, KeySym *);
+int    parsequirks(const char *, unsigned long *);
+int    parse_rgb(const char *, uint16_t *, uint16_t *, uint16_t *);
+void   pressbutton(struct swm_region *, union arg *);
+void   priorws(struct swm_region *, union arg *);
+void   propertynotify(xcb_property_notify_event_t *);
+void   quirk_free(struct quirk *);
+void   quirk_insert(const char *, const char *, const char *,unsigned long);
+void   quirk_remove(struct quirk *);
+void   quirk_replace(struct quirk *, const char *, const char *, const char *,
+           unsigned long);
+void   quit(struct swm_region *, union arg *);
+void   raise_toggle(struct swm_region *, union arg *);
+void   region_containment(struct ws_win *, struct swm_region *, int);
+void   regionize(struct ws_win *, int, int);
+void   resize(struct ws_win *, union arg *);
+void   resize_step(struct swm_region *, union arg *);
+void   restart(struct swm_region *, union arg *);
+void   screenchange(xcb_randr_screen_change_notify_event_t *);
+void   scan_xrandr(int);
+void   spawn_remove(struct spawn_prog *);
+void   spawn_replace(struct spawn_prog *, const char *, const char *, int);
+void   spawn_select(struct swm_region *, union arg *, const char *, int *);
+void   stack_config(struct swm_region *, union arg *);
+void   stack_floater(struct ws_win *, struct swm_region *);
+void   stack_master(struct workspace *, struct swm_geometry *, int, int);
+void   store_float_geom(struct ws_win *);
+char   *strdupsafe(const char *);
+void   swapwin(struct swm_region *, union arg *);
+void   switchws(struct swm_region *, union arg *);
+void   teardown_ewmh(void);
+void   unescape_selector(char *);
+void   unfocus_win(struct ws_win *);
+void   uniconify(struct swm_region *, union arg *);
+void   unmanage_window(struct ws_win *);
+void   unmapnotify(xcb_unmap_notify_event_t *);
+void   unmap_all(void);
+void   unmap_window(struct ws_win *);
+void   updatenumlockmask(void);
+void   update_modkey(unsigned int);
+void   update_window(struct ws_win *);
+void   validate_spawns(void);
+int    validate_win(struct ws_win *);
+int    validate_ws(struct workspace *);
+void   version(struct swm_region *, union arg *);
+void   win_to_ws(struct ws_win *, int, int);
+pid_t  window_get_pid(xcb_window_t);
+void   wkill(struct swm_region *, union arg *);
+void   workaround(void);
+void   xft_init(struct swm_region *);
+void   _add_startup_exception(const char *, va_list);
+void   add_startup_exception(const char *, ...);
+#ifdef SWM_DEBUG
+void   focusin(xcb_focus_in_event_t *);
+void   focusout(xcb_focus_out_event_t *);
+void   leavenotify(xcb_leave_notify_event_t *);
+void   print_win_geom(xcb_window_t);
+#endif
+
 struct pid_e   *find_pid(pid_t);
 struct ws_win  *find_unmanaged_window(xcb_window_t);
 struct ws_win  *find_window(xcb_window_t);
-void    floating_toggle(struct swm_region *, union arg *);
-int     floating_toggle_win(struct ws_win *);
-void    focus(struct swm_region *, union arg *);
-#ifdef SWM_DEBUG
-void    focusin(xcb_focus_in_event_t *);
-void    focusout(xcb_focus_out_event_t *);
-#endif
-void    focus_flush(void);
-void    focus_region(struct swm_region *);
-void    focusrg(struct swm_region *, union arg *);
-void    focus_win(struct ws_win *);
-void    fontset_init(void);
-void    free_window(struct ws_win *);
-xcb_atom_t get_atom_from_string(const char *);
-#ifdef SWM_DEBUG
-char   *get_atom_name(xcb_atom_t);
-#endif
-struct ws_win   *get_focus_magic(struct ws_win *);
-struct ws_win   *get_focus_prev(struct ws_win *);
-#ifdef SWM_DEBUG
-char   *get_notify_detail_label(uint8_t);
-char   *get_notify_mode_label(uint8_t);
-#endif
+struct ws_win  *get_focus_magic(struct ws_win *);
+struct ws_win  *get_focus_prev(struct ws_win *);
 struct ws_win  *get_pointer_win(xcb_window_t);
 struct ws_win  *get_region_focus(struct swm_region *);
-int     get_region_index(struct swm_region *);
+struct ws_win  *manage_window(xcb_window_t, uint16_t);
+int            get_region_index(struct swm_region *);
+xcb_atom_t     get_atom_from_string(const char *);
 xcb_screen_t   *get_screen(int);
-xcb_window_t    get_sibling(struct ws_win *, int);
-int     get_screen_count(void);
-#ifdef SWM_DEBUG
-char   *get_stack_mode_name(uint8_t);
-#endif
-int32_t         get_swm_iconic(struct ws_win *);
-char   *get_win_name(xcb_window_t);
-void    get_wm_protocols(struct ws_win *);
-int     get_ws_idx(xcb_window_t);
-uint32_t getstate(xcb_window_t);
-void    grabbuttons(struct ws_win *);
-void    grabkeys(void);
-void    grab_windows(void);
-void    iconify(struct swm_region *, union arg *);
-int     isxlfd(char *);
-void    keypress(xcb_key_press_event_t *);
-int     key_cmp(struct key *, struct key *);
-void    key_insert(unsigned int, KeySym, enum keyfuncid, const char *);
+xcb_window_t   get_sibling(struct ws_win *, int);
+int            get_screen_count(void);
+uint32_t       getstate(xcb_window_t);
 struct key     *key_lookup(unsigned int, KeySym);
-void    key_remove(struct key *);
-void    key_replace(struct key *, unsigned int, KeySym, enum keyfuncid,
-            const char *);
-void    kill_bar_extra_atexit(void);
-void    kill_refs(struct ws_win *);
-#ifdef SWM_DEBUG
-void    leavenotify(xcb_leave_notify_event_t *);
-#endif
-void    load_float_geom(struct ws_win *);
-struct ws_win  *manage_window(xcb_window_t, uint16_t);
-void    map_window(struct ws_win *, xcb_window_t);
-void    mapnotify(xcb_map_notify_event_t *);
-void    mappingnotify(xcb_mapping_notify_event_t *);
-void    maprequest(xcb_map_request_event_t *);
-void    motionnotify(xcb_motion_notify_event_t *);
-void    move(struct ws_win *, union arg *);
-void    move_step(struct swm_region *, union arg *);
-uint32_t name_to_pixel(int, const char *);
-void    name_workspace(struct swm_region *, union arg *);
-void    new_region(struct swm_screen *, int, int, int, int);
-int     parsekeys(char *, unsigned int, unsigned int *, KeySym *);
-int     parsequirks(char *, unsigned long *);
-int     parse_rgb(const char *, uint16_t *, uint16_t *, uint16_t *);
-void    pressbutton(struct swm_region *, union arg *);
-void    priorws(struct swm_region *, union arg *);
 #ifdef SWM_DEBUG
-void    print_win_geom(xcb_window_t);
+char           *get_atom_name(xcb_atom_t);
+char           *get_notify_detail_label(uint8_t);
+char           *get_notify_mode_label(uint8_t);
+char           *get_stack_mode_name(uint8_t);
 #endif
-void    propertynotify(xcb_property_notify_event_t *);
-void    quirk_insert(const char *, const char *, unsigned long);
-void    quirk_remove(struct quirk *);
-void    quirk_replace(struct quirk *, const char *, const char *,
-            unsigned long);
-void    quit(struct swm_region *, union arg *);
-void    raise_toggle(struct swm_region *, union arg *);
-void    region_containment(struct ws_win *, struct swm_region *, int);
+
 struct swm_region      *region_under(struct swm_screen *, int, int);
-void    regionize(struct ws_win *, int, int);
-void    resize(struct ws_win *, union arg *);
-void    resize_step(struct swm_region *, union arg *);
-void    restart(struct swm_region *, union arg *);
 struct swm_region      *root_to_region(xcb_window_t, int);
-void    screenchange(xcb_randr_screen_change_notify_event_t *);
-void    scan_xrandr(int);
-void    search_do_resp(void);
-void    search_resp_name_workspace(const char *, size_t);
-void    search_resp_search_window(const char *);
-void    search_resp_search_workspace(const char *);
-void    search_resp_uniconify(const char *, size_t);
-void    search_win(struct swm_region *, union arg *);
-void    search_win_cleanup(void);
-void    search_workspace(struct swm_region *, union arg *);
-void    send_to_rg(struct swm_region *, union arg *);
-void    send_to_ws(struct swm_region *, union arg *);
-void    set_region(struct swm_region *);
-int     setautorun(char *, char *, int);
-int     setconfbinding(char *, char *, int);
-int     setconfcolor(char *, char *, int);
-int     setconfmodkey(char *, char *, int);
-int     setconfquirk(char *, char *, int);
-int     setconfregion(char *, char *, int);
-int     setconfspawn(char *, char *, int);
-int     setconfvalue(char *, char *, int);
-void    setkeybinding(unsigned int, KeySym, enum keyfuncid, const char *);
-int     setkeymapping(char *, char *, int);
-int     setlayout(char *, char *, int);
-void    setquirk(const char *, const char *, unsigned long);
-void    setscreencolor(char *, int, int);
-void    setspawn(const char *, const char *, int);
-void    setup_ewmh(void);
-void    setup_globals(void);
-void    setup_keys(void);
-void    setup_quirks(void);
-void    setup_screens(void);
-void    setup_spawn(void);
-void    set_child_transient(struct ws_win *, xcb_window_t *);
-void    set_swm_iconic(struct ws_win *, int);
-void    set_win_state(struct ws_win *, uint16_t);
-void    shutdown_cleanup(void);
-void    sighdlr(int);
-void    socket_setnonblock(int);
-void    sort_windows(struct ws_win_list *);
-void    spawn(int, union arg *, int);
-void    spawn_custom(struct swm_region *, union arg *, const char *);
-int     spawn_expand(struct swm_region *, union arg *, const char *, char ***);
-void    spawn_insert(const char *, const char *, int);
 struct spawn_prog      *spawn_find(const char *);
-void    spawn_remove(struct spawn_prog *);
-void    spawn_replace(struct spawn_prog *, const char *, const char *, int);
-void    spawn_select(struct swm_region *, union arg *, const char *, int *);
-void    stack_config(struct swm_region *, union arg *);
-void    stack_floater(struct ws_win *, struct swm_region *);
-void    stack_master(struct workspace *, struct swm_geometry *, int, int);
-void    store_float_geom(struct ws_win *);
-char   *strdupsafe(const char *);
-void    swapwin(struct swm_region *, union arg *);
-void    switchws(struct swm_region *, union arg *);
-void    teardown_ewmh(void);
-void    unfocus_win(struct ws_win *);
-void    uniconify(struct swm_region *, union arg *);
-void    unmanage_window(struct ws_win *);
-void    unmapnotify(xcb_unmap_notify_event_t *);
-void    unmap_all(void);
-void    unmap_window(struct ws_win *);
-void    updatenumlockmask(void);
-void    update_modkey(unsigned int);
-void    update_window(struct ws_win *);
-void    validate_spawns(void);
-int     validate_win(struct ws_win *);
-int     validate_ws(struct workspace *);
-void    version(struct swm_region *, union arg *);
-void    win_to_ws(struct ws_win *, int, int);
-pid_t   window_get_pid(xcb_window_t);
-void    wkill(struct swm_region *, union arg *);
-void    workaround(void);
-void    xft_init(struct swm_region *);
-void    _add_startup_exception(const char *, va_list);
-void    add_startup_exception(const char *, ...);
 
 RB_PROTOTYPE(key_tree, key, entry, key_cmp);
 RB_GENERATE(key_tree, key, entry, key_cmp);
-struct key_tree                 keys;
+struct key_tree                keys;
 
 void
 cursors_load(void)
@@ -1131,11 +1134,11 @@ cursors_cleanup(void)
 char *
 expand_tilde(const char *s)
 {
-       struct passwd           *ppwd;
-       int                     i;
+       struct passwd           *ppwd;
+       int                     i;
        long                    max;
-       char                    *user;
-       const char              *sc = s;
+       char                    *user;
+       const char              *sc = s;
        char                    *result;
 
        if (s == NULL)
@@ -1468,9 +1471,9 @@ ewmh_update_actions(struct ws_win *win)
            ewmh[_NET_WM_ALLOWED_ACTIONS].atom, XCB_ATOM_ATOM, 32, 1, actions);
 }
 
-#define _NET_WM_STATE_REMOVE   0    /* remove/unset property */
-#define _NET_WM_STATE_ADD      1    /* add/set property */
-#define _NET_WM_STATE_TOGGLE   2    /* toggle property */
+#define _NET_WM_STATE_REMOVE   0       /* remove/unset property */
+#define _NET_WM_STATE_ADD      1       /* add/set property */
+#define _NET_WM_STATE_TOGGLE   2       /* toggle property */
 
 void
 ewmh_update_win_state(struct ws_win *win, xcb_atom_t state, long action)
@@ -1754,7 +1757,7 @@ name_to_pixel(int sidx, const char *colorname)
 }
 
 void
-setscreencolor(char *val, int i, int c)
+setscreencolor(const char *val, int i, int c)
 {
        int     num_screens;
 
@@ -1803,7 +1806,7 @@ plain_stacker(struct workspace *ws)
 }
 
 void
-custom_region(char *val)
+custom_region(const char *val)
 {
        unsigned int                    x, y, w, h;
        int                             sidx, num_screens;
@@ -1905,12 +1908,12 @@ bar_print_legacy(struct swm_region *r, const char *s)
 void
 bar_print(struct swm_region *r, const char *s)
 {
-       size_t                          len;
-       xcb_rectangle_t                 rect;
-       uint32_t                        gcv[1];
-       int32_t                         x = 0;
-       XGlyphInfo                      info;
-       XftDraw                         *draw;
+       size_t          len;
+       xcb_rectangle_t rect;
+       uint32_t        gcv[1];
+       int32_t         x = 0;
+       XGlyphInfo      info;
+       XftDraw         *draw;
 
        len = strlen(s);
 
@@ -1973,7 +1976,7 @@ bar_extra_stop(void)
 }
 
 void
-bar_class_name(char *s, size_t sz, struct swm_region *r)
+bar_window_class(char *s, size_t sz, struct swm_region *r)
 {
        if (r == NULL || r->ws == NULL || r->ws->focus == NULL)
                return;
@@ -1982,7 +1985,7 @@ bar_class_name(char *s, size_t sz, struct swm_region *r)
 }
 
 void
-bar_title_name(char *s, size_t sz, struct swm_region *r)
+bar_window_instance(char *s, size_t sz, struct swm_region *r)
 {
        if (r == NULL || r->ws == NULL || r->ws->focus == NULL)
                return;
@@ -1991,14 +1994,14 @@ bar_title_name(char *s, size_t sz, struct swm_region *r)
 }
 
 void
-bar_class_title_name(char *s, size_t sz, struct swm_region *r)
+bar_window_class_instance(char *s, size_t sz, struct swm_region *r)
 {
        if (r == NULL || r->ws == NULL || r->ws->focus == NULL)
                return;
 
-       bar_class_name(s, sz, r);
+       bar_window_class(s, sz, r);
        strlcat(s, ":", sz);
-       bar_title_name(s, sz, r);
+       bar_window_instance(s, sz, r);
 }
 
 void
@@ -2017,9 +2020,8 @@ bar_window_name(char *s, size_t sz, struct swm_region *r)
 
        if (r == NULL || r->ws == NULL || r->ws->focus == NULL)
                return;
-       if ((title = get_win_name(r->ws->focus->id)) == NULL)
-               return;
 
+       title = get_win_name(r->ws->focus->id);
        strlcat(s, title, sz);
        free(title);
 }
@@ -2071,6 +2073,8 @@ bar_workspace_name(char *s, size_t sz, struct swm_region *r)
 void
 bar_fmt(const char *fmtexp, char *fmtnew, struct swm_region *r, size_t sz)
 {
+       struct ws_win           *w;
+
        /* if format provided, just copy the buffers */
        if (bar_format != NULL) {
                strlcpy(fmtnew, fmtexp, sz);
@@ -2088,6 +2092,15 @@ bar_fmt(const char *fmtexp, char *fmtnew, struct swm_region *r, size_t sz)
        /* only show the workspace name if there's actually one */
        if (r != NULL && r->ws != NULL && r->ws->name != NULL)
                strlcat(fmtnew, "<+D>", sz);
+
+       /* If enabled, only show the iconic count if there are iconic wins. */
+       if (iconic_enabled && r != NULL && r->ws != NULL)
+               TAILQ_FOREACH(w, &r->ws->winlist, entry)
+                       if (w->iconic) {
+                               strlcat(fmtnew, "{+M}", sz);
+                               break;
+                       }
+
        strlcat(fmtnew, "+3<", sz);
 
        if (clock_enabled) {
@@ -2099,16 +2112,16 @@ bar_fmt(const char *fmtexp, char *fmtnew, struct swm_region *r, size_t sz)
        if (urgent_enabled)
                strlcat(fmtnew, "* +U*+4<", sz);
 
-       if (title_class_enabled) {
+       if (window_class_enabled) {
                strlcat(fmtnew, "+C", sz);
-               if (!title_name_enabled)
+               if (!window_instance_enabled)
                        strlcat(fmtnew, "+4<", sz);
        }
 
        /* checks needed by the colon and floating strlcat(3) calls below */
        if (r != NULL && r->ws != NULL && r->ws->focus != NULL) {
-               if (title_name_enabled) {
-                       if (title_class_enabled)
+               if (window_instance_enabled) {
+                       if (window_class_enabled)
                                strlcat(fmtnew, ":", sz);
                        strlcat(fmtnew, "+T+4<", sz);
                }
@@ -2137,9 +2150,10 @@ char *
 bar_replace_seq(char *fmt, char *fmtrep, struct swm_region *r, size_t *offrep,
     size_t sz)
 {
+       struct ws_win           *w;
        char                    *ptr;
        char                    tmp[SWM_BAR_MAX];
-       int                     limit, size;
+       int                     limit, size, count;
        size_t                  len;
 
        /* reset strlcat(3) buffer */
@@ -2166,7 +2180,7 @@ bar_replace_seq(char *fmt, char *fmtrep, struct swm_region *r, size_t *offrep,
                snprintf(tmp, sizeof tmp, "%s", bar_ext);
                break;
        case 'C':
-               bar_class_name(tmp, sizeof tmp, r);
+               bar_window_class(tmp, sizeof tmp, r);
                break;
        case 'D':
                bar_workspace_name(tmp, sizeof tmp, r);
@@ -2177,17 +2191,25 @@ bar_replace_seq(char *fmt, char *fmtrep, struct swm_region *r, size_t *offrep,
        case 'I':
                snprintf(tmp, sizeof tmp, "%d", r->ws->idx + 1);
                break;
+       case 'M':
+               count = 0;
+               TAILQ_FOREACH(w, &r->ws->winlist, entry)
+                       if (w->iconic)
+                               ++count;
+
+               snprintf(tmp, sizeof tmp, "%d", count);
+               break;
        case 'N':
                snprintf(tmp, sizeof tmp, "%d", r->s->idx + 1);
                break;
        case 'P':
-               bar_class_title_name(tmp, sizeof tmp, r);
+               bar_window_class_instance(tmp, sizeof tmp, r);
                break;
        case 'S':
                snprintf(tmp, sizeof tmp, "%s", r->ws->stacker);
                break;
        case 'T':
-               bar_title_name(tmp, sizeof tmp, r);
+               bar_window_instance(tmp, sizeof tmp, r);
                break;
        case 'U':
                bar_urgent(tmp, sizeof tmp);
@@ -3652,7 +3674,7 @@ cyclews(struct swm_region *r, union arg *args)
 void
 priorws(struct swm_region *r, union arg *args)
 {
-       union arg               a;
+       union arg       a;
 
        (void)args;
 
@@ -4914,27 +4936,25 @@ get_win_name(xcb_window_t win)
        c = xcb_get_property(conn, 0, win, ewmh[_NET_WM_NAME].atom,
            XCB_GET_PROPERTY_TYPE_ANY, 0, UINT_MAX);
        r = xcb_get_property_reply(conn, c, NULL);
-       if (r) {
-               if (r->type == XCB_NONE) {
-                       free(r);
-                       /* Use WM_NAME instead; no UTF-8. */
-                       c = xcb_get_property(conn, 0, win, XCB_ATOM_WM_NAME,
-                           XCB_GET_PROPERTY_TYPE_ANY, 0, UINT_MAX);
-                       r = xcb_get_property_reply(conn, c, NULL);
-                       if (r == NULL)
-                               return (NULL);
-                       if (r->type == XCB_NONE) {
-                               free(r);
-                               return (NULL);
-                       }
-               }
-               if (r->length > 0)
-                       name = strndup(xcb_get_property_value(r),
-                           xcb_get_property_value_length(r));
-
+       if (r && r->type == XCB_NONE) {
                free(r);
+               /* Use WM_NAME instead; no UTF-8. */
+               c = xcb_get_property(conn, 0, win, XCB_ATOM_WM_NAME,
+                   XCB_GET_PROPERTY_TYPE_ANY, 0, UINT_MAX);
+               r = xcb_get_property_reply(conn, c, NULL);
        }
 
+       if (r && r->type != XCB_NONE && r->length > 0)
+               name = strndup(xcb_get_property_value(r),
+                   xcb_get_property_value_length(r));
+       else
+               name = strdup("");
+
+       if (name == NULL)
+               err(1, "get_win_name: failed to allocate memory.");
+
+       free(r);
+
        return (name);
 }
 
@@ -4977,8 +4997,6 @@ uniconify(struct swm_region *r, union arg *args)
                        continue;
 
                name = get_win_name(win->id);
-               if (name == NULL)
-                       continue;
                fprintf(lfile, "%s.%u\n", name, win->id);
                free(name);
        }
@@ -5172,8 +5190,6 @@ search_resp_uniconify(const char *resp, size_t len)
                if (!win->iconic)
                        continue;
                name = get_win_name(win->id);
-               if (name == NULL)
-                       continue;
                if (asprintf(&s, "%s.%u", name, win->id) == -1) {
                        free(name);
                        continue;
@@ -6234,24 +6250,73 @@ spawn_select(struct swm_region *r, union arg *args, const char *spawn_name,
        free(real_args);
 }
 
+/* Argument tokenizer. */
+char *
+argsep(char **sp) {
+       int                     single_quoted = 0, double_quoted = 0;
+       char                    *arg, *cp, *next;
+
+       if (*sp == NULL)
+               return NULL;
+
+       /* Eat and move characters until end of argument is found. */
+       for (arg = next = cp = *sp; *cp != '\0'; ++cp) {
+               if (!double_quoted && *cp == '\'') {
+                       /* Eat single-quote. */
+                       single_quoted = !single_quoted;
+               } else if (!single_quoted && *cp == '"') {
+                       /* Eat double-quote. */
+                       double_quoted = !double_quoted;
+               } else if (!single_quoted && *cp == '\\' && *(cp + 1) == '"') {
+                       /* Eat backslash; copy escaped character to arg. */
+                       *next++ = *(++cp);
+               } else if (!single_quoted && !double_quoted && *cp == '\\' &&
+                   (*(cp + 1) == '\'' || *(cp + 1) == ' ')) {
+                       /* Eat backslash; move escaped character. */
+                       *next++ = *(++cp);
+               } else if (!single_quoted && !double_quoted &&
+                   (*cp == ' ' || *cp == '\t')) {
+                       /* Terminate argument. */
+                       *next++ = '\0';
+                       /* Point sp to beginning of next argument. */
+                       *sp = ++cp;
+                       break;
+               } else {
+                       /* Move regular character. */
+                       *next++ = *cp;
+               }
+       }
+
+       /* Terminate argument if end of string. */
+       if (*cp == '\0') {
+               *next = '\0';
+               *sp = NULL;
+       }
+
+       return arg;
+}
+
 void
 spawn_insert(const char *name, const char *args, int flags)
 {
-       char                    *arg, *cp, *ptr;
        struct spawn_prog       *sp;
+       char                    *arg, *dup, *ptr;
+
+       DNPRINTF(SWM_D_SPAWN, "spawn_insert: %s[%s]\n", name, args);
 
-       DNPRINTF(SWM_D_SPAWN, "spawn_insert: %s\n", name);
+       if (args == NULL || *args == '\0')
+               return;
 
        if ((sp = calloc(1, sizeof *sp)) == NULL)
                err(1, "spawn_insert: calloc");
        if ((sp->name = strdup(name)) == NULL)
                err(1, "spawn_insert: strdup");
 
-       /* convert the arguments to an argument list */
-       if ((ptr = cp = strdup(args)) == NULL)
+       /* Convert the arguments to an argument list. */
+       if ((ptr = dup = strdup(args)) == NULL)
                err(1, "spawn_insert: strdup");
-       while ((arg = strsep(&ptr, " \t")) != NULL) {
-               /* empty field; skip it */
+       while ((arg = argsep(&ptr)) != NULL) {
+               /* Null argument; skip it. */
                if (*arg == '\0')
                        continue;
 
@@ -6262,10 +6327,11 @@ spawn_insert(const char *name, const char *args, int flags)
                if ((sp->argv[sp->argc - 1] = strdup(arg)) == NULL)
                        err(1, "spawn_insert: strdup");
        }
-       free(cp);
+       free(dup);
 
        sp->flags = flags;
 
+       DNPRINTF(SWM_D_SPAWN, "arg %d: [%s]\n", sp->argc, sp->argv[sp->argc-1]);
        TAILQ_INSERT_TAIL(&spawns, sp, entry);
        DNPRINTF(SWM_D_SPAWN, "spawn_insert: leave\n");
 }
@@ -6322,10 +6388,13 @@ setspawn(const char *name, const char *args, int flags)
 }
 
 int
-setconfspawn(char *selector, char *value, int flags)
+setconfspawn(const char *selector, const char *value, int flags)
 {
        char            *args;
 
+       if (selector == NULL || strlen(selector) == 0)
+               return (1);
+
        args = expand_tilde(value);
 
        DNPRINTF(SWM_D_SPAWN, "setconfspawn: [%s] [%s]\n", selector, args);
@@ -6414,10 +6483,11 @@ setup_spawn(void)
 #define SWM_MODNAME_SIZE       32
 #define        SWM_KEY_WS              "\n+ \t"
 int
-parsekeys(char *keystr, unsigned int currmod, unsigned int *mod, KeySym *ks)
+parsekeys(const char *keystr, unsigned int currmod, unsigned int *mod, KeySym *ks)
 {
-       char                    *cp, *name;
+       char                    *str, *cp, *name;
        KeySym                  uks;
+
        DNPRINTF(SWM_D_KEY, "parsekeys: enter [%s]\n", keystr);
        if (mod == NULL || ks == NULL) {
                DNPRINTF(SWM_D_KEY, "parsekeys: no mod or key vars\n");
@@ -6427,7 +6497,10 @@ parsekeys(char *keystr, unsigned int currmod, unsigned int *mod, KeySym *ks)
                DNPRINTF(SWM_D_KEY, "parsekeys: no keystr\n");
                return (1);
        }
-       cp = keystr;
+
+       if ((cp = str = strdup(keystr)) == NULL)
+               err(1, "parsekeys: strdup");
+
        *ks = XCB_NO_SYMBOL;
        *mod = 0;
        while ((name = strsep(&cp, SWM_KEY_WS)) != NULL) {
@@ -6455,10 +6528,13 @@ parsekeys(char *keystr, unsigned int currmod, unsigned int *mod, KeySym *ks)
                                DNPRINTF(SWM_D_KEY,
                                    "parsekeys: invalid key %s\n",
                                    name);
+                               free(str);
                                return (1);
                        }
                }
        }
+
+       free(str);
        DNPRINTF(SWM_D_KEY, "parsekeys: leave ok\n");
        return (0);
 }
@@ -6557,7 +6633,7 @@ setkeybinding(unsigned int mod, KeySym ks, enum keyfuncid kfid,
 }
 
 int
-setconfbinding(char *selector, char *value, int flags)
+setconfbinding(const char *selector, const char *value, int flags)
 {
        enum keyfuncid          kfid;
        unsigned int            mod;
@@ -6567,8 +6643,9 @@ setconfbinding(char *selector, char *value, int flags)
        /* suppress unused warning since var is needed */
        (void)flags;
 
-       DNPRINTF(SWM_D_KEY, "setconfbinding: enter\n");
-       if (selector == NULL) {
+       DNPRINTF(SWM_D_KEY, "setconfbinding: enter selector: [%s], "
+           "value: [%s]\n", selector, value);
+       if (selector == NULL || strlen(selector) == 0) {
                DNPRINTF(SWM_D_KEY, "setconfbinding: unbind %s\n", value);
                if (parsekeys(value, mod_key, &mod, &ks) == 0) {
                        kfid = KF_INVALID;
@@ -6745,7 +6822,7 @@ clear_keys(void)
 }
 
 int
-setkeymapping(char *selector, char *value, int flags)
+setkeymapping(const char *selector, const char *value, int flags)
 {
        char                    *keymapping_file;
 
@@ -6790,9 +6867,11 @@ updatenumlockmask(void)
                                    + j];
                                keycode = xcb_key_symbols_get_keycode(syms,
                                                XK_Num_Lock);
-                               if (kc == *keycode)
-                                       numlockmask = (1 << i);
-                               free(keycode);
+                               if (keycode) {
+                                       if (kc == *keycode)
+                                               numlockmask = (1 << i);
+                                       free(keycode);
+                               }
                        }
                }
                free(modmap_r);
@@ -6886,15 +6965,18 @@ const char *quirkname[] = {
 /* SWM_Q_WS: retain '|' for back compat for now (2009-08-11) */
 #define        SWM_Q_WS                "\n|+ \t"
 int
-parsequirks(char *qstr, unsigned long *quirk)
+parsequirks(const char *qstr, unsigned long *quirk)
 {
-       char                    *cp, *name;
+       char                    *str, *cp, *name;
        int                     i;
 
-       if (quirk == NULL)
+       if (quirk == NULL || qstr == NULL)
                return (1);
 
-       cp = qstr;
+       if ((str = strdup(qstr)) == NULL)
+               err(1, "parsequirks: strdup");
+
+       cp = str;
        *quirk = 0;
        while ((name = strsep(&cp, SWM_Q_WS)) != NULL) {
                if (cp)
@@ -6906,6 +6988,7 @@ parsequirks(char *qstr, unsigned long *quirk)
                                    "parsequirks: %s\n", name);
                                if (i == 0) {
                                        *quirk = 0;
+                                       free(str);
                                        return (0);
                                }
                                *quirk |= 1 << (i-1);
@@ -6915,30 +6998,73 @@ parsequirks(char *qstr, unsigned long *quirk)
                if (i >= LENGTH(quirkname)) {
                        DNPRINTF(SWM_D_QUIRK,
                            "parsequirks: invalid quirk [%s]\n", name);
+                       free(str);
                        return (1);
                }
        }
+
+       free(str);
        return (0);
 }
 
 void
-quirk_insert(const char *class, const char *name, unsigned long quirk)
+quirk_insert(const char *class, const char *instance, const char *name,
+    unsigned long quirk)
 {
        struct quirk            *qp;
+       int                     failed = 0;
+       char                    *str;
 
-       DNPRINTF(SWM_D_QUIRK, "quirk_insert: %s:%s [%lu]\n", class, name,
-           quirk);
+       DNPRINTF(SWM_D_QUIRK, "quirk_insert: class: %s, instance: %s, name: %s,"
+           " value: %lu\n", class, instance, name, quirk);
 
        if ((qp = malloc(sizeof *qp)) == NULL)
                err(1, "quirk_insert: malloc");
+
        if ((qp->class = strdup(class)) == NULL)
                err(1, "quirk_insert: strdup");
+       if ((qp->instance = strdup(instance)) == NULL)
+               err(1, "quirk_insert: strdup");
        if ((qp->name = strdup(name)) == NULL)
                err(1, "quirk_insert: strdup");
 
-       qp->quirk = quirk;
-       TAILQ_INSERT_TAIL(&quirks, qp, entry);
-
+       if (asprintf(&str, "^%s$", class) == -1)
+               err(1, "quirk_insert: asprintf");
+       if (regcomp(&qp->regex_class, str, REG_EXTENDED | REG_NOSUB)) {
+               add_startup_exception("regex failed to compile quirk 'class' "
+                   "field: %s", class);
+               failed = 1;
+       }
+       DNPRINTF(SWM_D_QUIRK, "quirk_insert: compiled: %s\n", str);
+       free(str);
+
+       if (asprintf(&str, "^%s$", instance) == -1)
+               err(1, "quirk_insert: asprintf");
+       if (regcomp(&qp->regex_instance, str, REG_EXTENDED | REG_NOSUB)) {
+               add_startup_exception("regex failed to compile quirk 'instance'"
+                   " field: %s", instance);
+               failed = 1;
+       }
+       DNPRINTF(SWM_D_QUIRK, "quirk_insert: compiled: %s\n", str);
+       free(str);
+
+       if (asprintf(&str, "^%s$", name) == -1)
+               err(1, "quirk_insert: asprintf");
+       if (regcomp(&qp->regex_name, str, REG_EXTENDED | REG_NOSUB)) {
+               add_startup_exception("regex failed to compile quirk 'name' "
+                   "field: %s", name);
+               failed = 1;
+       }
+       DNPRINTF(SWM_D_QUIRK, "quirk_insert: compiled: %s\n", str);
+       free(str);
+
+       if (failed) {
+               DNPRINTF(SWM_D_QUIRK, "quirk_insert: regex error; skipping.\n");
+               quirk_free(qp);
+       } else {
+               qp->quirk = quirk;
+               TAILQ_INSERT_TAIL(&quirks, qp, entry);
+       }
        DNPRINTF(SWM_D_QUIRK, "quirk_insert: leave\n");
 }
 
@@ -6949,42 +7075,54 @@ quirk_remove(struct quirk *qp)
            qp->name, qp->quirk);
 
        TAILQ_REMOVE(&quirks, qp, entry);
+       quirk_free(qp);
+
+       DNPRINTF(SWM_D_QUIRK, "quirk_remove: leave\n");
+}
+
+void
+quirk_free(struct quirk *qp)
+{
+       regfree(&qp->regex_class);
+       regfree(&qp->regex_instance);
+       regfree(&qp->regex_name);
        free(qp->class);
+       free(qp->instance);
        free(qp->name);
        free(qp);
-
-       DNPRINTF(SWM_D_QUIRK, "quirk_remove: leave\n");
 }
 
 void
-quirk_replace(struct quirk *qp, const char *class, const char *name,
-    unsigned long quirk)
+quirk_replace(struct quirk *qp, const char *class, const char *instance,
+    const char *name, unsigned long quirk)
 {
-       DNPRINTF(SWM_D_QUIRK, "quirk_replace: %s:%s [%lu]\n", qp->class,
-           qp->name, qp->quirk);
+       DNPRINTF(SWM_D_QUIRK, "quirk_replace: %s:%s:%s [%lu]\n", qp->class,
+           qp->instance, qp->name, qp->quirk);
 
        quirk_remove(qp);
-       quirk_insert(class, name, quirk);
+       quirk_insert(class, instance, name, quirk);
 
        DNPRINTF(SWM_D_QUIRK, "quirk_replace: leave\n");
 }
 
 void
-setquirk(const char *class, const char *name, unsigned long quirk)
+setquirk(const char *class, const char *instance, const char *name,
+    unsigned long quirk)
 {
        struct quirk            *qp;
 
-       DNPRINTF(SWM_D_QUIRK, "setquirk: enter %s:%s [%lu]\n", class, name,
-          quirk);
+       DNPRINTF(SWM_D_QUIRK, "setquirk: enter %s:%s:%s [%lu]\n", class,
+           instance, name, quirk);
 
        /* Remove/replace existing quirk. */
        TAILQ_FOREACH(qp, &quirks, entry) {
                if (strcmp(qp->class, class) == 0 &&
+                   strcmp(qp->instance, instance) == 0 &&
                    strcmp(qp->name, name) == 0) {
                        if (quirk == 0)
                                quirk_remove(qp);
                        else
-                               quirk_replace(qp, class, name, quirk);
+                               quirk_replace(qp, class, instance, name, quirk);
                        DNPRINTF(SWM_D_QUIRK, "setquirk: leave\n");
                        return;
                }
@@ -6992,49 +7130,99 @@ setquirk(const char *class, const char *name, unsigned long quirk)
 
        /* Only insert if quirk is not NONE. */
        if (quirk)
-               quirk_insert(class, name, quirk);
+               quirk_insert(class, instance, name, quirk);
 
        DNPRINTF(SWM_D_QUIRK, "setquirk: leave\n");
 }
 
+/* Eat '\' in str used to escape square brackets and colon. */
+void
+unescape_selector(char *str)
+{
+       char *cp;
+
+       for (cp = str; *str != '\0'; ++str, ++cp) {
+               if (*str == '\\' && (*(str + 1) == ':' || *(str + 1) == ']' ||
+                   *(str + 1) == '['))
+                       ++str;
+
+               *cp = *str;
+       }
+       *cp = '\0';
+}
+
 int
-setconfquirk(char *selector, char *value, int flags)
+setconfquirk(const char *selector, const char *value, int flags)
 {
-       char                    *cp, *class, *name;
-       int                     retval;
+       char                    *str, *cp, *class;
+       char                    *instance = NULL, *name = NULL;
+       int                     retval, count = 0;
        unsigned long           qrks;
 
        /* suppress unused warning since var is needed */
        (void)flags;
 
-       if (selector == NULL)
-               return (0);
-       if ((cp = strchr(selector, ':')) == NULL)
+       if (selector == NULL || strlen(selector) == 0)
                return (0);
-       *cp = '\0';
-       class = selector;
-       name = cp + 1;
+
+       if ((str = strdup(selector)) == NULL)
+               err(1, "setconfquirk: strdup");
+
+       /* Find non-escaped colon. */
+       class = cp = str;
+       if (*cp == ':') {
+               *cp = '\0';
+               ++count;
+       }
+
+       for (++cp; *cp != '\0'; ++cp) {
+               if (*cp == ':' && *(cp - 1) != '\\') {
+                       *cp = '\0';
+                       ++count;
+               }
+       }
+
+       unescape_selector(class);
+       if (count) {
+               instance = class + strlen(class) + 1;
+               unescape_selector(instance);
+       } else {
+               instance = ".*";
+       }
+
+       if (count > 1) {
+               name = instance + strlen(instance) + 1;
+               unescape_selector(name);
+       } else {
+               name = ".*";
+       }
+
+       DNPRINTF(SWM_D_CONF, "setconfquirk: class: %s, instance: %s, "
+           "name: %s\n", class, instance, name);
+
        if ((retval = parsequirks(value, &qrks)) == 0)
-               setquirk(class, name, qrks);
+               setquirk(class, instance, name, qrks);
+
+       free(str);
        return (retval);
 }
 
 void
 setup_quirks(void)
 {
-       setquirk("MPlayer",             "xv",           SWM_Q_FLOAT | SWM_Q_FULLSCREEN | SWM_Q_FOCUSPREV);
-       setquirk("OpenOffice.org 3.2",  "VCLSalFrame",  SWM_Q_FLOAT);
-       setquirk("Firefox-bin",         "firefox-bin",  SWM_Q_TRANSSZ);
-       setquirk("Firefox",             "Dialog",       SWM_Q_FLOAT);
-       setquirk("Gimp",                "gimp",         SWM_Q_FLOAT | SWM_Q_ANYWHERE);
-       setquirk("XTerm",               "xterm",        SWM_Q_XTERM_FONTADJ);
-       setquirk("xine",                "Xine Window",  SWM_Q_FLOAT | SWM_Q_ANYWHERE);
-       setquirk("Xitk",                "Xitk Combo",   SWM_Q_FLOAT | SWM_Q_ANYWHERE);
-       setquirk("xine",                "xine Panel",   SWM_Q_FLOAT | SWM_Q_ANYWHERE);
-       setquirk("Xitk",                "Xine Window",  SWM_Q_FLOAT | SWM_Q_ANYWHERE);
-       setquirk("xine",                "xine Video Fullscreen Window", SWM_Q_FULLSCREEN | SWM_Q_FLOAT);
-       setquirk("pcb",                 "pcb",          SWM_Q_FLOAT);
-       setquirk("SDL_App",             "SDL_App",      SWM_Q_FLOAT | SWM_Q_FULLSCREEN);
+       setquirk("MPlayer",             "xv",           ".*",   SWM_Q_FLOAT | SWM_Q_FULLSCREEN | SWM_Q_FOCUSPREV);
+       setquirk("OpenOffice.org 3.2",  "VCLSalFrame",  ".*",   SWM_Q_FLOAT);
+       setquirk("Firefox-bin",         "firefox-bin",  ".*",   SWM_Q_TRANSSZ);
+       setquirk("Firefox",             "Dialog",       ".*",   SWM_Q_FLOAT);
+       setquirk("Gimp",                "gimp",         ".*",   SWM_Q_FLOAT | SWM_Q_ANYWHERE);
+       setquirk("XTerm",               "xterm",        ".*",   SWM_Q_XTERM_FONTADJ);
+       setquirk("xine",                "Xine Window",  ".*",   SWM_Q_FLOAT | SWM_Q_ANYWHERE);
+       setquirk("Xitk",                "Xitk Combo",   ".*",   SWM_Q_FLOAT | SWM_Q_ANYWHERE);
+       setquirk("xine",                "xine Panel",   ".*",   SWM_Q_FLOAT | SWM_Q_ANYWHERE);
+       setquirk("Xitk",                "Xine Window",  ".*",   SWM_Q_FLOAT | SWM_Q_ANYWHERE);
+       setquirk("xine",                "xine Video Fullscreen Window", ".*",   SWM_Q_FULLSCREEN | SWM_Q_FLOAT);
+       setquirk("pcb",                 "pcb",          ".*",   SWM_Q_FLOAT);
+       setquirk("SDL_App",             "SDL_App",      ".*",   SWM_Q_FLOAT | SWM_Q_FULLSCREEN);
 }
 
 /* conf file stuff */
@@ -7063,6 +7251,7 @@ enum {
        SWM_S_FOCUS_CLOSE_WRAP,
        SWM_S_FOCUS_DEFAULT,
        SWM_S_FOCUS_MODE,
+       SWM_S_ICONIC_ENABLED,
        SWM_S_REGION_PADDING,
        SWM_S_SPAWN_ORDER,
        SWM_S_SPAWN_TERM,
@@ -7071,23 +7260,20 @@ enum {
        SWM_S_STACK_ENABLED,
        SWM_S_TERM_WIDTH,
        SWM_S_TILE_GAP,
-       SWM_S_TITLE_CLASS_ENABLED,
-       SWM_S_TITLE_NAME_ENABLED,
        SWM_S_URGENT_ENABLED,
        SWM_S_VERBOSE_LAYOUT,
+       SWM_S_WINDOW_CLASS_ENABLED,
+       SWM_S_WINDOW_INSTANCE_ENABLED,
        SWM_S_WINDOW_NAME_ENABLED,
        SWM_S_WORKSPACE_LIMIT
 };
 
 int
-setconfvalue(char *selector, char *value, int flags)
+setconfvalue(const char *selector, const char *value, int flags)
 {
        struct workspace        *ws;
        int                     i, ws_id, num_screens;
-       char                    *b;
-
-       /* suppress unused warning since var is needed */
-       (void)selector;
+       char                    *b, *str;
 
        switch (flags) {
        case SWM_S_BAR_ACTION:
@@ -7132,8 +7318,11 @@ setconfvalue(char *selector, char *value, int flags)
                if (!bar_font_legacy)
                        break;
 
+               if ((str = strdup(value)) == NULL)
+                       err(1, "setconfvalue: strdup");
+
                /* If there are any non-XLFD entries, switch to Xft mode. */
-               while ((b = strsep(&value, ",")) != NULL) {
+               while ((b = strsep(&str, ",")) != NULL) {
                        if (*b == '\0')
                                continue;
                        if (!isxlfd(b)) {
@@ -7141,6 +7330,8 @@ setconfvalue(char *selector, char *value, int flags)
                                break;
                        }
                }
+
+               free(str);
                break;
        case SWM_S_BAR_FORMAT:
                free(bar_format);
@@ -7225,6 +7416,9 @@ setconfvalue(char *selector, char *value, int flags)
                else
                        errx(1, "focus_mode");
                break;
+       case SWM_S_ICONIC_ENABLED:
+               iconic_enabled = atoi(value);
+               break;
        case SWM_S_REGION_PADDING:
                region_padding = atoi(value);
                if (region_padding < 0)
@@ -7263,12 +7457,6 @@ setconfvalue(char *selector, char *value, int flags)
        case SWM_S_TILE_GAP:
                tile_gap = atoi(value);
                break;
-       case SWM_S_TITLE_CLASS_ENABLED:
-               title_class_enabled = atoi(value);
-               break;
-       case SWM_S_TITLE_NAME_ENABLED:
-               title_name_enabled = atoi(value);
-               break;
        case SWM_S_URGENT_ENABLED:
                urgent_enabled = atoi(value);
                break;
@@ -7281,6 +7469,12 @@ setconfvalue(char *selector, char *value, int flags)
                                layouts[i].l_string = plain_stacker;
                }
                break;
+       case SWM_S_WINDOW_CLASS_ENABLED:
+               window_class_enabled = atoi(value);
+               break;
+       case SWM_S_WINDOW_INSTANCE_ENABLED:
+               window_instance_enabled = atoi(value);
+               break;
        case SWM_S_WINDOW_NAME_ENABLED:
                window_name_enabled = atoi(value);
                break;
@@ -7298,7 +7492,7 @@ setconfvalue(char *selector, char *value, int flags)
 }
 
 int
-setconfmodkey(char *selector, char *value, int flags)
+setconfmodkey(const char *selector, const char *value, int flags)
 {
        /* suppress unused warnings since vars are needed */
        (void)selector;
@@ -7318,14 +7512,16 @@ setconfmodkey(char *selector, char *value, int flags)
 }
 
 int
-setconfcolor(char *selector, char *value, int flags)
+setconfcolor(const char *selector, const char *value, int flags)
 {
-       setscreencolor(value, ((selector == NULL)?-1:atoi(selector)), flags);
+       setscreencolor(value,
+           (selector == NULL || strlen(selector) == 0) ? -1 : atoi(selector),
+           flags);
        return (0);
 }
 
 int
-setconfregion(char *selector, char *value, int flags)
+setconfregion(const char *selector, const char *value, int flags)
 {
        /* suppress unused warnings since vars are needed */
        (void)selector;
@@ -7336,7 +7532,7 @@ setconfregion(char *selector, char *value, int flags)
 }
 
 int
-setautorun(char *selector, char *value, int flags)
+setautorun(const char *selector, const char *value, int flags)
 {
        int                     ws_id;
        char                    s[1024];
@@ -7408,7 +7604,7 @@ setautorun(char *selector, char *value, int flags)
 }
 
 int
-setlayout(char *selector, char *value, int flags)
+setlayout(const char *selector, const char *value, int flags)
 {
        int                     ws_id, i, x, mg, ma, si, ar, f = 0;
        int                     st = SWM_V_STACK, num_screens;
@@ -7495,7 +7691,7 @@ setlayout(char *selector, char *value, int flags)
 /* config options */
 struct config_option {
        char                    *optname;
-       int                     (*func)(char*, char*, int);
+       int                     (*func)(const char*, const char*, int);
        int                     funcflags;
 };
 struct config_option configopt[] = {
@@ -7528,6 +7724,7 @@ struct config_option configopt[] = {
        { "focus_close_wrap",           setconfvalue,   SWM_S_FOCUS_CLOSE_WRAP },
        { "focus_default",              setconfvalue,   SWM_S_FOCUS_DEFAULT },
        { "focus_mode",                 setconfvalue,   SWM_S_FOCUS_MODE },
+       { "iconic_enabled",             setconfvalue,   SWM_S_ICONIC_ENABLED },
        { "keyboard_mapping",           setkeymapping,  0 },
        { "layout",                     setlayout,      0 },
        { "modkey",                     setconfmodkey,  0 },
@@ -7542,10 +7739,12 @@ struct config_option configopt[] = {
        { "stack_enabled",              setconfvalue,   SWM_S_STACK_ENABLED },
        { "term_width",                 setconfvalue,   SWM_S_TERM_WIDTH },
        { "tile_gap",                   setconfvalue,   SWM_S_TILE_GAP },
-       { "title_class_enabled",        setconfvalue,   SWM_S_TITLE_CLASS_ENABLED },
-       { "title_name_enabled",         setconfvalue,   SWM_S_TITLE_NAME_ENABLED },
+       { "title_class_enabled",        setconfvalue,   SWM_S_WINDOW_CLASS_ENABLED }, /* For backwards compat. */
+       { "title_name_enabled",         setconfvalue,   SWM_S_WINDOW_INSTANCE_ENABLED }, /* For backwards compat. */
        { "urgent_enabled",             setconfvalue,   SWM_S_URGENT_ENABLED },
        { "verbose_layout",             setconfvalue,   SWM_S_VERBOSE_LAYOUT },
+       { "window_class_enabled",       setconfvalue,   SWM_S_WINDOW_CLASS_ENABLED },
+       { "window_instance_enabled",    setconfvalue,   SWM_S_WINDOW_INSTANCE_ENABLED },
        { "window_name_enabled",        setconfvalue,   SWM_S_WINDOW_NAME_ENABLED },
        { "workspace_limit",            setconfvalue,   SWM_S_WORKSPACE_LIMIT },
 };
@@ -7579,9 +7778,9 @@ int
 conf_load(const char *filename, int keymapping)
 {
        FILE                    *config;
-       char                    *line = NULL, *cp, *optsub, *optval = NULL;
+       char                    *line = NULL, *cp, *ce, *optsub, *optval = NULL;
        size_t                  linelen, lineno = 0;
-       int                     wordlen, i, optidx;
+       int                     wordlen, i, optidx, count;
        struct config_option    *opt = NULL;
 
        DNPRINTF(SWM_D_CONF, "conf_load: begin\n");
@@ -7644,55 +7843,47 @@ conf_load(const char *filename, int keymapping)
                cp += wordlen;
                cp += strspn(cp, " \t\n"); /* eat whitespace */
 
-               /* from here on out we call goto invalid to continue */
-
                /* get [selector] if any */
                optsub = NULL;
+               count = 0;
                if (*cp == '[') {
-                       cp++;
-                       wordlen = strcspn(cp, "]");
-                       if (*cp != ']') {
-                               if (wordlen == 0) {
-                                       add_startup_exception("%s: line %zd: "
-                                           "syntax error", filename, lineno);
-                                       goto invalid;
+                       ++count;
+                       /* Get length of selector. */
+                       for (ce = ++cp; *ce != '\0'; ++ce) {
+                               /* Find matching (not escaped) bracket. */
+                               if (*ce == ']' && *(ce - 1) != '\\') {
+                                       --count;
+                                       break;
                                }
+                       }
 
-                               if (asprintf(&optsub, "%.*s", wordlen, cp) ==
-                                   -1) {
-                                       add_startup_exception("%s: line %zd: "
-                                           "unable to allocatememory for "
-                                           "selector", filename, lineno);
-                                       goto invalid;
-                               }
+                       if (count > 0) {
+                               add_startup_exception("%s: line %zd: syntax "
+                                   "error: unterminated selector", filename,
+                                   lineno);
+                               continue;
                        }
-                       cp += wordlen;
-                       cp += strspn(cp, "] \t\n"); /* eat trailing */
+
+                       /* ce points at ']'; terminate optsub. */
+                       *ce = '\0';
+                       optsub = cp;
+                       cp = ce + 1;
                }
                cp += strspn(cp, "= \t\n"); /* eat trailing */
                /* get RHS value */
-               optval = strdup(cp);
+               optval = cp;
                if (strlen(optval) == 0) {
                        add_startup_exception("%s: line %zd: must supply value "
                            "to %s", filename, lineno,
                            configopt[optidx].optname);
-                       goto invalid;
+                       continue;
                }
                /* call function to deal with it all */
                if (configopt[optidx].func(optsub, optval,
                    configopt[optidx].funcflags) != 0) {
                        add_startup_exception("%s: line %zd: invalid data for "
                            "%s", filename, lineno, configopt[optidx].optname);
-                       goto invalid;
-               }
-invalid:
-               if (optval) {
-                       free(optval);
-                       optval = NULL;
-               }
-               if (optsub) {
-                       free(optsub);
-                       optsub = NULL;
+                       continue;
                }
        }
 
@@ -7838,6 +8029,7 @@ manage_window(xcb_window_t id, uint16_t mapped)
        struct ws_win           *win, *ww;
        int                     ws_idx;
        char                    ws_idx_str[SWM_PROPLEN];
+       char                    *class, *instance, *name;
        struct swm_region       *r;
        struct pid_e            *p;
        struct quirk            *qp;
@@ -7963,31 +8155,38 @@ manage_window(xcb_window_t id, uint16_t mapped)
        ewmh_autoquirk(win);
 
        /* Determine initial quirks. */
-       if (xcb_icccm_get_wm_class_reply(conn,
+       xcb_icccm_get_wm_class_reply(conn,
            xcb_icccm_get_wm_class(conn, win->id),
-           &win->ch, NULL)) {
-               DNPRINTF(SWM_D_CLASS, "manage_window: class: %s, name: %s\n",
-                   win->ch.class_name, win->ch.instance_name);
-
-               /* java is retarded so treat it special */
-               if (strstr(win->ch.instance_name, "sun-awt")) {
-                       DNPRINTF(SWM_D_CLASS, "manage_window: java window "
-                           "detected.\n");
-                       win->java = 1;
-               }
-
-               TAILQ_FOREACH(qp, &quirks, entry) {
-                       if (strcmp(win->ch.class_name, qp->class) == 0 &&
-                           strcmp(win->ch.instance_name, qp->name) == 0) {
-                               DNPRINTF(SWM_D_CLASS, "manage_window: on quirks"
-                                   "list; mask: 0x%lx\n", qp->quirk);
-                               if (qp->quirk & SWM_Q_FLOAT)
-                                       win->floating = 1;
-                               win->quirks = qp->quirk;
-                       }
+           &win->ch, NULL);
+
+       class = win->ch.class_name ? win->ch.class_name : "";
+       instance = win->ch.instance_name ? win->ch.instance_name : "";
+       name = get_win_name(win->id);
+
+       DNPRINTF(SWM_D_CLASS, "manage_window: class: %s, instance: %s, "
+           "name: %s\n", class, instance, name);
+
+       /* java is retarded so treat it special */
+       if (win->ch.instance_name && strstr(win->ch.instance_name, "sun-awt")) {
+               DNPRINTF(SWM_D_CLASS, "manage_window: java window detected.\n");
+               win->java = 1;
+       }
+
+       TAILQ_FOREACH(qp, &quirks, entry) {
+               if (regexec(&qp->regex_class, class, 0, NULL, 0) == 0 &&
+                   regexec(&qp->regex_instance, instance, 0, NULL, 0) == 0 &&
+                   regexec(&qp->regex_name, name, 0, NULL, 0) == 0) {
+                       DNPRINTF(SWM_D_CLASS, "manage_window: matched "
+                           "quirk: %s:%s:%s mask: %#lx\n", qp->class,
+                           qp->instance, qp->name, qp->quirk);
+                       if (qp->quirk & SWM_Q_FLOAT)
+                               win->floating = 1;
+                       win->quirks = qp->quirk;
                }
        }
 
+       free(name);
+
        /* Alter window position if quirky */
        if (win->quirks & SWM_Q_ANYWHERE)
                win->manual = 1;
@@ -8620,10 +8819,22 @@ mapnotify(xcb_map_notify_event_t *e)
 void
 mappingnotify(xcb_mapping_notify_event_t *e)
 {
+       struct ws_win   *w;
+       int     i, j, num_screens;
+
        xcb_refresh_keyboard_mapping(syms, e);
 
-       if (e->request == XCB_MAPPING_KEYBOARD)
+       if (e->request == XCB_MAPPING_KEYBOARD) {
                grabkeys();
+
+               /* Regrab buttons on all managed windows. */
+               num_screens = get_screen_count();
+               for (i = 0; i < num_screens; i++)
+                       for (j = 0; j < workspace_limit; j++)
+                               TAILQ_FOREACH(w, &screens[i].ws[j].winlist,
+                                   entry)
+                                       grabbuttons(w);
+       }
 }
 
 void
@@ -8809,6 +9020,10 @@ propertynotify(xcb_property_notify_event_t *e)
                                focus_flush();
                        }
                } else if (e->state == XCB_PROPERTY_DELETE) {
+                       /* Reload floating geometry in case region changed. */
+                       if (win->floating)
+                               load_float_geom(win);
+
                        /* The window is no longer iconic, restack ws. */
                        if (focus_mode != SWM_FOCUS_FOLLOW)
                                ws->focus_pending = get_focus_magic(win);
@@ -8901,7 +9116,7 @@ unmapnotify(xcb_unmap_notify_event_t *e)
 void
 clientmessage(xcb_client_message_event_t *e)
 {
-       struct ws_win *win;
+       struct ws_win           *win;
        xcb_map_request_event_t mre;
 #ifdef SWM_DEBUG
        char                    *name;
@@ -9370,10 +9585,10 @@ grab_windows(void)
 void
 setup_screens(void)
 {
-       int                     i, j, k, num_screens;
-       struct workspace        *ws;
-       uint32_t                gcv[1], wa[1];
-       const xcb_query_extension_reply_t *qep;
+       int                                     i, j, k, num_screens;
+       struct workspace                        *ws;
+       uint32_t                                gcv[1], wa[1];
+       const xcb_query_extension_reply_t       *qep;
        xcb_screen_t                            *screen;
        xcb_randr_query_version_cookie_t        c;
        xcb_randr_query_version_reply_t         *r;
@@ -9626,7 +9841,7 @@ main(int argc, char *argv[])
        int                     xfd, i, num_screens, startup = 1;
        struct sigaction        sact;
        xcb_generic_event_t     *evt;
-       struct timeval          tv;
+       struct timeval          tv;
        fd_set                  rd;
        int                     rd_max;
        int                     stdin_ready = 0;