[8414] remove %no_command_commands

[gavinsmith0123]

Revision: 8414
          http://svn.sv.gnu.org/viewvc/?view=rev&root=texinfo&revision=8414
Author:   gavin
Date:     2018-10-26 15:48:33 -0400 (Fri, 26 Oct 2018)
Log Message:
-----------
remove %no_command_commands

Modified Paths:
--------------
    trunk/ChangeLog
    trunk/tp/Texinfo/Parser.pm
    trunk/tp/t/results/coverage/test_errormsg.pl
    trunk/tp/t/results/indices/explicit_sort_key.pl

Modified: trunk/ChangeLog
===================================================================
--- trunk/ChangeLog     2018-10-25 23:26:54 UTC (rev 8413)
+++ trunk/ChangeLog     2018-10-26 19:48:33 UTC (rev 8414)
@@ -1,5 +1,13 @@
 2018-10-25  Gavin Smith  <address@hidden>
 
+       * tp/Texinfo/Parser.pm (%no_command_commands): Remove this list 
+       of commands that shouldn't contain any other commands.  Although 
+       it is valid, it makes the picture of which commands can contain 
+       others more complicated for the sake of only a few commands.
+       Introduced on 2016-07-09 for @errormsg and @U.
+
+2018-10-25  Gavin Smith  <address@hidden>
+
        * tp/Texinfo/Common.pm (%brace_commands): Change meaning of 1
        as a value.  Now only commands that were in
        %simple_text_commands in Parser.pm have 1.  Use words as well as 

Modified: trunk/tp/Texinfo/Parser.pm
===================================================================
--- trunk/tp/Texinfo/Parser.pm  2018-10-25 23:26:54 UTC (rev 8413)
+++ trunk/tp/Texinfo/Parser.pm  2018-10-26 19:48:33 UTC (rev 8414)
@@ -442,12 +442,6 @@
   $simple_text_commands{$command} = 1;
 }
 
-# Commands that don't contain other @-commands.
-my %no_command_commands;
-foreach my $command ('errormsg', 'U', 'sortas') {
-  $no_command_commands{$command} = 1;
-}
-
 # commands that accept full text, but no block or top-level commands
 my %full_text_commands;
 foreach my $brace_command (keys (%brace_commands)) {  
@@ -481,9 +475,6 @@
 foreach my $command (keys(%simple_text_commands)) {
   $default_valid_nestings{$command} = \%in_simple_text_commands;
 }
-foreach my $command (keys(%no_command_commands)) {
-  $default_valid_nestings{$command} = {};
-}
 foreach my $command (keys(%full_line_commands)) {
   $default_valid_nestings{$command} = \%in_full_line_commands;
 }

Modified: trunk/tp/t/results/coverage/test_errormsg.pl
===================================================================
--- trunk/tp/t/results/coverage/test_errormsg.pl        2018-10-25 23:26:54 UTC 
(rev 8413)
+++ trunk/tp/t/results/coverage/test_errormsg.pl        2018-10-26 19:48:33 UTC 
(rev 8414)
@@ -152,24 +152,6 @@
 
 $result_errors{'test_errormsg'} = [
   {
-    'error_line' => ':2: warning: @~ should not appear in @errormsg
-',
-    'file_name' => '',
-    'line_nr' => 2,
-    'macro' => '',
-    'text' => '@~ should not appear in @errormsg',
-    'type' => 'warning'
-  },
-  {
-    'error_line' => ':2: warning: @code should not appear in @errormsg
-',
-    'file_name' => '',
-    'line_nr' => 2,
-    'macro' => '',
-    'text' => '@code should not appear in @errormsg',
-    'type' => 'warning'
-  },
-  {
     'error_line' => ':2: Text 
 ',
     'file_name' => '',

Modified: trunk/tp/t/results/indices/explicit_sort_key.pl
===================================================================
(Binary files differ)