diff options
author | Robin Haberkorn <robin.haberkorn@googlemail.com> | 2015-03-07 02:45:56 +0100 |
---|---|---|
committer | Robin Haberkorn <robin.haberkorn@googlemail.com> | 2015-03-07 02:54:09 +0100 |
commit | 2789e5da50987b908a4aa5758a17c86570d94d63 (patch) | |
tree | a60f11f3b82ff3d7ff0a07bf576f851f68165266 /src/interface-gtk.cpp | |
parent | 4aa51b925f5364511173c74277b78d2984e37218 (diff) | |
download | sciteco-2789e5da50987b908a4aa5758a17c86570d94d63.tar.gz |
cleaned up usage of the escape control character: introduced CTL_KEY_ESC and CTL_KEY_ESC_STR
* the reason for the CTL_KEY() macro is to get the control character
resulting from a CTRL+Key press -- at least this is how SciTECO
presents these key presses.
It is also a macro and may be resolved to a constant expression,
so it can be used in switch-case statements.
Sometimes it is clearer to use standard C escape sequences (like '\t').
* CTL_KEY('[') for escape is hard to read, so I always used '\x1B' which
is even more cryptic.
Diffstat (limited to 'src/interface-gtk.cpp')
-rw-r--r-- | src/interface-gtk.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/interface-gtk.cpp b/src/interface-gtk.cpp index e7f6927..8375dd8 100644 --- a/src/interface-gtk.cpp +++ b/src/interface-gtk.cpp @@ -196,7 +196,7 @@ InterfaceGtk::cmdline_insert_chr(gint &pos, gchar chr) * View::set_representations() */ switch (chr) { - case '\x1B': /* escape */ + case CTL_KEY_ESC: strcpy(buffer, "$"); break; case '\r': @@ -308,7 +308,7 @@ handle_key_press(bool is_shift, bool is_ctl, guint keyval) { switch (keyval) { case GDK_Escape: - cmdline.keypress('\x1B'); + cmdline.keypress(CTL_KEY_ESC); break; case GDK_BackSpace: cmdline.keypress('\b'); |