From: Steve Hancock <perltidy@users.sourceforge.net>
Date: Tue, 4 May 2021 03:51:55 +0000 (-0700)
Subject: added some debug code for extended_continuation_indentation
X-Git-Tag: 20210402.01~54
X-Git-Url: https://git.donarmstrong.com/?a=commitdiff_plain;h=347aa1e639711e20a7a7aae1e0d99b8c02fc8b80;p=perltidy.git

added some debug code for extended_continuation_indentation
---

diff --git a/lib/Perl/Tidy/Formatter.pm b/lib/Perl/Tidy/Formatter.pm
index b888c0c5..a97f6b38 100644
--- a/lib/Perl/Tidy/Formatter.pm
+++ b/lib/Perl/Tidy/Formatter.pm
@@ -8740,6 +8740,8 @@ sub break_before_list_opening_containers {
     return;
 }
 
+use constant DEBUG_XCI => 0;
+
 sub extended_ci {
 
     # This routine implements the -xci (--extended-continuation-indentation)
@@ -8884,14 +8886,18 @@ sub extended_ci {
         # Require different input lines. This will filter out a large number
         # of small hash braces and array brackets.  If we accidentally filter
         # out an important container, it will get fixed on the next pass.
-        next
-          if (
+        if (
             $rLL->[$K_opening]->[_LINE_INDEX_] ==
             $rLL->[$K_closing]->[_LINE_INDEX_]
             && ( $rLL->[$K_closing]->[_CUMULATIVE_LENGTH_] -
                 $rLL->[$K_opening]->[_CUMULATIVE_LENGTH_] >
                 $rOpts_maximum_line_length )
-          );
+          )
+        {
+            DEBUG_XCI
+              && print "XCI: Skipping seqno=$seqno, require different lines\n";
+            next;
+        }
 
         # Do not apply -xci if adding extra ci will put the container contents
         # beyond the line length limit (fixes cases b899 b935)
@@ -8905,7 +8911,12 @@ sub extended_ci {
         my $space =
           $maximum_text_length - $len_tol - $rOpts_continuation_indentation;
 
-        next if ( $space < 0 );
+        if ( $space < 0 ) {
+            DEBUG_XCI && print "XCI: Skipping seqno=$seqno, space=$space\n";
+            next;
+        }
+        DEBUG_XCI && print "XCI: OK seqno=$seqno, space=$space\n";
+
         $available_space{$seqno} = $space;
 
         # This becomes the next controlling container