texinfo-commits
[Top][All Lists]
Advanced

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

branch master updated: Abort empty line for empty @def* command line


From: Patrice Dumas
Subject: branch master updated: Abort empty line for empty @def* command line
Date: Sun, 04 Sep 2022 18:26:09 -0400

This is an automated email from the git hooks/post-receive script.

pertusus pushed a commit to branch master
in repository texinfo.

The following commit(s) were added to refs/heads/master by this push:
     new 0232f99a25 Abort empty line for empty @def* command line
0232f99a25 is described below

commit 0232f99a251dac188371ca844510463f85c0eb54
Author: Patrice Dumas <pertusus@free.fr>
AuthorDate: Mon Sep 5 00:25:56 2022 +0200

    Abort empty line for empty @def* command line
    
    * tp/Texinfo/ParserNonXS.pm (_parse_def, _end_line),
    tp/Texinfo/XS/parsetexi/def.c (parse_def),
    tp/Texinfo/XS/parsetexi/end_line.c (end_line): call
    abort_empty_line before parsing definition command line to
    collect space after command in case there are no arguments at all.
    
    * tp/Texinfo/Convert/TexinfoXML.pm (_convert): remove end of line
    from @def* leading spaces.
---
 ChangeLog                                          |  13 ++
 tp/Texinfo/Convert/TexinfoXML.pm                   |  13 +-
 tp/Texinfo/ParserNonXS.pm                          |  16 +-
 tp/Texinfo/XS/parsetexi/def.c                      |   5 -
 tp/Texinfo/XS/parsetexi/end_line.c                 |   3 +
 tp/t/results/def/all_empty_def.pl                  | 162 +++++----------------
 tp/t/results/def/empty_def_arguments.pl            |  18 +--
 tp/t/results/def/no_category_or_no_name.pl         |  36 +----
 .../res_parser/formatting_xml/formatting.xml       |   8 +-
 9 files changed, 83 insertions(+), 191 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index d55d2d6468..bb3622f110 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2022-09-04  Patrice Dumas  <pertusus@free.fr>
+
+       Abort empty line for empty @def* command line
+
+       * tp/Texinfo/ParserNonXS.pm (_parse_def, _end_line),
+       tp/Texinfo/XS/parsetexi/def.c (parse_def),
+       tp/Texinfo/XS/parsetexi/end_line.c (end_line): call
+       abort_empty_line before parsing definition command line to
+       collect space after command in case there are no arguments at all.
+
+       * tp/Texinfo/Convert/TexinfoXML.pm (_convert): remove end of line
+       from @def* leading spaces.
+
 2022-09-04  Patrice Dumas  <pertusus@free.fr>
 
        * tp/Texinfo/ParserNonXS.pm (_end_line): no empty content in
diff --git a/tp/Texinfo/Convert/TexinfoXML.pm b/tp/Texinfo/Convert/TexinfoXML.pm
index 6c73089925..46cebdb841 100644
--- a/tp/Texinfo/Convert/TexinfoXML.pm
+++ b/tp/Texinfo/Convert/TexinfoXML.pm
@@ -1547,8 +1547,19 @@ sub _convert($$;$)
     }
     if ($element->{'type'} eq 'def_line') {
       if ($element->{'cmdname'}) {
+        my $leading_spaces_attribute_spec = [];
+        if ($element->{'extra'}
+            and $element->{'extra'}->{'spaces_before_argument'}
+            and $element->{'extra'}->{'spaces_before_argument'} ne '') {
+          my $leading_spaces = $element->{'extra'}->{'spaces_before_argument'};
+          # may happen without any argument, remove as a \n is added below
+          $leading_spaces =~ s/\n//;
+          $leading_spaces_attribute_spec = ['spaces', _protect_in_spaces(
+                                                          $leading_spaces)]
+            if ($leading_spaces ne '');
+        }
         $result .= $self->open_element($element->{'cmdname'},
-                            [_leading_spaces_before_argument($element)]);
+                                       $leading_spaces_attribute_spec);
       }
       $result .= $self->open_element('definitionterm');
       $result .= $self->_index_entry($element);
diff --git a/tp/Texinfo/ParserNonXS.pm b/tp/Texinfo/ParserNonXS.pm
index 074b9a2a88..f62126622d 100644
--- a/tp/Texinfo/ParserNonXS.pm
+++ b/tp/Texinfo/ParserNonXS.pm
@@ -2517,18 +2517,9 @@ sub _parse_def($$$)
   my ($self, $command, $current) = @_;
 
   my $contents = $current->{'contents'};
-  my $empty_spaces_after_command;
   
   my @new_contents;
   my @contents = @$contents;
-  if ($contents[0] and $contents[0]->{'type'}
-        and ($contents[0]->{'type'} eq 'empty_spaces_after_command'
-          or $contents[0]->{'type'} eq 'empty_line_after_command')) {
-    # FIXME this only happens if there is no argument at all, otherwise the
-    # types should already be put in extra by _abort_empty_line.  Would
-    # probably be better to call _abort_empty_line before.
-    $empty_spaces_after_command = shift @contents;
-  }
 
   if ($def_aliases{$command}) {
     my $real_command = $def_aliases{$command};
@@ -2650,8 +2641,6 @@ sub _parse_def($$$)
   if (scalar(@contents) > 0) {
     splice @new_contents, -scalar(@contents);
   }
-  unshift @new_contents, $empty_spaces_after_command
-    if $empty_spaces_after_command;
 
   @contents = map (_split_delimiters($self, $_), @contents );
   @new_contents = (@new_contents, @contents);
@@ -2971,7 +2960,8 @@ sub _end_line($$$)
             and $current->{'parent'}->{'type'} eq 'def_line') {
     my ($error) = $self->_pop_context(['ct_def'], $source_info, $current);
     die if ($error);
-    #_abort_empty_line($self, $current);
+    # in case there are no arguments at all, it needs to be called here.
+    _abort_empty_line($self, $current);
     my $def_command = $current->{'parent'}->{'extra'}->{'def_command'};
     my $arguments = _parse_def($self, $def_command, $current);
     if (scalar(@$arguments)) {
@@ -2985,7 +2975,7 @@ sub _end_line($$$)
         $def_parsed_hash->{$arg->[0]} = $arg->[1];
       }
       $current->{'parent'}->{'extra'}->{'def_parsed_hash'} = $def_parsed_hash;
-      # do an standard index entry tree
+      # do a standard index entry tree
       my $index_entry;
       if (defined($def_parsed_hash->{'name'})) {
         $index_entry = $def_parsed_hash->{'name'}
diff --git a/tp/Texinfo/XS/parsetexi/def.c b/tp/Texinfo/XS/parsetexi/def.c
index b2bddb205f..e205f30bf2 100644
--- a/tp/Texinfo/XS/parsetexi/def.c
+++ b/tp/Texinfo/XS/parsetexi/def.c
@@ -274,11 +274,6 @@ parse_def (enum command_id command, ELEMENT *current)
   ret = malloc (sizeof (DEF_INFO));
   memset (ret, 0, sizeof (DEF_INFO));
 
-  if (current->contents.number > 0
-      && (current->contents.list[0]->type == ET_empty_spaces_after_command
-          || current->contents.list[0]->type == ET_empty_line_after_command))
-    contents_idx++;
-
   split_def_args (current, contents_idx);
 
   /* Check for "def alias" - for example @defun for @deffn. */
diff --git a/tp/Texinfo/XS/parsetexi/end_line.c 
b/tp/Texinfo/XS/parsetexi/end_line.c
index 1b4578482b..b9d17e1f7d 100644
--- a/tp/Texinfo/XS/parsetexi/end_line.c
+++ b/tp/Texinfo/XS/parsetexi/end_line.c
@@ -2170,6 +2170,9 @@ end_line (ELEMENT *current)
           free (stripped);
         }
 
+      /* in case there are no arguments at all, it needs to be called here. */
+      abort_empty_line (&current, NULL);
+
       def_info = parse_def (def_command, current);
 
       /* Record the index entry if def_info is not empty. */
diff --git a/tp/t/results/def/all_empty_def.pl 
b/tp/t/results/def/all_empty_def.pl
index 71cccccf3b..8a57d31a74 100644
--- a/tp/t/results/def/all_empty_def.pl
+++ b/tp/t/results/def/all_empty_def.pl
@@ -15,17 +15,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -74,7 +64,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -95,17 +87,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -154,7 +136,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -175,17 +159,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -234,7 +208,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -255,17 +231,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -314,7 +280,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -335,17 +303,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -394,7 +352,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -415,17 +375,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -474,7 +424,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -495,17 +447,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -554,7 +496,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -575,17 +519,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -634,7 +568,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -655,17 +591,7 @@ $result_trees{'all_empty_def'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -714,7 +640,9 @@ $result_trees{'all_empty_def'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -730,8 +658,6 @@ $result_trees{'all_empty_def'} = {
   ],
   'type' => 'document_root'
 };
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'args'}[0];
@@ -740,8 +666,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'par
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[0]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[1]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[1]{'args'}[0];
@@ -750,8 +674,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[1]{'par
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[2]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[3]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[1]{'args'}[0];
@@ -760,8 +682,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[1]{'par
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[4]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[5]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[1]{'args'}[0];
@@ -770,8 +690,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[1]{'par
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[6]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[7]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[1]{'args'}[0];
@@ -780,8 +698,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[1]{'par
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[8]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[9]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[1]{'args'}[0];
@@ -790,8 +706,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[1]{'pa
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[10]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[11]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[1]{'args'}[0];
@@ -800,8 +714,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[1]{'pa
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[12]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[13]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[1]{'args'}[0];
@@ -810,8 +722,6 @@ 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[1]{'pa
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'extra'}{'end_command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'contents'}[1];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[14]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
 $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[15]{'parent'} = 
$result_trees{'all_empty_def'}{'contents'}[0];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16];
-$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[0]{'args'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[0]{'args'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[0];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[0]{'parent'}
 = $result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16];
 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[1]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'all_empty_def'}{'contents'}[0]{'contents'}[16]{'contents'}[1]{'args'}[0];
diff --git a/tp/t/results/def/empty_def_arguments.pl 
b/tp/t/results/def/empty_def_arguments.pl
index dfd811ed93..a27ea77628 100644
--- a/tp/t/results/def/empty_def_arguments.pl
+++ b/tp/t/results/def/empty_def_arguments.pl
@@ -220,17 +220,7 @@ $result_trees{'empty_def_arguments'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => ' 
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'line_arg'
                 }
@@ -238,7 +228,9 @@ $result_trees{'empty_def_arguments'} = {
               'cmdname' => 'deffnx',
               'extra' => {
                 'def_command' => 'deffn',
-                'original_def_cmdname' => 'deffnx'
+                'original_def_cmdname' => 'deffnx',
+                'spaces_before_argument' => ' 
+'
               },
               'parent' => {},
               'source_info' => {
@@ -680,8 +672,6 @@ 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[1
 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'args'}[0]{'parent'}
 = 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[1];
 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'extra'}{'def_parsed_hash'}{'category'}
 = 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'args'}[0]{'contents'}[0];
 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'parent'}
 = $result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0];
-$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[2]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[2];
-$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[2]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[2]{'args'}[0];
 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[2]{'args'}[0]{'parent'}
 = 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[2];
 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[2]{'parent'}
 = $result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0];
 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[3]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'empty_def_arguments'}{'contents'}[0]{'contents'}[0]{'contents'}[3]{'args'}[0];
diff --git a/tp/t/results/def/no_category_or_no_name.pl 
b/tp/t/results/def/no_category_or_no_name.pl
index d098caf21e..14df950c70 100644
--- a/tp/t/results/def/no_category_or_no_name.pl
+++ b/tp/t/results/def/no_category_or_no_name.pl
@@ -15,17 +15,7 @@ $result_trees{'no_category_or_no_name'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => '
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'block_line_arg'
                 }
@@ -91,7 +81,9 @@ $result_trees{'no_category_or_no_name'} = {
             }
           ],
           'extra' => {
-            'end_command' => {}
+            'end_command' => {},
+            'spaces_before_argument' => '
+'
           },
           'parent' => {},
           'source_info' => {
@@ -374,17 +366,7 @@ $result_trees{'no_category_or_no_name'} = {
             {
               'args' => [
                 {
-                  'contents' => [
-                    {
-                      'extra' => {
-                        'command' => {}
-                      },
-                      'parent' => {},
-                      'text' => ' 
-',
-                      'type' => 'empty_line_after_command'
-                    }
-                  ],
+                  'contents' => [],
                   'parent' => {},
                   'type' => 'line_arg'
                 }
@@ -392,7 +374,9 @@ $result_trees{'no_category_or_no_name'} = {
               'cmdname' => 'deffnx',
               'extra' => {
                 'def_command' => 'deffn',
-                'original_def_cmdname' => 'deffnx'
+                'original_def_cmdname' => 'deffnx',
+                'spaces_before_argument' => ' 
+'
               },
               'parent' => {},
               'source_info' => {
@@ -468,8 +452,6 @@ $result_trees{'no_category_or_no_name'} = {
   ],
   'type' => 'document_root'
 };
-$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = $result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0];
-$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'args'}[0]{'parent'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[0];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[0]{'parent'}
 = $result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'contents'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[0]{'contents'}[1]{'contents'}[0];
@@ -516,8 +498,6 @@ 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[1]{'args'}[0]{'parent'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[1];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[1]{'extra'}{'def_parsed_hash'}{'category'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[1]{'args'}[0]{'contents'}[0];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[1]{'parent'}
 = $result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4];
-$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[2]{'args'}[0]{'contents'}[0]{'extra'}{'command'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[2];
-$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[2]{'args'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[2]{'args'}[0];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[2]{'args'}[0]{'parent'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[2];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[2]{'parent'}
 = $result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4];
 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[3]{'contents'}[0]{'contents'}[0]{'parent'}
 = 
$result_trees{'no_category_or_no_name'}{'contents'}[0]{'contents'}[4]{'contents'}[3]{'contents'}[0];
diff --git a/tp/tests/layout/res_parser/formatting_xml/formatting.xml 
b/tp/tests/layout/res_parser/formatting_xml/formatting.xml
index 15417765c4..9df19e4beb 100644
--- a/tp/tests/layout/res_parser/formatting_xml/formatting.xml
+++ b/tp/tests/layout/res_parser/formatting_xml/formatting.xml
@@ -2296,7 +2296,7 @@ in verbatim ''
 <!-- c  Paragraph ended before @deffngeneral was complete. -->
 <deffn spaces=" " endspaces=" "><definitionterm><indexterm index="fn" 
number="52"><var>invalid</var></indexterm><defcategory>fsetinv</defcategory> 
<deffunction><var>invalid</var></deffunction> <defparam>a</defparam> 
<defparam>g</defparam></definitionterm>
 <deffnx spaces=" "><definitionterm><defcategory><b>id <samp>i</samp> 
ule</b></defcategory>    </definitionterm></deffnx>
-<deffnx><definitionterm></definitionterm></deffnx>
+<deffnx spaces=" "><definitionterm></definitionterm></deffnx>
 <deffnx spaces=" 
"><definitionterm><defcategory>aaa</defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on"></defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on">truc</defcategory></definitionterm></deffnx>
@@ -3435,7 +3435,7 @@ in verbatim ''
 <!-- c  Paragraph ended before @deffngeneral was complete. -->
 <deffn spaces=" " endspaces=" "><definitionterm><indexterm index="fn" 
number="104"><var>invalid</var></indexterm><defcategory>fsetinv</defcategory> 
<deffunction><var>invalid</var></deffunction> <defparam>a</defparam> 
<defparam>g</defparam></definitionterm>
 <deffnx spaces=" "><definitionterm><defcategory><b>id <samp>i</samp> 
ule</b></defcategory>    </definitionterm></deffnx>
-<deffnx><definitionterm></definitionterm></deffnx>
+<deffnx spaces=" "><definitionterm></definitionterm></deffnx>
 <deffnx spaces=" 
"><definitionterm><defcategory>aaa</defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on"></defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on">truc</defcategory></definitionterm></deffnx>
@@ -4601,7 +4601,7 @@ in verbatim ''
 <!-- c  Paragraph ended before @deffngeneral was complete. -->
 <deffn spaces=" " endspaces=" "><definitionterm><indexterm index="fn" 
number="156"><var>invalid</var></indexterm><defcategory>fsetinv</defcategory> 
<deffunction><var>invalid</var></deffunction> <defparam>a</defparam> 
<defparam>g</defparam></definitionterm>
 <deffnx spaces=" "><definitionterm><defcategory><b>id <samp>i</samp> 
ule</b></defcategory>    </definitionterm></deffnx>
-<deffnx><definitionterm></definitionterm></deffnx>
+<deffnx spaces=" "><definitionterm></definitionterm></deffnx>
 <deffnx spaces=" 
"><definitionterm><defcategory>aaa</defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on"></defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on">truc</defcategory></definitionterm></deffnx>
@@ -5735,7 +5735,7 @@ Various deff lines
 <!-- c  Paragraph ended before @deffngeneral was complete. -->
 </pre><deffn spaces=" " endspaces=" "><definitionterm><indexterm index="fn" 
number="208"><var>invalid</var></indexterm><defcategory>fsetinv</defcategory> 
<deffunction><var>invalid</var></deffunction> <defparam>a</defparam> 
<defparam>g</defparam></definitionterm>
 <deffnx spaces=" "><definitionterm><defcategory><b>id <samp>i</samp> 
ule</b></defcategory>    </definitionterm></deffnx>
-<deffnx><definitionterm></definitionterm></deffnx>
+<deffnx spaces=" "><definitionterm></definitionterm></deffnx>
 <deffnx spaces=" 
"><definitionterm><defcategory>aaa</defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on"></defcategory></definitionterm></deffnx>
 <deffnx><definitionterm><defcategory 
bracketed="on">truc</defcategory></definitionterm></deffnx>



reply via email to

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