diff options
author | Hiltjo Posthuma <hiltjo@codemadness.org> | 2022-06-06 11:00:26 +0200 |
---|---|---|
committer | Hiltjo Posthuma <hiltjo@codemadness.org> | 2022-06-06 11:03:29 +0200 |
commit | 8737035999474283a28faa5780f43138f96ab82c (patch) | |
tree | 1cda39629165027d8f2fc7f6c1f0b38780d59bd7 /themes/mono.h | |
parent | cc3dd4534bcbbc750d563992b93b1131410a2a76 (diff) |
themes: simplify empty macros, remove do { } while(0)
Multiple function calls should still be nested in a do { } while(0), but this
is hopefully obvious looking at the other lines.
While testing, gcc with -O0 created the same binary.
clang added a single jmp instruction.
Diffstat (limited to 'themes/mono.h')
-rw-r--r-- | themes/mono.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/themes/mono.h b/themes/mono.h index 586ede0..253d76e 100644 --- a/themes/mono.h +++ b/themes/mono.h @@ -1,13 +1,13 @@ /* default mono theme */ -#define THEME_ITEM_NORMAL() do { } while(0) -#define THEME_ITEM_FOCUS() do { } while(0) +#define THEME_ITEM_NORMAL() +#define THEME_ITEM_FOCUS() #define THEME_ITEM_BOLD() do { attrmode(ATTR_BOLD_ON); } while(0) #define THEME_ITEM_SELECTED() do { if (p->focused) attrmode(ATTR_REVERSE_ON); } while(0) -#define THEME_SCROLLBAR_FOCUS() do { } while(0) +#define THEME_SCROLLBAR_FOCUS() #define THEME_SCROLLBAR_NORMAL() do { attrmode(ATTR_FAINT_ON); } while(0) #define THEME_SCROLLBAR_TICK_FOCUS() do { attrmode(ATTR_REVERSE_ON); } while(0) #define THEME_SCROLLBAR_TICK_NORMAL() do { attrmode(ATTR_REVERSE_ON); } while(0) #define THEME_LINEBAR() do { attrmode(ATTR_FAINT_ON); } while(0) #define THEME_STATUSBAR() do { attrmode(ATTR_REVERSE_ON); } while(0) #define THEME_INPUT_LABEL() do { attrmode(ATTR_REVERSE_ON); } while(0) -#define THEME_INPUT_NORMAL() do { } while(0) +#define THEME_INPUT_NORMAL() |