From b7f83adda5a32140811e8e7decc4394d64cada3d Mon Sep 17 00:00:00 2001 From: Paul Eggert Date: Mon, 19 Jan 2015 00:56:18 -0800 Subject: Prefer memset to repeatedly assigning Qnil * alloc.c (allocate_pseudovector): Catch more bogus values. * alloc.c (allocate_pseudovector): * callint.c (Fcall_interactively): * coding.c (syms_of_coding): * fringe.c (init_fringe): Verify that Qnil == 0. * callint.c (Fcall_interactively): * eval.c (Fapply, Ffuncall): * fns.c (mapcar1, larger_vector): * font.c (font_expand_wildcards): * fringe.c (init_fringe): Prefer memset to assigning zeros by hand. * callint.c (Fcall_interactively): Remove duplicate assignment of Qnil to args[i]. * coding.c (syms_of_coding): Prefer LISP_INITIALLY_ZERO to assigning zeros by hand. * fileio.c (Ffile_selinux_context): Rewrite to avoid need for Lisp_Object array. * lisp.h (XLI_BUILTIN_LISPSYM): New macro. (DEFINE_LISP_SYMBOL_END): Use it. (NIL_IS_ZERO): New constant. (memsetnil): New function. --- src/coding.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) (limited to 'src/coding.c') diff --git a/src/coding.c b/src/coding.c index b11143a32fb..77cea77cef5 100644 --- a/src/coding.c +++ b/src/coding.c @@ -11272,13 +11272,10 @@ internal character representation. */); Vtranslation_table_for_input = Qnil; { - Lisp_Object args[coding_arg_undecided_max]; - Lisp_Object plist[16]; - int i; - - for (i = 0; i < coding_arg_undecided_max; i++) - args[i] = Qnil; + verify (NIL_IS_ZERO); + Lisp_Object args[coding_arg_undecided_max] = { LISP_INITIALLY_ZERO, }; + Lisp_Object plist[16]; plist[0] = intern_c_string (":name"); plist[1] = args[coding_arg_name] = Qno_conversion; plist[2] = intern_c_string (":mnemonic"); -- cgit v1.2.1