Remove more deprecated stuff (#289)

Author: fingolfin

CHANGES.md

@@ -1,10 +1,15 @@
 This file describes changes in the AutoDoc package.
 
 2025.MM.DD
-  - Remove `scaffold.gapdoc_latex_options` -- it was deprecated and
-    replaced by `gapdoc.LaTeXOptions` in 2019
-  - Remove `maketest`, it was deprecated and superseded by
-    `extract_examples` in 2019
+  - Remove a bunch of features that were deprecated since 2019:
+    - `AutoDoc` option `scaffold.gapdoc_latex_options` has been
+      replaced by `gapdoc.LaTeXOptions`
+    - `AutoDoc` option `maketest` has been superseded by `extract_examples`
+    - Various AutoDoc commands were removed (see the manual for replacements)
+      - `@EndSection`, `@EndSubsection`
+      - `@AutoDoc`, `@BeginAutoDoc`, `@EndAutoDoc`
+      - `@System`, `@BeginSystem`, `@EndSystem`, `@InsertSystem`
+      - `@AutoDocPlainText`, `@BeginAutoDocPlainText`, `@EndAutoDocPlainText`
 
 2025.05.09
   - Add `InfoAutoDoc` info class for messages

doc/Comments.xml

@@ -742,8 +742,7 @@ can be important.</E>
 <Section Label="Deprecated">
 <Heading>Deprecated commands</Heading>
 
-The following commands used to be supported, but should not generally be used anymore.
-They will be removed in a future version of &AutoDoc;.
+The following commands used to be supported, but are not anymore.
 
 <List>
 <Mark><C>@EndSection</C></Mark>

gap/Parser.gi

@@ -522,13 +522,6 @@ InstallGlobalFunction( AutoDoc_Parser_ReadFiles,
             Reset();
             rest_of_file_skipped := true;
         end,
-        @BeginAutoDoc := deprecated("@BeginAutoDoc", function()
-            autodoc_read_line := fail;
-        end),
-        @AutoDoc := ~.@BeginAutoDoc,
-        @EndAutoDoc := deprecated("@EndAutoDoc", function()
-            autodoc_read_line := false;
-        end),
 
         @Chapter := function()
             local scope_chapter;
@@ -581,14 +574,6 @@ InstallGlobalFunction( AutoDoc_Parser_ReadFiles,
             scope_section := SectionInTree( tree, chapter_info[ 1 ], chapter_info[ 2 ] );
             scope_section!.title_string := current_command[ 2 ];
         end,
-        @EndSection := deprecated("@EndSection", function()
-            if not IsBound( chapter_info[ 2 ] ) then
-                ErrorWithPos( "found @EndSection with no active section" );
-            fi;
-            Unbind( chapter_info[ 2 ] );
-            Unbind( chapter_info[ 3 ] );
-            current_item := ChapterInTree( tree, chapter_info[ 1 ] );
-        end),
 
         @Subsection := function()
             local scope_subsection;
@@ -616,13 +601,6 @@ InstallGlobalFunction( AutoDoc_Parser_ReadFiles,
             scope_subsection := SubsectionInTree( tree, chapter_info[ 1 ], chapter_info[ 2 ], chapter_info[ 3 ] );
             scope_subsection!.title_string := current_command[ 2 ];
         end,
-        @EndSubsection := deprecated("@EndSubsection", function()
-            if not IsBound( chapter_info[ 3 ] ) then
-                ErrorWithPos( "found @EndSubsection with no active subsection" );
-            fi;
-            Unbind( chapter_info[ 3 ] );
-            current_item := SectionInTree( tree, chapter_info[ 1 ], chapter_info[ 2 ] );
-        end),
 
         @BeginGroup := function()
             local grp;
@@ -731,11 +709,6 @@ InstallGlobalFunction( AutoDoc_Parser_ReadFiles,
             fi;
         end,
 
-        @InsertSystem := deprecated("@InsertSystem", ~.@InsertChunk),
-        @System := deprecated("@System", ~.@BeginChunk),
-        @BeginSystem := ~.@System,
-        @EndSystem := deprecated("@EndSystem", ~.@EndChunk),
-
         @BeginCode := function()
             local label_name, tmp_system;
             label_name := ReplacedString( current_command[ 2 ], " ", "_" );
@@ -811,13 +784,6 @@ InstallGlobalFunction( AutoDoc_Parser_ReadFiles,
             NormalizeWhitespace( current_command[ 2 ] );
             Add( tree!.worksheet_dependencies, SplitString( current_command[ 2 ], " " ) );
         end,
-        @BeginAutoDocPlainText := deprecated("@BeginAutoDocPlainText", function()
-            plain_text_mode := true;
-        end),
-        @AutoDocPlainText := ~.@BeginAutoDocPlainText,
-        @EndAutoDocPlainText := deprecated("@EndAutoDocPlainText", function()
-            plain_text_mode := false;
-        end),
         @ExampleSession := function()
             local example_node;
             example_node := read_session_example( true, plain_text_mode );