From 1b34d4f0b1d634eb8219d52c693fa80b99ed1bfa Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Wed, 30 Sep 2009 02:39:57 +0200 Subject: find: add zeroing of G.xxx; ftpd - remove extraneous zeroing of G.xxx Brought "G trick" to the same shape in a few more places. Signed-off-by: Denys Vlasenko --- coreutils/dd.c | 6 ++++-- coreutils/ls.c | 12 ++++++------ 2 files changed, 10 insertions(+), 8 deletions(-) (limited to 'coreutils') diff --git a/coreutils/dd.c b/coreutils/dd.c index 627e7e7b5..c5c9476af 100644 --- a/coreutils/dd.c +++ b/coreutils/dd.c @@ -36,8 +36,10 @@ struct globals { off_t out_full, out_part, in_full, in_part; }; #define G (*(struct globals*)&bb_common_bufsiz1) -/* We have to zero it out because of NOEXEC */ -#define INIT_G() memset(&G, 0, sizeof(G)) +#define INIT_G() do { \ + /* we have to zero it out because of NOEXEC */ \ + memset(&G, 0, sizeof(G)); \ +} while (0) static void dd_output_status(int UNUSED_PARAM cur_signal) diff --git a/coreutils/ls.c b/coreutils/ls.c index 827b35089..19d38049c 100644 --- a/coreutils/ls.c +++ b/coreutils/ls.c @@ -264,15 +264,15 @@ struct globals { }; #define G (*(struct globals*)&bb_common_bufsiz1) #if ENABLE_FEATURE_LS_COLOR -#define show_color (G.show_color ) +# define show_color (G.show_color ) #else enum { show_color = 0 }; #endif -#define exit_code (G.exit_code ) -#define all_fmt (G.all_fmt ) +#define exit_code (G.exit_code ) +#define all_fmt (G.all_fmt ) #if ENABLE_FEATURE_AUTOWIDTH -#define tabstops (G.tabstops ) -#define terminal_width (G.terminal_width) +# define tabstops (G.tabstops ) +# define terminal_width (G.terminal_width) #else enum { tabstops = COLUMN_GAP, @@ -280,8 +280,8 @@ enum { }; #endif #define current_time_t (G.current_time_t) -/* memset: we have to zero it out because of NOEXEC */ #define INIT_G() do { \ + /* we have to zero it out because of NOEXEC */ \ memset(&G, 0, sizeof(G)); \ IF_FEATURE_AUTOWIDTH(tabstops = COLUMN_GAP;) \ IF_FEATURE_AUTOWIDTH(terminal_width = TERMINAL_WIDTH;) \ -- cgit v1.2.3