# HG changeset patch # User Jim Meyering # Date 1212386957 0 # Node ID 4390d64d33287cd7a28b630dda4222f6995e00a8 # Parent 1c16540a232918a154af4034aaef6ae74a9afba4 Make "xfree (NULL)" a no-op; remove useless if-before-xfree. * src/alloc.c (xfree): Return right away for a NULL arg. * src/lread.c (nosuffix): Remove now-useless if-before-xfree tests. * src/gtkutil.c (xg_gtk_scroll_destroy): Likewise. * src/mac.c (create_apple_event_from_event_ref): Likewise. (create_apple_event_from_drag_ref, cfstring_create_normalized): Likewise. * src/doprnt.c (doprnt1): Likewise. * src/frame.c (frame): Likewise. * src/keyboard.c (wipe_kboard): Likewise. * src/macterm.c (x_free_frame_resources, xlfdpat_destroy, XFreePixmap): (init_font_name_table, mac_unload_font, x_delete_display): Likewise. * src/term.c (tty_default_color_capabilities, maybe_fatal) (delete_tty): Likewise. * src/w16select.c (string): Likewise. * src/w32.c (w32_get_resource, SET_ENV_BUF_SIZE): Likewise. * src/w32bdf.c (w32_free_bdf_font): Likewise. * src/w32fns.c (w32_unload_font): Likewise. * src/w32font.c (w32font_close): Likewise. * src/window.c (size_window): Likewise. * src/xselect.c (receive_incremental_selection): Likewise. * src/xterm.c (x_free_frame_resources, x_delete_display): Likewise. * src/mactoolbox.c (create_apple_event_from_drag_ref): Likewise. * src/w32.c (stat): Likewise. diff -r 1c16540a2329 -r 4390d64d3328 lib-src/ChangeLog --- a/lib-src/ChangeLog Mon Jun 02 06:02:30 2008 +0000 +++ b/lib-src/ChangeLog Mon Jun 02 06:09:17 2008 +0000 @@ -1,12 +1,12 @@ 2008-06-02 Jim Meyering - * lib-src/ebrowse.c (xfree): Remove definition; s/xfree/free/ + * ebrowse.c (xfree): Remove definition; s/xfree/free/ remove useless if-before-free tests - * lib-src/ebrowse.c (xfree): Likewise. - * lib-src/etags.c (process_file_name, free_tree, free_fdesc): Likewise. + * ebrowse.c (xfree): Likewise. + * etags.c (process_file_name, free_tree, free_fdesc): Likewise. (popclass_above, Prolog_functions, Erlang_functions): Likewise. - * lib-src/pop.c (pop_quit): Likewise. + * pop.c (pop_quit): Likewise. 2008-05-30 Juanma Barranquero diff -r 1c16540a2329 -r 4390d64d3328 lwlib/ChangeLog --- a/lwlib/ChangeLog Mon Jun 02 06:02:30 2008 +0000 +++ b/lwlib/ChangeLog Mon Jun 02 06:09:17 2008 +0000 @@ -1,8 +1,8 @@ 2008-06-02 Jim Meyering remove useless if-before-free tests - * lwlib/lwlib-Xm.c (xm_update_one_value): Likewise. - * lwlib/lwlib.c (safe_free_str, free_widget_value_tree): Likewise. + * lwlib-Xm.c (xm_update_one_value): Likewise. + * lwlib.c (safe_free_str, free_widget_value_tree): Likewise. 2008-04-18 Stefan Monnier diff -r 1c16540a2329 -r 4390d64d3328 src/ChangeLog --- a/src/ChangeLog Mon Jun 02 06:02:30 2008 +0000 +++ b/src/ChangeLog Mon Jun 02 06:09:17 2008 +0000 @@ -1,13 +1,37 @@ 2008-06-02 Jim Meyering + Make "xfree (NULL)" a no-op; remove useless if-before-xfree. + * alloc.c (xfree): Return right away for a NULL arg. + * lread.c (nosuffix): Remove now-useless if-before-xfree tests. + * gtkutil.c (xg_gtk_scroll_destroy): Likewise. + * mac.c (create_apple_event_from_event_ref): Likewise. + (create_apple_event_from_drag_ref, cfstring_create_normalized): Likewise. + * doprnt.c (doprnt1): Likewise. + * frame.c (frame): Likewise. + * keyboard.c (wipe_kboard): Likewise. + * macterm.c (x_free_frame_resources, xlfdpat_destroy, XFreePixmap): + (init_font_name_table, mac_unload_font, x_delete_display): Likewise. + * term.c (tty_default_color_capabilities, maybe_fatal) + (delete_tty): Likewise. + * w16select.c (string): Likewise. + * w32.c (w32_get_resource, SET_ENV_BUF_SIZE): Likewise. + * w32bdf.c (w32_free_bdf_font): Likewise. + * w32fns.c (w32_unload_font): Likewise. + * w32font.c (w32font_close): Likewise. + * window.c (size_window): Likewise. + * xselect.c (receive_incremental_selection): Likewise. + * xterm.c (x_free_frame_resources, x_delete_display): Likewise. + * mactoolbox.c (create_apple_event_from_drag_ref): Likewise. + * w32.c (stat): Likewise. + remove useless if-before-free tests - * src/editfns.c (Fset_time_zone_rule): Likewise. - * src/lread.c (nosuffix): Likewise. - * src/ralloc.c (get_bloc): Likewise. - * src/regex.c (reg_free): Likewise. - * src/xftfont.c (xftfont_open, xftfont_close): Likewise. - * src/xrdb.c (get_user_app, get_environ_db, x_load_resources): Likewise. - * src/xsmfns.c (smc_save_yourself_CB): Likewise. + * editfns.c (Fset_time_zone_rule): Likewise. + * lread.c (nosuffix): Likewise. + * ralloc.c (get_bloc): Likewise. + * regex.c (reg_free): Likewise. + * xftfont.c (xftfont_open, xftfont_close): Likewise. + * xrdb.c (get_user_app, get_environ_db, x_load_resources): Likewise. + * xsmfns.c (smc_save_yourself_CB): Likewise. 2008-06-02 Kenichi Handa diff -r 1c16540a2329 -r 4390d64d3328 src/alloc.c --- a/src/alloc.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/alloc.c Mon Jun 02 06:09:17 2008 +0000 @@ -797,6 +797,8 @@ xfree (block) POINTER_TYPE *block; { + if (!block) + return; MALLOC_BLOCK_INPUT; free (block); MALLOC_UNBLOCK_INPUT; diff -r 1c16540a2329 -r 4390d64d3328 src/doprnt.c --- a/src/doprnt.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/doprnt.c Mon Jun 02 06:09:17 2008 +0000 @@ -326,8 +326,7 @@ }; /* If we had to malloc something, free it. */ - if (big_buffer) - xfree (big_buffer); + xfree (big_buffer); *bufptr = 0; /* Make sure our string end with a '\0' */ return bufptr - buffer; diff -r 1c16540a2329 -r 4390d64d3328 src/frame.c --- a/src/frame.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/frame.c Mon Jun 02 06:09:17 2008 +0000 @@ -1477,20 +1477,13 @@ Vframe_list = Fdelq (frame, Vframe_list); FRAME_SET_VISIBLE (f, 0); - if (f->namebuf) - xfree (f->namebuf); - if (f->decode_mode_spec_buffer) - xfree (f->decode_mode_spec_buffer); - if (FRAME_INSERT_COST (f)) - xfree (FRAME_INSERT_COST (f)); - if (FRAME_DELETEN_COST (f)) - xfree (FRAME_DELETEN_COST (f)); - if (FRAME_INSERTN_COST (f)) - xfree (FRAME_INSERTN_COST (f)); - if (FRAME_DELETE_COST (f)) - xfree (FRAME_DELETE_COST (f)); - if (FRAME_MESSAGE_BUF (f)) - xfree (FRAME_MESSAGE_BUF (f)); + xfree (f->namebuf); + xfree (f->decode_mode_spec_buffer); + xfree (FRAME_INSERT_COST (f)); + xfree (FRAME_DELETEN_COST (f)); + xfree (FRAME_INSERTN_COST (f)); + xfree (FRAME_DELETE_COST (f)); + xfree (FRAME_MESSAGE_BUF (f)); /* Since some events are handled at the interrupt level, we may get an event for f at any time; if we zero out the frame's terminal diff -r 1c16540a2329 -r 4390d64d3328 src/gtkutil.c --- a/src/gtkutil.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/gtkutil.c Mon Jun 02 06:09:17 2008 +0000 @@ -3106,7 +3106,7 @@ int id = (int) (EMACS_INT) data; /* The EMACS_INT cast avoids a warning. */ p = g_object_get_data (G_OBJECT (widget), XG_LAST_SB_DATA); - if (p) xfree (p); + xfree (p); xg_remove_widget_from_map (id); } diff -r 1c16540a2329 -r 4390d64d3328 src/keyboard.c --- a/src/keyboard.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/keyboard.c Mon Jun 02 06:09:17 2008 +0000 @@ -11575,8 +11575,7 @@ wipe_kboard (kb) KBOARD *kb; { - if (kb->kbd_macro_buffer) - xfree (kb->kbd_macro_buffer); + xfree (kb->kbd_macro_buffer); } #ifdef MULTI_KBOARD diff -r 1c16540a2329 -r 4390d64d3328 src/lread.c --- a/src/lread.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/lread.c Mon Jun 02 06:09:17 2008 +0000 @@ -1273,8 +1273,7 @@ saved_doc_string = 0; saved_doc_string_size = 0; - if (prev_saved_doc_string) - xfree (prev_saved_doc_string); + xfree (prev_saved_doc_string); prev_saved_doc_string = 0; prev_saved_doc_string_size = 0; diff -r 1c16540a2329 -r 4390d64d3328 src/mac.c --- a/src/mac.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/mac.c Mon Jun 02 06:09:17 2008 +0000 @@ -906,8 +906,7 @@ break; } } - if (buf) - xfree (buf); + xfree (buf); return result; } @@ -4778,10 +4777,8 @@ out_len / sizeof (UniChar)); if (uni) DisposeUnicodeToTextInfo (&uni); - if (out_buf) - xfree (out_buf); - if (buffer) - xfree (buffer); + xfree (out_buf); + xfree (buffer); } else { diff -r 1c16540a2329 -r 4390d64d3328 src/macterm.c --- a/src/macterm.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/macterm.c Mon Jun 02 06:09:17 2008 +0000 @@ -910,8 +910,7 @@ #if USE_MAC_IMAGE_IO if (pixmap) { - if (pixmap->data) - xfree (pixmap->data); + xfree (pixmap->data); xfree (pixmap); } #else @@ -7390,8 +7389,7 @@ x_free_gcs (f); - if (FRAME_SIZE_HINTS (f)) - xfree (FRAME_SIZE_HINTS (f)); + xfree (FRAME_SIZE_HINTS (f)); xfree (f->output_data.mac); f->output_data.mac = NULL; @@ -7640,8 +7638,7 @@ { if (pat->buf) { - if (pat->blocks) - xfree (pat->blocks); + xfree (pat->blocks); xfree (pat->buf); } xfree (pat); @@ -8364,8 +8361,7 @@ HASH_VALUE (h, j)); prev_family = family; } - if (font_ids) - xfree (font_ids); + xfree (font_ids); } #endif @@ -9242,20 +9238,17 @@ int i; for (i = font->min_byte1; i <= font->max_byte1; i++) - if (font->bounds.rows[i]) - xfree (font->bounds.rows[i]); + xfree (font->bounds.rows[i]); xfree (font->bounds.rows); ATSUDisposeStyle (font->mac_style); } else #endif - if (font->bounds.per_char) - xfree (font->bounds.per_char); + xfree (font->bounds.per_char); #if USE_CG_TEXT_DRAWING if (font->cg_font) CGFontRelease (font->cg_font); - if (font->cg_glyphs) - xfree (font->cg_glyphs); + xfree (font->cg_glyphs); #endif xfree (font); } @@ -13026,12 +13019,10 @@ if (dpyinfo->font_table) { - if (dpyinfo->font_table->font_encoder) - xfree (dpyinfo->font_table->font_encoder); + xfree (dpyinfo->font_table->font_encoder); xfree (dpyinfo->font_table); } - if (dpyinfo->mac_id_name) - xfree (dpyinfo->mac_id_name); + xfree (dpyinfo->mac_id_name); if (x_display_list == 0) { diff -r 1c16540a2329 -r 4390d64d3328 src/mactoolbox.c --- a/src/mactoolbox.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/mactoolbox.c Mon Jun 02 06:09:17 2008 +0000 @@ -6091,8 +6091,7 @@ if (err != noErr) break; } - if (buf) - xfree (buf); + xfree (buf); if (err == noErr) { diff -r 1c16540a2329 -r 4390d64d3328 src/term.c --- a/src/term.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/term.c Mon Jun 02 06:09:17 2008 +0000 @@ -2090,17 +2090,14 @@ if (save) { - if (default_orig_pair) - xfree (default_orig_pair); + xfree (default_orig_pair); default_orig_pair = tty->TS_orig_pair ? xstrdup (tty->TS_orig_pair) : NULL; - if (default_set_foreground) - xfree (default_set_foreground); + xfree (default_set_foreground); default_set_foreground = tty->TS_set_foreground ? xstrdup (tty->TS_set_foreground) : NULL; - if (default_set_background) - xfree (default_set_background); + xfree (default_set_background); default_set_background = tty->TS_set_background ? xstrdup (tty->TS_set_background) : NULL; @@ -3833,8 +3830,7 @@ struct terminal *terminal; char *str1, *str2, *arg1, *arg2; { - if (buffer) - xfree (buffer); + xfree (buffer); if (terminal) delete_tty (terminal); @@ -3915,11 +3911,8 @@ delete_terminal (terminal); - if (tty->name) - xfree (tty->name); - - if (tty->type) - xfree (tty->type); + xfree (tty->name); + xfree (tty->type); if (tty->input) { @@ -3932,11 +3925,8 @@ if (tty->termscript) fclose (tty->termscript); - if (tty->old_tty) - xfree (tty->old_tty); - - if (tty->Wcm) - xfree (tty->Wcm); + xfree (tty->old_tty); + xfree (tty->Wcm); bzero (tty, sizeof (struct tty_display_info)); xfree (tty); diff -r 1c16540a2329 -r 4390d64d3328 src/w16select.c --- a/src/w16select.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/w16select.c Mon Jun 02 06:09:17 2008 +0000 @@ -565,8 +565,7 @@ ok = 0; unblock: - if (dst) - xfree (dst); + xfree (dst); UNBLOCK_INPUT; /* Notify user if the text is too large to fit into DOS memory. diff -r 1c16540a2329 -r 4390d64d3328 src/w32.c --- a/src/w32.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/w32.c Mon Jun 02 06:09:17 2008 +0000 @@ -1128,7 +1128,7 @@ return (lpvalue); } - if (lpvalue) xfree (lpvalue); + xfree (lpvalue); RegCloseKey (hrootkey); } @@ -1145,7 +1145,7 @@ return (lpvalue); } - if (lpvalue) xfree (lpvalue); + xfree (lpvalue); RegCloseKey (hrootkey); } @@ -1346,7 +1346,7 @@ /* Also ignore empty environment variables. */ || *lpval == 0) { - if (lpval) xfree (lpval); + xfree (lpval); lpval = env_vars[i].def_value; dwType = REG_EXPAND_SZ; dont_free = 1; @@ -2962,8 +2962,7 @@ get_file_owner_and_group (NULL, name, buf); } - if (psd) - xfree (psd); + xfree (psd); #if 0 /* Not sure if there is any point in this. */ diff -r 1c16540a2329 -r 4390d64d3328 src/w32bdf.c --- a/src/w32bdf.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/w32bdf.c Mon Jun 02 06:09:17 2008 +0000 @@ -304,10 +304,10 @@ CloseHandle(fontp->hfilemap); CloseHandle(fontp->hfile); - if (fontp->registry) xfree(fontp->registry); - if (fontp->encoding) xfree(fontp->encoding); - if (fontp->slant) xfree(fontp->slant); -/* if (fontp->width) xfree(fontp->width); */ + xfree(fontp->registry); + xfree(fontp->encoding); + xfree(fontp->slant); +/* xfree(fontp->width); */ xfree(fontp->filename); for(i = 0;i < BDF_FIRST_OFFSET_TABLE;i++) diff -r 1c16540a2329 -r 4390d64d3328 src/w32fns.c --- a/src/w32fns.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/w32fns.c Mon Jun 02 06:09:17 2008 +0000 @@ -4955,7 +4955,7 @@ { if (font) { - if (font->per_char) xfree (font->per_char); + xfree (font->per_char); if (font->bdf) w32_free_bdf_font (font->bdf); if (font->hfont) DeleteObject (font->hfont); diff -r 1c16540a2329 -r 4390d64d3328 src/window.c --- a/src/window.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/window.c Mon Jun 02 06:09:17 2008 +0000 @@ -3195,7 +3195,7 @@ last_pos += new_size; } - if (new_sizes) xfree (new_sizes); + xfree (new_sizes); /* We should have covered the parent exactly with child windows. */ xassert (size == last_pos - first_pos); diff -r 1c16540a2329 -r 4390d64d3328 src/xselect.c --- a/src/xselect.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/xselect.c Mon Jun 02 06:09:17 2008 +0000 @@ -1673,7 +1673,7 @@ XSelectInput (display, window, STANDARD_EVENT_SET); /* Use xfree, not XFree, because x_get_window_property calls xmalloc itself. */ - if (tmp_data) xfree (tmp_data); + xfree (tmp_data); break; } diff -r 1c16540a2329 -r 4390d64d3328 src/xterm.c --- a/src/xterm.c Mon Jun 02 06:02:30 2008 +0000 +++ b/src/xterm.c Mon Jun 02 06:09:17 2008 +0000 @@ -9513,9 +9513,7 @@ XFlush (FRAME_X_DISPLAY (f)); } - if (f->output_data.x->saved_menu_event) - xfree (f->output_data.x->saved_menu_event); - + xfree (f->output_data.x->saved_menu_event); xfree (f->output_data.x); f->output_data.x = NULL; @@ -10522,10 +10520,8 @@ xim_close_dpy (dpyinfo); #endif - if (dpyinfo->x_id_name) - xfree (dpyinfo->x_id_name); - if (dpyinfo->color_cells) - xfree (dpyinfo->color_cells); + xfree (dpyinfo->x_id_name); + xfree (dpyinfo->color_cells); xfree (dpyinfo); }