Re: Duplicate variable use in Texinfo/ManipulateTree.pm

2025-06-05 Thread Patrice Dumas
On Thu, Jun 05, 2025 at 05:03:38PM +0100, Gavin Smith wrote: > FAIL: t/converters_tests.t 1527 - > indices_in_begin_tables_lists_entries_after_item tree This started to fail after commit 359b73825d2b68f3fd76d07a54c231d77464a3df Store Texinfo tree element SV in C I realized that in the new code o

Re: Duplicate variable use in Texinfo/ManipulateTree.pm

2025-06-05 Thread Gavin Smith
On Thu, Jun 05, 2025 at 01:07:51AM +0200, Patrice Dumas wrote: > On Wed, Jun 04, 2025 at 09:37:20PM +0100, Gavin Smith wrote: > > When I run the texi2any test suite with TEXINFO_XS=omit, there is > > a warning printed: > > > > "my" variable $current_nr masks earlier declaration in same scope at >

Re: Duplicate variable use in Texinfo/ManipulateTree.pm

2025-06-04 Thread Patrice Dumas
On Wed, Jun 04, 2025 at 09:37:20PM +0100, Gavin Smith wrote: > When I run the texi2any test suite with TEXINFO_XS=omit, there is > a warning printed: > > "my" variable $current_nr masks earlier declaration in same scope at > t//../../perl/Texinfo/ManipulateTree.pm line 839. > > This is presumabl

Duplicate variable use in Texinfo/ManipulateTree.pm

2025-06-04 Thread Gavin Smith
When I run the texi2any test suite with TEXINFO_XS=omit, there is a warning printed: "my" variable $current_nr masks earlier declaration in same scope at t//../../perl/Texinfo/ManipulateTree.pm line 839. This is presumably a bug. Here is the relevant part of the code: sub print_element_detail