texinfo-commits
[Top][All Lists]
Advanced

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

branch master updated: * info/nodes.c (info_create_node): initialize nod


From: Patrice Dumas
Subject: branch master updated: * info/nodes.c (info_create_node): initialize nodelen field to 0. Previously it was set to -1, but it is confusing because the value was never used. In most cases nodelen is set right after the call to info_create_node. In case of man page, it may not be set right away, but then it is the contents field that is used to determine if the man page node was setup.
Date: Sat, 12 Oct 2024 03:50:24 -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 6e59f996b3 * info/nodes.c (info_create_node): initialize nodelen field 
to 0. Previously it was set to -1, but it is confusing because the value was 
never used.  In most cases nodelen is set right after the call to 
info_create_node.  In case of man page, it may not be set right away, but then 
it is the contents field that is used to determine if the man page node was 
setup.
6e59f996b3 is described below

commit 6e59f996b32f59a310819c8f7cf149a9315965a3
Author: Patrice Dumas <pertusus@free.fr>
AuthorDate: Sat Oct 12 09:50:12 2024 +0200

    * info/nodes.c (info_create_node): initialize nodelen field to 0.
    Previously it was set to -1, but it is confusing because the value was
    never used.  In most cases nodelen is set right after the call to
    info_create_node.  In case of man page, it may not be set right away,
    but then it is the contents field that is used to determine if the man
    page node was setup.
---
 ChangeLog    | 9 +++++++++
 info/nodes.c | 2 +-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index a32f22a896..3548d1145c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2024-10-12  Patrice Dumas  <pertusus@free.fr>
+
+       * info/nodes.c (info_create_node): initialize nodelen field to 0.
+       Previously it was set to -1, but it is confusing because the value was
+       never used.  In most cases nodelen is set right after the call to
+       info_create_node.  In case of man page, it may not be set right away,
+       but then it is the contents field that is used to determine if the man
+       page node was setup.
+
 2024-10-12  Patrice Dumas  <pertusus@free.fr>
 
        * info/dir.c, info/display.c, info/scan.c: reindent, replace tabs by
diff --git a/info/nodes.c b/info/nodes.c
index 28d0e477f0..8e823d7f1b 100644
--- a/info/nodes.c
+++ b/info/nodes.c
@@ -908,7 +908,7 @@ info_create_node (void)
   n->subfile = 0;
   n->nodename = 0;
   n->contents = 0;
-  n->nodelen = -1;
+  n->nodelen = 0;
   n->display_pos = 0;
   n->body_start = 0;
   n->flags = 0;



reply via email to

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