Re: [PATCH 2/3] desktop-shell: Make zapping configurable

2016-01-12 Thread Bob Ham
On Mon, 2016-01-11 at 12:04 -0600, Derek Foreman wrote: > On 11/01/16 09:41 AM, Bob Ham wrote: > > + weston_config_section_get_bool(section, > > + "zap", &zap, true); > > I think "allow_zap" would be better... Ack > > + if (shell->zap != false) > > Wouldn't

Re: [PATCH 2/3] desktop-shell: Make zapping configurable

2016-01-11 Thread Derek Foreman
On 11/01/16 09:41 AM, Bob Ham wrote: > Add a new boolean weston.ini option, "zap" to enable or disable the > Ctrl-Alt-Backspace key combination. > > Signed-off-by: Bob Ham > --- > desktop-shell/shell.c | 14 +++--- > desktop-shell/shell.h | 1 + > man/weston.ini.man| 4 > 3 fi

[PATCH 2/3] desktop-shell: Make zapping configurable

2016-01-11 Thread Bob Ham
Add a new boolean weston.ini option, "zap" to enable or disable the Ctrl-Alt-Backspace key combination. Signed-off-by: Bob Ham --- desktop-shell/shell.c | 14 +++--- desktop-shell/shell.h | 1 + man/weston.ini.man| 4 3 files changed, 16 insertions(+), 3 deletions(-) diff --g