branch master updated: * tp/Texinfo/XS/main/parser_conf.c: add a NOTE in comment explaining why the registered confs are never unallocated.

[Patrice Dumas]

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 2adeb6bbf9 * tp/Texinfo/XS/main/parser_conf.c: add a NOTE in comment 
explaining why the registered confs are never unallocated.
2adeb6bbf9 is described below

commit 2adeb6bbf9857930a0931e87e9631787ecf1a6ed
Author: Patrice Dumas <pertusus@free.fr>
AuthorDate: Fri May 24 20:06:03 2024 +0200

    * tp/Texinfo/XS/main/parser_conf.c: add a NOTE in comment explaining
    why the registered confs are never unallocated.
---
 ChangeLog                        |  5 +++++
 tp/Texinfo/XS/main/parser_conf.c | 14 +++++++++++++-
 2 files changed, 18 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 5fdc5aec0d..19a0ea2ea4 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2024-05-24  Patrice Dumas  <pertusus@free.fr>
+
+       * tp/Texinfo/XS/main/parser_conf.c: add a NOTE in comment explaining
+       why the registered confs are never unallocated.
+
 2024-05-24  Patrice Dumas  <pertusus@free.fr>
 
        * tp/t/no_structure_test.t: uncomment the call to
diff --git a/tp/Texinfo/XS/main/parser_conf.c b/tp/Texinfo/XS/main/parser_conf.c
index f0b8e7ddb4..662622d2fa 100644
--- a/tp/Texinfo/XS/main/parser_conf.c
+++ b/tp/Texinfo/XS/main/parser_conf.c
@@ -23,7 +23,9 @@
 #include "utils.h"
 #include "parser_conf.h"
 
-/* Configuration values. */
+/* Configuration values.  This global variables is used in the parser
+   code.  The registered parser configurations is copied over it if
+   a parser reused configuration is needed. */
 PARSER_CONF global_parser_conf;
 
 /* registered parser configurations */
@@ -31,6 +33,16 @@ static PARSER_CONF **parser_conf_list;
 static size_t parser_conf_number;
 static size_t parser_conf_space;
 
+/* NOTE the registered parser configurations are never unallocated.
+   The conf memory use is not important.  Also, there is no parser
+   configuration allocated thought gdt (with accept_internalvalue
+   set being used as an indication that the parser is setup from gdt,
+   as seen in Perl code).  Therefore, there is only one conf allocated
+   per converted file in the main program, which means a very low need
+   of memory for the confs.  In the *.t tests there is one registered
+   conf per test, but it is not an issue as it is for tests. */
+
+
 PARSER_CONF *
 retrieve_parser_conf (int parser_conf_descriptor)
 {