remove some GNUisms. by Dan Fandrich (dan AT coneharvesters.com)

function                                             old     new   delta
logdirs_reopen                                      1310    1308      -2
read_line_input                                     4757    4753      -4
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-6)               Total: -6 bytes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2009-09-06 02:36:23 +02:00
parent 1f7c167b23
commit 90a9904e9e
15 changed files with 22 additions and 17 deletions

View File

@ -2078,7 +2078,7 @@ int gzip_main(int argc UNUSED_PARAM, char **argv)
//if (opt & 0x4) // -v //if (opt & 0x4) // -v
argv += optind; argv += optind;
SET_PTR_TO_GLOBALS(xzalloc(sizeof(struct globals) + sizeof(struct globals2)) SET_PTR_TO_GLOBALS((char *)xzalloc(sizeof(struct globals)+sizeof(struct globals2))
+ sizeof(struct globals)); + sizeof(struct globals));
barrier(); barrier();

View File

@ -486,7 +486,7 @@ struct globals2 {
#define fsplitter (G.fsplitter ) #define fsplitter (G.fsplitter )
#define rsplitter (G.rsplitter ) #define rsplitter (G.rsplitter )
#define INIT_G() do { \ #define INIT_G() do { \
SET_PTR_TO_GLOBALS(xzalloc(sizeof(G1) + sizeof(G)) + sizeof(G1)); \ SET_PTR_TO_GLOBALS((char*)xzalloc(sizeof(G1)+sizeof(G)) + sizeof(G1)); \
G.next_token__ltclass = TC_OPTERM; \ G.next_token__ltclass = TC_OPTERM; \
G.evaluate__seed = 1; \ G.evaluate__seed = 1; \
} while (0) } while (0)

View File

@ -1186,7 +1186,7 @@ static void process_files(void)
case 'x': /* Exchange hold and pattern space */ case 'x': /* Exchange hold and pattern space */
{ {
char *tmp = pattern_space; char *tmp = pattern_space;
pattern_space = G.hold_space ? : xzalloc(1); pattern_space = G.hold_space ? G.hold_space : xzalloc(1);
last_gets_char = '\n'; last_gets_char = '\n';
G.hold_space = tmp; G.hold_space = tmp;
break; break;

View File

@ -1479,7 +1479,7 @@ extern struct globals *const ptr_to_globals;
/* At least gcc 3.4.6 on mipsel system needs optimization barrier */ /* At least gcc 3.4.6 on mipsel system needs optimization barrier */
#define barrier() __asm__ __volatile__("":::"memory") #define barrier() __asm__ __volatile__("":::"memory")
#define SET_PTR_TO_GLOBALS(x) do { \ #define SET_PTR_TO_GLOBALS(x) do { \
(*(struct globals**)&ptr_to_globals) = (x); \ (*(struct globals**)&ptr_to_globals) = (void*)(x); \
barrier(); \ barrier(); \
} while (0) } while (0)

View File

@ -862,7 +862,7 @@ int init_main(int argc UNUSED_PARAM, char **argv)
struct sysinfo info; struct sysinfo info;
if (sysinfo(&info) == 0 if (sysinfo(&info) == 0
&& (info.mem_unit ? : 1) * (long long)info.totalram < 1024*1024 && (info.mem_unit ? info.mem_unit : 1) * (long long)info.totalram < 1024*1024
) { ) {
message(L_CONSOLE, "Low memory, forcing swapon"); message(L_CONSOLE, "Low memory, forcing swapon");
/* swapon -a requires /proc typically */ /* swapon -a requires /proc typically */

View File

@ -2037,7 +2037,8 @@ int FAST_FUNC read_line_input(const char *prompt, char *command, int maxsize, li
rewrite_line: rewrite_line:
/* Rewrite the line with the selected history item */ /* Rewrite the line with the selected history item */
/* change command */ /* change command */
command_len = load_string(state->history[state->cur_history] ? : "", maxsize); command_len = load_string(state->history[state->cur_history] ?
state->history[state->cur_history] : "", maxsize);
/* redraw and go to eol (bol, in vi) */ /* redraw and go to eol (bol, in vi) */
redraw(cmdedit_y, (state->flags & VI_MODE) ? 9999 : 0); redraw(cmdedit_y, (state->flags & VI_MODE) ? 9999 : 0);
break; break;

View File

@ -105,7 +105,7 @@ int FAST_FUNC set_loop(char **device, const char *file, unsigned long long offse
} }
/* Find a loop device. */ /* Find a loop device. */
try = *device ? : dev; try = *device ? *device : dev;
for (i = 0; rc; i++) { for (i = 0; rc; i++) {
sprintf(dev, LOOP_FORMAT, i); sprintf(dev, LOOP_FORMAT, i);

View File

@ -177,7 +177,8 @@ int makemime_main(int argc UNUSED_PARAM, char **argv)
static const char *find_token(const char *const string_array[], const char *key, const char *defvalue) static const char *find_token(const char *const string_array[], const char *key, const char *defvalue)
{ {
const char *r = NULL; const char *r = NULL;
for (int i = 0; string_array[i] != 0; i++) { int i;
for (i = 0; string_array[i] != NULL; i++) {
if (strcasecmp(string_array[i], key) == 0) { if (strcasecmp(string_array[i], key) == 0) {
r = (char *)string_array[i+1]; r = (char *)string_array[i+1];
break; break;

View File

@ -624,7 +624,7 @@ static void print_found(const char *line)
while (match_status == 0) { while (match_status == 0) {
char *new = xasprintf("%s%.*s"HIGHLIGHT"%.*s"NORMAL, char *new = xasprintf("%s%.*s"HIGHLIGHT"%.*s"NORMAL,
growline ? : "", growline ? growline : "",
match_structs.rm_so, str, match_structs.rm_so, str,
match_structs.rm_eo - match_structs.rm_so, match_structs.rm_eo - match_structs.rm_so,
str + match_structs.rm_so); str + match_structs.rm_so);

View File

@ -314,6 +314,7 @@ static int load_dep_bb(void)
while ((line = xmalloc_fgetline(fp)) != NULL) { while ((line = xmalloc_fgetline(fp)) != NULL) {
char* space; char* space;
char* linebuf;
int cur; int cur;
if (!line[0]) { if (!line[0]) {
@ -328,7 +329,8 @@ static int load_dep_bb(void)
if (*space) if (*space)
*space++ = '\0'; *space++ = '\0';
modinfo[cur].aliases = space; modinfo[cur].aliases = space;
modinfo[cur].deps = xmalloc_fgetline(fp) ? : xzalloc(1); linebuf = xmalloc_fgetline(fp);
modinfo[cur].deps = linebuf ? linebuf : xzalloc(1);
if (modinfo[cur].deps[0]) { if (modinfo[cur].deps[0]) {
/* deps are not "", so next line must be empty */ /* deps are not "", so next line must be empty */
line = xmalloc_fgetline(fp); line = xmalloc_fgetline(fp);

View File

@ -670,7 +670,7 @@ static unsigned logdir_open(struct logdir *ld, const char *fn)
memRchr = memchr; memRchr = memchr;
/* Add '\n'-terminated line to ld->inst */ /* Add '\n'-terminated line to ld->inst */
while (1) { while (1) {
int l = asprintf(&new, "%s%s\n", ld->inst ? : "", s); int l = asprintf(&new, "%s%s\n", ld->inst ? ld->inst : "", s);
if (l >= 0 && new) if (l >= 0 && new)
break; break;
pause_nomem(); pause_nomem();

View File

@ -347,7 +347,7 @@ static int restore(const char *file)
* same. For "-vv", emit everything. */ * same. For "-vv", emit everything. */
if (verbose > 1 || !user_only_changed) { if (verbose > 1 || !user_only_changed) {
bb_info_msg("%s: reset %s context %s->%s", bb_info_msg("%s: reset %s context %s->%s",
applet_name, my_file, context ?: "", newcon); applet_name, my_file, context ? context : "", newcon);
} }
} }

View File

@ -212,7 +212,7 @@ int bbsh_main(int argc, char **argv)
unsigned cmdlen=0; unsigned cmdlen=0;
for (;;) { for (;;) {
if (!f) putchar('$'); if (!f) putchar('$');
if (1 > getline(&command, &cmdlen,f ? : stdin)) break; if (1 > getline(&command, &cmdlen, f ? f : stdin)) break;
handle(command); handle(command);
} }

View File

@ -5794,7 +5794,7 @@ static struct pipe *parse_stream(char **pstring,
* found. When recursing, quote state is passed in via dest->o_escape. * found. When recursing, quote state is passed in via dest->o_escape.
*/ */
debug_printf_parse("parse_stream entered, end_trigger='%c'\n", debug_printf_parse("parse_stream entered, end_trigger='%c'\n",
end_trigger ? : 'X'); end_trigger ? end_trigger : 'X');
debug_enter(); debug_enter();
G.ifs = get_local_var_value("IFS"); G.ifs = get_local_var_value("IFS");
@ -6860,7 +6860,8 @@ static int FAST_FUNC builtin_cd(char **argv)
* bash says "bash: cd: HOME not set" and does nothing * bash says "bash: cd: HOME not set" and does nothing
* (exitcode 1) * (exitcode 1)
*/ */
newdir = get_local_var_value("HOME") ? : "/"; const char *home = get_local_var_value("HOME");
newdir = home ? home : "/";
} }
if (chdir(newdir)) { if (chdir(newdir)) {
/* Mimic bash message exactly */ /* Mimic bash message exactly */

View File

@ -254,7 +254,7 @@ static int read_mode_db(struct fb_var_screeninfo *base, const char *fn,
} }
#endif #endif
static void setmode(struct fb_var_screeninfo *base, static void setfbmode(struct fb_var_screeninfo *base,
struct fb_var_screeninfo *set) struct fb_var_screeninfo *set)
{ {
if ((int32_t) set->xres > 0) if ((int32_t) set->xres > 0)
@ -402,7 +402,7 @@ int fbset_main(int argc, char **argv)
} }
if (options & OPT_CHANGE) { if (options & OPT_CHANGE) {
setmode(&var, &varset); setfbmode(&var, &varset);
if (options & OPT_ALL) if (options & OPT_ALL)
var.activate = FB_ACTIVATE_ALL; var.activate = FB_ACTIVATE_ALL;
xioctl(fh, FBIOPUT_VSCREENINFO, &var); xioctl(fh, FBIOPUT_VSCREENINFO, &var);