]> git.eshelyaron.com Git - emacs.git/commitdiff
Minor fixes for signed vs unsigned integers.
authorPaul Eggert <eggert@cs.ucla.edu>
Mon, 6 Jun 2011 05:48:28 +0000 (22:48 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Mon, 6 Jun 2011 05:48:28 +0000 (22:48 -0700)
* character.h (MAYBE_UNIFY_CHAR):
* charset.c (maybe_unify_char):
* keyboard.c (read_char, reorder_modifiers):
XINT -> XFASTINT, since the integer must be nonnegative.
* ftfont.c (ftfont_spec_pattern):
* keymap.c (access_keymap, silly_event_symbol_error):
XUINT -> XFASTINT, since the integer must be nonnegative.
(Fsingle_key_description, preferred_sequence_p): XUINT -> XINT,
since it makes no difference and we prefer signed.
* keyboard.c (record_char): Use XUINT when all the neighbors do.
(access_keymap): NATNUMP -> INTEGERP, since the integer must be
nonnegative.

1  2 
src/ChangeLog
src/keyboard.c

diff --cc src/ChangeLog
index 230e282e210f5b6130cb5fb04cce1f4448151202,431bd2acdb54853d2d8d25f736085e89ce72388f..05d0baa3ac5a7a307b89b7f2b639ac97ba2ffb36
 -2011-06-03  Paul Eggert  <eggert@cs.ucla.edu>
++2011-06-06  Paul Eggert  <eggert@cs.ucla.edu>
+       Minor fixes for signed vs unsigned integers.
+       * character.h (MAYBE_UNIFY_CHAR):
+       * charset.c (maybe_unify_char):
+       * keyboard.c (read_char, reorder_modifiers):
+       XINT -> XFASTINT, since the integer must be nonnegative.
+       * ftfont.c (ftfont_spec_pattern):
+       * keymap.c (access_keymap, silly_event_symbol_error):
+       XUINT -> XFASTINT, since the integer must be nonnegative.
+       (Fsingle_key_description, preferred_sequence_p): XUINT -> XINT,
+       since it makes no difference and we prefer signed.
+       * keyboard.c (record_char): Use XUINT when all the neighbors do.
+       (access_keymap): NATNUMP -> INTEGERP, since the integer must be
+       nonnegative.
 +2011-06-06  Paul Eggert  <eggert@cs.ucla.edu>
 +
 +      * alloc.c (memory_full) [SYSTEM_MALLOC]: Port to MacOS (Bug#8800).
 +      Do not assume that spare memory exists; that assumption is valid
 +      only if SYSTEM_MALLOC.
 +      (LARGE_REQUEST): New macro, so that the issue of large requests
 +      is separated from the issue of spare memory.
 +
 +2011-06-05  Andreas Schwab  <schwab@linux-m68k.org>
 +
 +      * editfns.c (Fformat): Correctly handle zero flag with hexadecimal
 +      format.  (Bug#8806)
 +
 +      * gtkutil.c (xg_get_default_scrollbar_width): Avoid warning.
 +
 +      * xfns.c (x_set_scroll_bar_default_width): Move declarations
 +      before statements.
 +
 +2011-06-05  Jan Djärv  <jan.h.d@swipnet.se>
 +
 +      * gtkutil.c (xg_get_default_scrollbar_width): New function.
 +
 +      * gtkutil.h: Declare xg_get_default_scrollbar_width.
 +
 +      * xfns.c (x_set_scroll_bar_default_width): If USE_GTK, get
 +      min width by calling x_set_scroll_bar_default_width (Bug#8505).
 +
 +2011-06-05  Juanma Barranquero  <lekktu@gmail.com>
 +
 +      * xdisp.c (single_display_spec_intangible_p): Remove declaration.
 +
 +2011-06-04  Chong Yidong  <cyd@stupidchicken.com>
 +
 +      * xselect.c (x_clipboard_manager_save): Remove redundant arg.
 +      (x_clipboard_manager_save): Add return value.
 +      (x_clipboard_manager_error_1, x_clipboard_manager_error_2): New
 +      error handlers.
 +      (x_clipboard_manager_save_frame, x_clipboard_manager_save_all):
 +      Obey Vx_select_enable_clipboard_manager.  Catch errors in
 +      x_clipboard_manager_save (Bug#8779).
 +      (Vx_select_enable_clipboard_manager): New variable.
 +      (x_get_foreign_selection): Reduce scope of x_catch_errors (Bug#8790).
 +
 +2011-06-04  Dan Nicolaescu  <dann@ics.uci.edu>
 +
 +      * emacs.c (main): Warn when starting a GTK emacs in daemon mode.
 +
 +2011-06-04  YAMAMOTO Mitsuharu  <mituharu@math.s.chiba-u.ac.jp>
 +
 +      * fringe.c (update_window_fringes): Don't update overlay arrow bitmap
 +      in the current matrix if keep_current_p is non-zero.
 +
 +2011-06-04  Eli Zaretskii  <eliz@gnu.org>
 +
 +      * bidi.c (bidi_level_of_next_char): Fix last change.
 +
 +2011-06-03  Eli Zaretskii  <eliz@gnu.org>
 +
 +      Support bidi reordering of text covered by display properties.
 +
 +      * bidi.c (bidi_copy_it): Use offsetof instead of emulating it.
 +      (bidi_fetch_char, bidi_fetch_char_advance): New functions.
 +      (bidi_cache_search, bidi_cache_iterator_state)
 +      (bidi_paragraph_init, bidi_resolve_explicit, bidi_resolve_weak)
 +      (bidi_level_of_next_char, bidi_move_to_visually_next): Support
 +      character positions inside a run of characters covered by a
 +      display string.
 +      (bidi_paragraph_init, bidi_resolve_explicit_1)
 +      (bidi_level_of_next_char): Call bidi_fetch_char and
 +      bidi_fetch_char_advance instead of FETCH_CHAR and
 +      FETCH_CHAR_ADVANCE.
 +      (bidi_init_it): Initialize new members.
 +      (LRE_CHAR, RLE_CHAR, PDF_CHAR, LRO_CHAR, RLO_CHAR): Remove macro
 +      definitions.
 +      (bidi_explicit_dir_char): Lookup character type in bidi_type_table,
 +      instead of using explicit *_CHAR codes.
 +      (bidi_resolve_explicit, bidi_resolve_weak): Use
 +      FETCH_MULTIBYTE_CHAR instead of FETCH_CHAR, as reordering of
 +      bidirectional text is supported only in multibyte buffers.
 +      (bidi_init_it): Accept additional argument FRAME_WINDOW_P and use
 +      it to initialize the frame_window_p member of struct bidi_it.
 +      (bidi_cache_iterator_state, bidi_resolve_explicit_1)
 +      (bidi_resolve_explicit, bidi_resolve_weak)
 +      (bidi_level_of_next_char, bidi_move_to_visually_next): Abort if
 +      bidi_it->nchars is non-positive.
 +      (bidi_level_of_next_char): Don't try to lookup the cache for the
 +      next/previous character if nothing is cached there yet, or if we
 +      were just reseat()'ed to a new position.
 +
 +      * xdisp.c (set_cursor_from_row): Set start and stop points
 +      according to the row's direction when priming the loop that looks
 +      for the glyph on which to display cursor.
 +      (single_display_spec_intangible_p): Function deleted.
 +      (display_prop_intangible_p): Reimplement to call
 +      handle_display_spec instead of single_display_spec_intangible_p.
 +      Accept 3 additional arguments needed by handle_display_spec.  This
 +      fixes incorrect cursor motion across display property with complex
 +      values: lists, `(when COND...)' forms, etc.
 +      (single_display_spec_string_p): Support property values that are
 +      lists with the argument STRING its top-level element.
 +      (display_prop_string_p): Fix the condition for processing a
 +      property that is a list to be consistent with handle_display_spec.
 +      (handle_display_spec): New function, refactored from the
 +      last portion of handle_display_prop.
 +      (compute_display_string_pos): Accept additional argument
 +      FRAME_WINDOW_P.  Call handle_display_spec to determine whether the
 +      value of a `display' property is a "replacing spec".
 +      (handle_single_display_spec): Accept 2 additional arguments BUFPOS
 +      and FRAME_WINDOW_P.  If IT is NULL, don't set up the iterator from
 +      the display property, but just return a value indicating whether
 +      the display property will replace the characters it covers.
 +      (Fcurrent_bidi_paragraph_direction): Initialize the nchars and
 +      frame_window_p members of struct bidi_it.
 +      (compute_display_string_pos, compute_display_string_end): New
 +      functions.
 +      (push_it): Accept second argument POSITION, where pop_it should
 +      jump to continue iteration.
 +      (reseat_1): Initialize bidi_it.disp_pos.
 +
 +      * keyboard.c (adjust_point_for_property): Adjust the call to
 +      display_prop_intangible_p to its new signature.
 +
 +      * dispextern.h (struct bidi_it): New member frame_window_p.
 +      (bidi_init_it): Update prototypes.
 +      (display_prop_intangible_p): Update prototype.
 +      (compute_display_string_pos, compute_display_string_end): Declare
 +      prototypes.
 +      (struct bidi_it): New members nchars and disp_pos.  ch_len is now
 +      EMACS_INT.
 +
  2011-06-02  Paul Eggert  <eggert@cs.ucla.edu>
  
        Malloc failure behavior now depends on size of allocation.
diff --cc src/keyboard.c
Simple merge