]> git.eshelyaron.com Git - emacs.git/commitdiff
Merge: Use socklen_t, not int, for socket lengths.
authorPaul Eggert <eggert@cs.ucla.edu>
Sun, 20 Mar 2011 16:55:33 +0000 (09:55 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Sun, 20 Mar 2011 16:55:33 +0000 (09:55 -0700)
1  2 
src/ChangeLog
src/process.c

diff --cc src/ChangeLog
index 744d3ef53f078646c85a68665e3d001fcf5483b9,f1a6dcb3c1764365379d59abd1eb6d1920830a2b..0b2a0d44c5f46f22943f083b5f69def60a0ee392
 -2011-03-18  Paul Eggert  <eggert@cs.ucla.edu>
 +2011-03-20  Paul Eggert  <eggert@cs.ucla.edu>
  
+       * process.c (Fmake_network_process): Use socklen_t, not int,
+       where POSIX says socklen_t is required in portable programs.
+       This fixes a porting bug on hosts like 64-bit HP-UX, where
+       socklen_t is wider than int (Bug#8277).
+       (Fmake_network_process, server_accept_connection):
+       (wait_reading_process_output, read_process_output):
+       Likewise.
 +      * process.c: Rename or move locals to avoid shadowing.
 +      (list_processes_1, Fmake_network_process):
 +      (read_process_output_error_handler, exec_sentinel_error_handler):
 +      Rename or move locals.
 +      (Fmake_network_process): Define label "retry_connect" only if needed.
 +      (Fnetwork_interface_info): Fix pointer signedness.
 +      (process_send_signal): Add cast to avoid pointer signedness problem.
 +      (FIRST_PROC_DESC, IF_NON_BLOCKING_CONNECT): Remove unused macros.
 +      (create_process): Use 'volatile' to avoid vfork clobbering (Bug#8298).
 +
 +2011-03-19  Paul Eggert  <eggert@cs.ucla.edu>
 +
 +      Make tparam.h and terminfo.c consistent.
 +      * cm.c (tputs, tgoto, BC, UP): Remove extern decls.  Include
 +      tparam.h instead, since it declares them.
 +      * cm.h (PC): Remove extern decl; tparam.h now does this.
 +      * deps.mk (cm.o, terminfo.o): Depend on tparam.h.
 +      * terminfo.c: Include tparam.h, to check interfaces.
 +      (tparm): Make 1st arg a const pointer in decl.  Put it at top level.
 +      (tparam): Adjust signature to match interface in tparam.h;
 +      this removes some undefined behavior.  Check that outstring and len
 +      are zero, which they always are with Emacs.
 +      * tparam.h (PC, BC, UP): New extern decls.
 +
 +      * xftfont.c (xftfont_shape): Now static, and defined only if needed.
 +      (xftfont_open): Rename locals to avoid shadowing.
 +
 +      * ftfont.c (ftfont_resolve_generic_family): Fix pointer signedness.
 +      (ftfont_otf_capability, ftfont_shape): Omit decls if not needed.
 +      (OTF_TAG_SYM): Omit macro if not needed.
 +      (ftfont_list): Remove unused local.
 +      (get_adstyle_property, ftfont_pattern_entity):
 +      (ftfont_lookup_cache, ftfont_open, ftfont_anchor_point):
 +      Rename locals to avoid shadowing.
 +
 +      * xfont.c (xfont_list_family): Mark var as initialized.
 +
 +      * xml.c (make_dom): Now static.
 +
 +      * composite.c (composition_compute_stop_pos): Rename local to
 +      avoid shadowing.
 +      (composition_reseat_it): Remove unused locals.
 +      (find_automatic_composition, composition_adjust_point): Likewise.
 +      (composition_update_it): Mark var as initialized.
 +      (find_automatic_composition): Mark vars as initialized,
 +      with a FIXME (Bug#8290).
 +
 +2011-03-18  Paul Eggert  <eggert@cs.ucla.edu>
 +
 +      character.h: Rename locals to avoid shadowing.
 +      * character.h (PREV_CHAR_BOUNDARY, FETCH_STRING_CHAR_ADVANCE):
 +      (FETCH_STRING_CHAR_AS_MULTIBYTE_ADVANCE, FETCH_CHAR_ADVANCE):
 +      (FETCH_CHAR_ADVANCE_NO_CHECK, INC_POS, DEC_POS, BUF_INC_POS):
 +      (BUF_DEC_POS): Be more systematic about renaming local temporaries
 +      to avoid shadowing.
 +
 +      * textprop.c (property_change_between_p): Remove; unused.
 +
 +      * intervals.c (interval_start_pos): Now static.
 +
 +      * intervals.h (CHECK_TOTAL_LENGTH): Avoid empty "else".
 +
 +      * atimer.c (start_atimer, append_atimer_lists, set_alarm): Rename
 +      locals to avoid shadowing.
 +
 +      * sound.c (wav_play, au_play, Fplay_sound_internal):
 +      Fix pointer signedness.
 +      (alsa_choose_format): Remove unused local var.
 +      (wav_play): Initialize a variable to 0, to prevent undefined
 +      behavior (Bug#8278).
 +
 +      * region-cache.c (insert_cache_boundary): Redo var to avoid shadowing.
 +
 +      * region-cache.h (pp_cache): New decl, for gcc -Wmissing-prototypes.
 +
 +      * callproc.c (Fcall_process): Use 'volatile' to avoid vfork
 +      clobbering (Bug#8298).
 +      * sysdep.c (sys_subshell): Likewise.
 +      Previously, the sys_subshell 'volatile' was incorrectly IF_LINTted out.
 +
 +      * lisp.h (child_setup): Now NO_RETURN unless DOS_NT.
 +      This should get cleaned up, so that child_setup has the
 +      same signature on all platforms.
 +
 +      * callproc.c (call_process_cleanup): Now static.
 +      (relocate_fd): Rename locals to avoid shadowing.
 +
 +2011-03-17  Eli Zaretskii  <eliz@gnu.org>
 +
 +      * makefile.w32-in ($(BLD)/unexw32.$(O)): Depend on $(SRC)/unexec.h.
 +
  2011-03-17  Paul Eggert  <eggert@cs.ucla.edu>
  
        Fix more problems found by GCC 4.5.2's static checks.
diff --cc src/process.c
Simple merge