[no subject]

[Patrice Dumas]

branch: master
commit 838f61425f70709870d62d7883032b2345d28e24
Author: Patrice Dumas <pertusus@free.fr>
AuthorDate: Mon Jun 17 14:47:00 2024 +0200

    * tp/Texinfo/XS/parsetexi/parser.c (merge_text): modify code to have
    the new_text label before a statement and not a declaration.
    
    * tp/Texinfo/XS/parsetexi/parser.c (process_remaining_on_line):
    comment out unused value_valid label.
---
 ChangeLog                        | 14 +++++++++++---
 tp/TODO                          |  6 +-----
 tp/Texinfo/XS/parsetexi/parser.c |  5 +++--
 3 files changed, 15 insertions(+), 10 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 2060e0f552..4ae34d64b4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,7 +1,10 @@
-2024-09-29  Patrice Dumas  <pertusus@free.fr>
+2024-06-17  Patrice Dumas  <pertusus@free.fr>
 
-       * tp/Texinfo/XS/convert/convert_html.c (html_command_description):
-       update for move of cmd to CONTAINER.
+       * tp/Texinfo/XS/parsetexi/parser.c (merge_text): modify code to have
+       the new_text label before a statement and not a declaration.
+
+       * tp/Texinfo/XS/parsetexi/parser.c (process_remaining_on_line):
+       comment out unused value_valid label.
 
 2024-06-16  Patrice Dumas  <pertusus@free.fr>
 
@@ -90,6 +93,11 @@
        (protect_first_parenthesis_in_targets_internal): check that the
        element is an @-command element before calling get_label_element.
 
+2024-09-29  Patrice Dumas  <pertusus@free.fr>
+
+       * tp/Texinfo/XS/convert/convert_html.c (html_command_description):
+       update for move of cmd to CONTAINER.
+
 2024-06-16  Patrice Dumas  <pertusus@free.fr>
 
        Turn source_mark_list to a pointer
diff --git a/tp/TODO b/tp/TODO
index 9bd3abb1fd..407b4c43f2 100644
--- a/tp/TODO
+++ b/tp/TODO
@@ -60,11 +60,7 @@ More generally, the plaintext ref_commands formatting code
 could be completly different from the Info code, which is the current
 code, as most of the code deals with specific contraints of Info.
 
-
-Some ideas for the reduction of the XS/C ELEMENT memory footprint, from
-Gavin and Patrice
-
-1) make building "source marks" optional.
+Make building "source marks" optional?
 
 valgrinf massif useful-heap approximate distribution in 2024 (obsolete)
 valgrind --tool=massif --massif-out-file=massif_info.out perl -w texi2any.pl 
../doc/texinfo.texi
diff --git a/tp/Texinfo/XS/parsetexi/parser.c b/tp/Texinfo/XS/parsetexi/parser.c
index 5f39a1b500..97bef6686d 100644
--- a/tp/Texinfo/XS/parsetexi/parser.c
+++ b/tp/Texinfo/XS/parsetexi/parser.c
@@ -808,8 +808,9 @@ merge_text (ELEMENT *current, const char *text, size_t 
len_text,
     }
   else
     {
+      ELEMENT *e;
      new_text:
-      ELEMENT *e = new_text_element (ET_normal_text);
+      e = new_text_element (ET_normal_text);
       if (transfer_marks_element)
         transfer_source_marks (transfer_marks_element, e);
       text_append_n (e->e.text, text, len_text);
@@ -2192,7 +2193,7 @@ process_remaining_on_line (ELEMENT **current_inout, const 
char **line_inout)
 
           if (1) /* @value syntax is valid */
             {
-          value_valid:
+          /* value_valid: */
               if (cmd == CM_value)
                 {
                   char *value;