[Emacs-diffs] /srv/bzr/emacs/trunk r110345: Prefer plain 'static' to 'static inline'.

[Paul Eggert]

------------------------------------------------------------
revno: 110345
fixes bug: http://debbugs.gnu.org/12541
committer: Paul Eggert <address@hidden>
branch nick: trunk
timestamp: Mon 2012-10-01 23:55:27 -0700
message:
  Prefer plain 'static' to 'static inline'.
  
  * bidi.c (bidi_get_type, bidi_check_type, bidi_get_category)
  (bidi_set_sor_type, bidi_push_embedding_level)
  (bidi_pop_embedding_level, bidi_remember_char, bidi_copy_it)
  (bidi_cache_reset, bidi_cache_shrink, bidi_cache_fetch_state)
  (bidi_cache_search, bidi_cache_ensure_space)
  (bidi_cache_iterator_state, bidi_cache_find)
  (bidi_peek_at_next_level, bidi_set_paragraph_end)
  (bidi_count_bytes, bidi_char_at_pos, bidi_fetch_char)
  (bidi_explicit_dir_char, bidi_resolve_neutral_1):
  Now 'static', not 'static inline'.
modified:
  src/ChangeLog
  src/bidi.c

=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2012-10-02 06:30:40 +0000
+++ b/src/ChangeLog     2012-10-02 06:55:27 +0000
@@ -1,5 +1,17 @@
 2012-10-02  Paul Eggert  <address@hidden>
 
+       Prefer plain 'static' to 'static inline' (Bug#12541).
+       * bidi.c (bidi_get_type, bidi_check_type, bidi_get_category)
+       (bidi_set_sor_type, bidi_push_embedding_level)
+       (bidi_pop_embedding_level, bidi_remember_char, bidi_copy_it)
+       (bidi_cache_reset, bidi_cache_shrink, bidi_cache_fetch_state)
+       (bidi_cache_search, bidi_cache_ensure_space)
+       (bidi_cache_iterator_state, bidi_cache_find)
+       (bidi_peek_at_next_level, bidi_set_paragraph_end)
+       (bidi_count_bytes, bidi_char_at_pos, bidi_fetch_char)
+       (bidi_explicit_dir_char, bidi_resolve_neutral_1):
+       Now 'static', not 'static inline'.
+
        Count overruns when profiling; change units to ns.
        * profiler.c (handle_profiler_signal): Count sampling intervals, not ms.
        Give extra weight to samples after overruns, to attempt to count

=== modified file 'src/bidi.c'
--- a/src/bidi.c        2012-10-01 06:19:52 +0000
+++ b/src/bidi.c        2012-10-02 06:55:27 +0000
@@ -96,7 +96,7 @@
 
 /* Return the bidi type of a character CH, subject to the current
    directional OVERRIDE.  */
-static inline bidi_type_t
+static bidi_type_t
 bidi_get_type (int ch, bidi_dir_t override)
 {
   bidi_type_t default_type;
@@ -145,14 +145,14 @@
     }
 }
 
-static inline void
+static void
 bidi_check_type (bidi_type_t type)
 {
   eassert (UNKNOWN_BT <= type && type <= NEUTRAL_ON);
 }
 
 /* Given a bidi TYPE of a character, return its category.  */
-static inline bidi_category_t
+static bidi_category_t
 bidi_get_category (bidi_type_t type)
 {
   switch (type)
@@ -226,7 +226,7 @@
    embedding levels on either side of the run boundary.  Also, update
    the saved info about previously seen characters, since that info is
    generally valid for a single level run.  */
-static inline void
+static void
 bidi_set_sor_type (struct bidi_it *bidi_it, int level_before, int level_after)
 {
   int higher_level = (level_before > level_after ? level_before : level_after);
@@ -257,7 +257,7 @@
 
 /* Push the current embedding level and override status; reset the
    current level to LEVEL and the current override status to OVERRIDE.  */
-static inline void
+static void
 bidi_push_embedding_level (struct bidi_it *bidi_it,
                           int level, bidi_dir_t override)
 {
@@ -269,7 +269,7 @@
 
 /* Pop the embedding level and directional override status from the
    stack, and return the new level.  */
-static inline int
+static int
 bidi_pop_embedding_level (struct bidi_it *bidi_it)
 {
   /* UAX#9 says to ignore invalid PDFs.  */
@@ -279,7 +279,7 @@
 }
 
 /* Record in SAVED_INFO the information about the current character.  */
-static inline void
+static void
 bidi_remember_char (struct bidi_saved_info *saved_info,
                    struct bidi_it *bidi_it)
 {
@@ -295,7 +295,7 @@
 
 /* Copy the bidi iterator from FROM to TO.  To save cycles, this only
    copies the part of the level stack that is actually in use.  */
-static inline void
+static void
 bidi_copy_it (struct bidi_it *to, struct bidi_it *from)
 {
   /* Copy everything from the start through the active part of
@@ -340,7 +340,7 @@
    intact.  This is called when the cached information is no more
    useful for the current iteration, e.g. when we were reseated to a
    new position on the same object.  */
-static inline void
+static void
 bidi_cache_reset (void)
 {
   bidi_cache_idx = bidi_cache_start;
@@ -351,7 +351,7 @@
    iterator for reordering a buffer or a string that does not come
    from display properties, because that means all the previously
    cached info is of no further use.  */
-static inline void
+static void
 bidi_cache_shrink (void)
 {
   if (bidi_cache_size > BIDI_CACHE_CHUNK)
@@ -362,7 +362,7 @@
   bidi_cache_reset ();
 }
 
-static inline void
+static void
 bidi_cache_fetch_state (ptrdiff_t idx, struct bidi_it *bidi_it)
 {
   int current_scan_dir = bidi_it->scan_dir;
@@ -379,7 +379,7 @@
    level less or equal to LEVEL.  if LEVEL is -1, disregard the
    resolved levels in cached states.  DIR, if non-zero, means search
    in that direction from the last cache hit.  */
-static inline ptrdiff_t
+static ptrdiff_t
 bidi_cache_search (ptrdiff_t charpos, int level, int dir)
 {
   ptrdiff_t i, i_start;
@@ -484,7 +484,7 @@
   return -1;
 }
 
-static inline void
+static void
 bidi_cache_ensure_space (ptrdiff_t idx)
 {
   /* Enlarge the cache as needed.  */
@@ -506,7 +506,7 @@
     }
 }
 
-static inline void
+static void
 bidi_cache_iterator_state (struct bidi_it *bidi_it, bool resolved)
 {
   ptrdiff_t idx;
@@ -563,7 +563,7 @@
     bidi_cache_idx = idx + 1;
 }
 
-static inline bidi_type_t
+static bidi_type_t
 bidi_cache_find (ptrdiff_t charpos, int level, struct bidi_it *bidi_it)
 {
   ptrdiff_t i = bidi_cache_search (charpos, level, bidi_it->scan_dir);
@@ -583,7 +583,7 @@
   return UNKNOWN_BT;
 }
 
-static inline int
+static int
 bidi_peek_at_next_level (struct bidi_it *bidi_it)
 {
   if (bidi_cache_idx == bidi_cache_start || bidi_cache_last_idx == -1)
@@ -786,7 +786,7 @@
 
 /* Do whatever UAX#9 clause X8 says should be done at paragraph's
    end.  */
-static inline void
+static void
 bidi_set_paragraph_end (struct bidi_it *bidi_it)
 {
   bidi_it->invalid_levels = 0;
@@ -868,7 +868,7 @@
 /* Count bytes in string S between BEG/BEGBYTE and END.  BEG and END
    are zero-based character positions in S, BEGBYTE is byte position
    corresponding to BEG.  UNIBYTE means S is a unibyte string.  */
-static inline ptrdiff_t
+static ptrdiff_t
 bidi_count_bytes (const unsigned char *s, const ptrdiff_t beg,
                  const ptrdiff_t begbyte, const ptrdiff_t end, bool unibyte)
 {
@@ -896,7 +896,7 @@
    non-NULL, fetch the character from string S; otherwise fetch the
    character from the current buffer.  UNIBYTE means S is a
    unibyte string.  */
-static inline int
+static int
 bidi_char_at_pos (ptrdiff_t bytepos, const unsigned char *s, bool unibyte)
 {
   if (s)
@@ -924,7 +924,7 @@
    u+2029 to handle it as a paragraph separator.  STRING->s is the C
    string to iterate, or NULL if iterating over a buffer or a Lisp
    string; in the latter case, STRING->lstring is the Lisp string.  */
-static inline int
+static int
 bidi_fetch_char (ptrdiff_t bytepos, ptrdiff_t charpos, ptrdiff_t *disp_pos,
                 int *disp_prop, struct bidi_string_data *string,
                 bool frame_window_p, ptrdiff_t *ch_len, ptrdiff_t *nchars)
@@ -1314,7 +1314,7 @@
   The rest of this file constitutes the core of the UBA implementation.
  ***********************************************************************/
 
-static inline bool
+static bool
 bidi_explicit_dir_char (int ch)
 {
   bidi_type_t ch_type;
@@ -1837,7 +1837,7 @@
 
 /* Resolve the type of a neutral character according to the type of
    surrounding strong text and the current embedding level.  */
-static inline bidi_type_t
+static bidi_type_t
 bidi_resolve_neutral_1 (bidi_type_t prev_type, bidi_type_t next_type, int lev)
 {
   /* N1: European and Arabic numbers are treated as though they were R.  */