Mercurial > hg > xemacs-beta
comparison src/file-coding.c @ 4976:16112448d484
Rename xfree(FOO, TYPE) -> xfree(FOO)
-------------------- ChangeLog entries follow: --------------------
src/ChangeLog addition:
2010-02-04 Ben Wing <ben@xemacs.org>
* alloc.c (release_breathing_space):
* alloc.c (resize_string):
* alloc.c (sweep_lcrecords_1):
* alloc.c (SWEEP_FIXED_TYPE_BLOCK_1):
* alloc.c (ADDITIONAL_FREE_compiled_function):
* alloc.c (compact_string_chars):
* alloc.c (ADDITIONAL_FREE_string):
* alloc.c (sweep_strings):
* alloca.c (xemacs_c_alloca):
* alsaplay.c (alsa_play_sound_file):
* buffer.c (init_initial_directory):
* buffer.h:
* buffer.h (BUFFER_FREE):
* console-stream.c (stream_delete_console):
* console-tty.c (free_tty_console_struct):
* data.c (Fnumber_to_string):
* device-gtk.c (gtk_init_device):
* device-gtk.c (free_gtk_device_struct):
* device-gtk.c (gtk_delete_device):
* device-msw.c (mswindows_delete_device):
* device-msw.c (msprinter_delete_device):
* device-tty.c (free_tty_device_struct):
* device-tty.c (tty_delete_device):
* device-x.c (x_init_device):
* device-x.c (free_x_device_struct):
* device-x.c (x_delete_device):
* dialog-msw.c (handle_directory_dialog_box):
* dialog-x.c (dbox_descriptor_to_widget_value):
* dired-msw.c (Fmswindows_insert_directory):
* dired.c (free_user_cache):
* dired.c (user_name_completion_unwind):
* doc.c (unparesseuxify_doc_string):
* doc.c (Fsubstitute_command_keys):
* doprnt.c (emacs_doprnt_1):
* dumper.c (pdump_load_finish):
* dumper.c (pdump_file_free):
* dumper.c (pdump_file_unmap):
* dynarr.c:
* dynarr.c (Dynarr_free):
* editfns.c (uncache_home_directory):
* editfns.c (Fset_time_zone_rule):
* elhash.c:
* elhash.c (pdump_reorganize_hash_table):
* elhash.c (maphash_unwind):
* emacs.c (make_arg_list_1):
* emacs.c (free_argc_argv):
* emacs.c (sort_args):
* emacs.c (Frunning_temacs_p):
* emodules.c (attempt_module_delete):
* eval.c (free_pointer):
* event-Xt.c (unselect_filedesc):
* event-Xt.c (emacs_Xt_select_process):
* event-gtk.c (unselect_filedesc):
* event-gtk.c (dragndrop_data_received):
* event-msw.c (winsock_closer):
* event-msw.c (mswindows_dde_callback):
* event-msw.c (mswindows_wnd_proc):
* event-stream.c (finalize_command_builder):
* event-stream.c (free_command_builder):
* extents.c (free_gap_array):
* extents.c (free_extent_list):
* extents.c (free_soe):
* extents.c (extent_fragment_delete):
* extents.c (extent_priority_sort_function):
* file-coding.c (make_coding_system_1):
* file-coding.c (coding_finalizer):
* file-coding.c (set_coding_stream_coding_system):
* file-coding.c (chain_finalize_coding_stream_1):
* file-coding.c (chain_finalize):
* file-coding.c (free_detection_state):
* file-coding.c (coding_category_symbol_to_id):
* fileio.c:
* fileio.c (Ffile_name_directory):
* fileio.c (if):
* fileio.c (Ffile_symlink_p):
* filelock.c (FREE_LOCK_INFO):
* filelock.c (current_lock_owner):
* font-mgr.c (Ffc_name_unparse):
* font-mgr.c (Ffc_pattern_duplicate):
* frame-gtk.c (gtk_delete_frame):
* frame-msw.c (mswindows_delete_frame):
* frame-msw.c (msprinter_delete_frame):
* frame-x.c (x_cde_destroy_callback):
* frame-x.c (Fcde_start_drag_internal):
* frame-x.c (x_cde_transfer_callback):
* frame-x.c (x_delete_frame):
* frame.c (update_frame_title):
* frame.c (Fset_frame_pointer):
* gc.c (register_for_finalization):
* gccache-gtk.c (free_gc_cache):
* gccache-gtk.c (gc_cache_lookup):
* gccache-x.c (free_gc_cache):
* gccache-x.c (gc_cache_lookup):
* glyphs-eimage.c:
* glyphs-eimage.c (jpeg_instantiate_unwind):
* glyphs-eimage.c (gif_instantiate_unwind):
* glyphs-eimage.c (png_instantiate_unwind):
* glyphs-eimage.c (png_instantiate):
* glyphs-eimage.c (tiff_instantiate_unwind):
* glyphs-gtk.c (convert_EImage_to_GDKImage):
* glyphs-gtk.c (gtk_finalize_image_instance):
* glyphs-gtk.c (gtk_init_image_instance_from_eimage):
* glyphs-gtk.c (gtk_xpm_instantiate):
* glyphs-msw.c (convert_EImage_to_DIBitmap):
* glyphs-msw.c (mswindows_init_image_instance_from_eimage):
* glyphs-msw.c (mswindows_initialize_image_instance_mask):
* glyphs-msw.c (xpm_to_eimage):
* glyphs-msw.c (mswindows_xpm_instantiate):
* glyphs-msw.c (xbm_create_bitmap_from_data):
* glyphs-msw.c (mswindows_finalize_image_instance):
* glyphs-x.c (convert_EImage_to_XImage):
* glyphs-x.c (x_finalize_image_instance):
* glyphs-x.c (x_init_image_instance_from_eimage):
* glyphs-x.c (x_xpm_instantiate):
* gui-x.c (free_popup_widget_value_tree):
* hash.c (free_hash_table):
* hash.c (grow_hash_table):
* hash.c (pregrow_hash_table_if_necessary):
* imgproc.c (build_EImage_quantable):
* insdel.c (uninit_buffer_text):
* intl-win32.c (convert_multibyte_to_internal_malloc):
* intl.c:
* intl.c (Fset_current_locale):
* keymap.c:
* keymap.c (where_is_recursive_mapper):
* keymap.c (where_is_internal):
* lisp.h:
* lisp.h (xfree):
* lstream.c (Lstream_close):
* lstream.c (resizing_buffer_closer):
* mule-coding.c:
* mule-coding.c (iso2022_finalize_detection_state):
* nt.c:
* nt.c (mswindows_get_long_filename):
* nt.c (nt_get_resource):
* nt.c (init_mswindows_environment):
* nt.c (get_cached_volume_information):
* nt.c (mswindows_opendir):
* nt.c (mswindows_closedir):
* nt.c (mswindows_readdir):
* nt.c (mswindows_stat):
* nt.c (mswindows_getdcwd):
* nt.c (Fmswindows_long_file_name):
* ntplay.c (nt_play_sound_file):
* ntplay.c (play_sound_data_1):
* number-gmp.c (gmp_free):
* number-gmp.c (init_number_gmp):
* number-mp.c (bignum_to_string):
* number-mp.c (BIGNUM_TO_TYPE):
* number.c (bignum_print):
* number.c (bignum_convfree):
* number.c (ratio_print):
* number.c (bigfloat_print):
* number.c (bigfloat_finalize):
* objects-gtk.c (gtk_finalize_color_instance):
* objects-gtk.c (gtk_finalize_font_instance):
* objects-msw.c (mswindows_finalize_color_instance):
* objects-msw.c (mswindows_finalize_font_instance):
* objects-tty.c (tty_finalize_color_instance):
* objects-tty.c (tty_finalize_font_instance):
* objects-tty.c (tty_font_list):
* objects-x.c (x_finalize_color_instance):
* objects-x.c (x_finalize_font_instance):
* process.c:
* process.c (finalize_process):
* realpath.c:
* redisplay.c (add_propagation_runes):
* regex.c:
* regex.c (xfree):
* regex.c (REGEX_FREE_STACK):
* regex.c (FREE_STACK_RETURN):
* regex.c (regex_compile):
* regex.c (regexec):
* regex.c (regfree):
* scrollbar-gtk.c (gtk_free_scrollbar_instance):
* scrollbar-gtk.c (gtk_release_scrollbar_instance):
* scrollbar-msw.c (mswindows_free_scrollbar_instance):
* scrollbar-msw.c (unshow_that_mofo):
* scrollbar-x.c (x_free_scrollbar_instance):
* scrollbar-x.c (x_release_scrollbar_instance):
* select-gtk.c (emacs_gtk_selection_handle):
* select-msw.c (mswindows_own_selection):
* select-x.c:
* select-x.c (x_handle_selection_request):
* select-x.c (unexpect_property_change):
* select-x.c (x_handle_property_notify):
* select-x.c (receive_incremental_selection):
* select-x.c (x_get_window_property_as_lisp_data):
* select-x.c (Fx_get_cutbuffer_internal):
* specifier.c (finalize_specifier):
* syntax.c (uninit_buffer_syntax_cache):
* sysdep.c (qxe_allocating_getcwd):
* sysdep.c (qxe_lstat):
* sysdep.c (copy_in_passwd):
* sysdep.c (qxe_ctime):
* sysdep.c (closedir):
* sysdep.c (DIRSIZ):
* termcap.c (tgetent):
* termcap.c (tprint):
* tests.c (Ftest_data_format_conversion):
* text.c (new_dfc_convert_copy_data):
* text.h (eifree):
* text.h (eito_alloca):
* text.h (eito_external):
* toolbar-msw.c (mswindows_output_toolbar):
* ui-gtk.c (CONVERT_RETVAL):
* ui-gtk.c (__allocate_object_storage):
* unicode.c (free_from_unicode_table):
* unicode.c (free_to_unicode_table):
* unicode.c (free_charset_unicode_tables):
* win32.c (mswindows_read_link_1):
Rename: xfree(VAL, TYPE)->xfree(VAL)
Command used:
gr 'xfree *\((.*),.*\);' 'xfree (\1);' *.[ch]
Followed by grepping for 'xfree.*,' and fixing anything left.
Rationale: Having to specify the TYPE argument is annoying and
error-prone. It was originally put in to work around warnings
due to strict aliasing but years and years ago I rewrote it
in a way that doesn't use the TYPE argument at all and no one
has complained since then. (And anyway, XEmacs is far from
ever being in compliance with strict aliasing and would require
far-reaching changes to get that way.)
author | Ben Wing <ben@xemacs.org> |
---|---|
date | Thu, 04 Feb 2010 07:28:14 -0600 |
parents | 0d4c9d0f6a8d |
children | 9410323e4b0d b5df3737028a |
comparison
equal
deleted
inserted
replaced
4972:c448f4c38d65 | 4976:16112448d484 |
---|---|
1206 NILP (name_or_existing) ? (Ibyte *) "nil" : | 1206 NILP (name_or_existing) ? (Ibyte *) "nil" : |
1207 XSTRING_DATA (Fsymbol_name (XCODING_SYSTEM_NAME | 1207 XSTRING_DATA (Fsymbol_name (XCODING_SYSTEM_NAME |
1208 (name_or_existing))), | 1208 (name_or_existing))), |
1209 ++coding_system_tick); | 1209 ++coding_system_tick); |
1210 name_or_existing = intern_istring (newname); | 1210 name_or_existing = intern_istring (newname); |
1211 xfree (newname, Ibyte *); | 1211 xfree (newname); |
1212 | 1212 |
1213 if (UNBOUNDP (description)) | 1213 if (UNBOUNDP (description)) |
1214 { | 1214 { |
1215 newname = | 1215 newname = |
1216 emacs_sprintf_malloc | 1216 emacs_sprintf_malloc |
1217 (NULL, "For Internal Use (%s)", | 1217 (NULL, "For Internal Use (%s)", |
1218 XSTRING_DATA (Fsymbol_name (name_or_existing))); | 1218 XSTRING_DATA (Fsymbol_name (name_or_existing))); |
1219 description = build_istring (newname); | 1219 description = build_istring (newname); |
1220 xfree (newname, Ibyte *); | 1220 xfree (newname); |
1221 } | 1221 } |
1222 | 1222 |
1223 newname = emacs_sprintf_malloc (NULL, "Int%d", coding_system_tick); | 1223 newname = emacs_sprintf_malloc (NULL, "Int%d", coding_system_tick); |
1224 defmnem = build_istring (newname); | 1224 defmnem = build_istring (newname); |
1225 xfree (newname, Ibyte *); | 1225 xfree (newname); |
1226 } | 1226 } |
1227 else | 1227 else |
1228 CHECK_SYMBOL (name_or_existing); | 1228 CHECK_SYMBOL (name_or_existing); |
1229 | 1229 |
1230 /* See is there an entry for name_or_existing in the defined coding system | 1230 /* See is there an entry for name_or_existing in the defined coding system |
1370 (NULL, "internal-eol-copy-%s-%d", | 1370 (NULL, "internal-eol-copy-%s-%d", |
1371 XSTRING_DATA (Fsymbol_name (name_or_existing)), | 1371 XSTRING_DATA (Fsymbol_name (name_or_existing)), |
1372 ++coding_system_tick); | 1372 ++coding_system_tick); |
1373 Lisp_Object newnamesym = intern_istring (newname); | 1373 Lisp_Object newnamesym = intern_istring (newname); |
1374 Lisp_Object copied = Fcopy_coding_system (csobj, newnamesym); | 1374 Lisp_Object copied = Fcopy_coding_system (csobj, newnamesym); |
1375 xfree (newname, Ibyte *); | 1375 xfree (newname); |
1376 | 1376 |
1377 XCODING_SYSTEM_CANONICAL (csobj) = | 1377 XCODING_SYSTEM_CANONICAL (csobj) = |
1378 make_internal_coding_system | 1378 make_internal_coding_system |
1379 (csobj, | 1379 (csobj, |
1380 "internal-eol-wrapper", | 1380 "internal-eol-wrapper", |
2051 | 2051 |
2052 assert (!str->finalized); | 2052 assert (!str->finalized); |
2053 MAYBE_XCODESYSMETH (str->codesys, finalize_coding_stream, (str)); | 2053 MAYBE_XCODESYSMETH (str->codesys, finalize_coding_stream, (str)); |
2054 if (str->data) | 2054 if (str->data) |
2055 { | 2055 { |
2056 xfree (str->data, void *); | 2056 xfree (str->data); |
2057 str->data = 0; | 2057 str->data = 0; |
2058 } | 2058 } |
2059 str->finalized = 1; | 2059 str->finalized = 1; |
2060 } | 2060 } |
2061 | 2061 |
2110 str->orig_codesys = codesys; | 2110 str->orig_codesys = codesys; |
2111 str->codesys = coding_system_real_canonical (codesys); | 2111 str->codesys = coding_system_real_canonical (codesys); |
2112 | 2112 |
2113 if (str->data) | 2113 if (str->data) |
2114 { | 2114 { |
2115 xfree (str->data, void *); | 2115 xfree (str->data); |
2116 str->data = 0; | 2116 str->data = 0; |
2117 } | 2117 } |
2118 if (XCODING_SYSTEM_METHODS (str->codesys)->coding_data_size) | 2118 if (XCODING_SYSTEM_METHODS (str->codesys)->coding_data_size) |
2119 { | 2119 { |
2120 str->data = | 2120 str->data = |
2717 both directions, you'd have to disconnect the pointers to a | 2717 both directions, you'd have to disconnect the pointers to a |
2718 particular object before deleting it. */ | 2718 particular object before deleting it. */ |
2719 for (i = 0; i < data->lstream_count; i++) | 2719 for (i = 0; i < data->lstream_count; i++) |
2720 Lstream_delete (XLSTREAM ((data->lstreams)[i])); | 2720 Lstream_delete (XLSTREAM ((data->lstreams)[i])); |
2721 } | 2721 } |
2722 xfree (data->lstreams, Lisp_Object *); | 2722 xfree (data->lstreams); |
2723 } | 2723 } |
2724 } | 2724 } |
2725 | 2725 |
2726 static void | 2726 static void |
2727 chain_finalize_coding_stream (struct coding_stream *str) | 2727 chain_finalize_coding_stream (struct coding_stream *str) |
2731 | 2731 |
2732 static void | 2732 static void |
2733 chain_finalize (Lisp_Object c) | 2733 chain_finalize (Lisp_Object c) |
2734 { | 2734 { |
2735 if (XCODING_SYSTEM_CHAIN_CHAIN (c)) | 2735 if (XCODING_SYSTEM_CHAIN_CHAIN (c)) |
2736 xfree (XCODING_SYSTEM_CHAIN_CHAIN (c), Lisp_Object *); | 2736 xfree (XCODING_SYSTEM_CHAIN_CHAIN (c)); |
2737 } | 2737 } |
2738 | 2738 |
2739 static int | 2739 static int |
2740 chain_putprop (Lisp_Object codesys, Lisp_Object key, Lisp_Object value) | 2740 chain_putprop (Lisp_Object codesys, Lisp_Object key, Lisp_Object value) |
2741 { | 2741 { |
3358 if (Dynarr_at (all_coding_detectors, i).finalize_detection_state_method) | 3358 if (Dynarr_at (all_coding_detectors, i).finalize_detection_state_method) |
3359 Dynarr_at (all_coding_detectors, i).finalize_detection_state_method | 3359 Dynarr_at (all_coding_detectors, i).finalize_detection_state_method |
3360 (st); | 3360 (st); |
3361 } | 3361 } |
3362 | 3362 |
3363 xfree (st, struct detection_state *); | 3363 xfree (st); |
3364 } | 3364 } |
3365 | 3365 |
3366 static int | 3366 static int |
3367 coding_category_symbol_to_id (Lisp_Object symbol) | 3367 coding_category_symbol_to_id (Lisp_Object symbol) |
3368 { | 3368 { |