texinfo-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[no subject]


From: Gavin D. Smith
Date: Tue, 15 Aug 2023 08:51:03 -0400 (EDT)

branch: master
commit c2ebd66bf5817efcdae63e2b5a60f90015b49cb1
Author: Gavin Smith <gavinsmith0123@gmail.com>
AuthorDate: Tue Aug 15 13:35:13 2023 +0100

    parsetexi compiler warnings
    
    * tp/Texinfo/XS/parsetexi/parser.c (process_remaining_on_line):
    Do not save separator characters into unused variables.
    * tp/Texinfo/XS/parsetexi/separator.c: Removed unused "funexit:" label.
---
 ChangeLog                           | 8 ++++++++
 tp/Texinfo/XS/parsetexi/parser.c    | 8 ++++----
 tp/Texinfo/XS/parsetexi/separator.c | 1 -
 3 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index bd59f769ff..18f8e6beff 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2023-08-15  Gavin Smith <gavinsmith0123@gmail.com>
+
+       parsetexi compiler warnings
+
+       * tp/Texinfo/XS/parsetexi/parser.c (process_remaining_on_line):
+       Do not save separator characters into unused variables.
+       * tp/Texinfo/XS/parsetexi/separator.c: Removed unused "funexit:" label.
+
 2023-08-15  Gavin Smith <gavinsmith0123@gmail.com>
 
        * README-hacking: add pod2texi to list of programs with
diff --git a/tp/Texinfo/XS/parsetexi/parser.c b/tp/Texinfo/XS/parsetexi/parser.c
index 0fd50709db..ed664f823c 100644
--- a/tp/Texinfo/XS/parsetexi/parser.c
+++ b/tp/Texinfo/XS/parsetexi/parser.c
@@ -2358,17 +2358,17 @@ process_remaining_on_line (ELEMENT **current_inout, 
char **line_inout)
   /* "Separator" characters */
   else if (*line == '{')
     {
-      char separator = *line++;
+      line++;
       current = handle_open_brace (current, &line);
     }
   else if (*line == '}')
     {
-      char separator = *line++;
+      line++;
       current = handle_close_brace (current, &line);
     }
   else if (*line == ',')
     {
-      char separator = *line++;
+      line++;
       /* comma as a command argument separator */
       if (counter_value (&count_remaining_args, current->parent) > 0)
         current = handle_comma (current, &line);
@@ -2388,7 +2388,7 @@ process_remaining_on_line (ELEMENT **current_inout, char 
**line_inout)
     }
   else if (*line == '\f')
     {
-      char separator = *line++;
+      line++;
       debug_nonl ("FORM FEED in "); debug_print_element (current, 1);
       debug_nonl (": "); debug_print_protected_string (line); debug ("");
       if (current->type == ET_paragraph)
diff --git a/tp/Texinfo/XS/parsetexi/separator.c 
b/tp/Texinfo/XS/parsetexi/separator.c
index 14df28bbc2..64b80bc0a2 100644
--- a/tp/Texinfo/XS/parsetexi/separator.c
+++ b/tp/Texinfo/XS/parsetexi/separator.c
@@ -563,7 +563,6 @@ handle_close_brace (ELEMENT *current, char **line_inout)
       line_error ("misplaced }");
     }
 
-funexit:
   *line_inout = line;
   return current;
 }



reply via email to

[Prev in Thread] Current Thread [Next in Thread]