]> git.donarmstrong.com Git - perltidy.git/commitdiff
update version to 20201207 20201207
authorSteve Hancock <perltidy@users.sourceforge.net>
Sun, 6 Dec 2020 19:26:52 +0000 (11:26 -0800)
committerSteve Hancock <perltidy@users.sourceforge.net>
Sun, 6 Dec 2020 19:26:52 +0000 (11:26 -0800)
25 files changed:
CHANGES.md
bin/perltidy
docs/BugLog.html
docs/ChangeLog.html
docs/Tidy.html
docs/perltidy.html
lib/Perl/Tidy.pm
lib/Perl/Tidy.pod
lib/Perl/Tidy/Debugger.pm
lib/Perl/Tidy/DevNull.pm
lib/Perl/Tidy/Diagnostics.pm
lib/Perl/Tidy/FileWriter.pm
lib/Perl/Tidy/Formatter.pm
lib/Perl/Tidy/HtmlWriter.pm
lib/Perl/Tidy/IOScalar.pm
lib/Perl/Tidy/IOScalarArray.pm
lib/Perl/Tidy/IndentationItem.pm
lib/Perl/Tidy/LineBuffer.pm
lib/Perl/Tidy/LineSink.pm
lib/Perl/Tidy/LineSource.pm
lib/Perl/Tidy/Logger.pm
lib/Perl/Tidy/Tokenizer.pm
lib/Perl/Tidy/VerticalAligner.pm
lib/Perl/Tidy/VerticalAligner/Alignment.pm
lib/Perl/Tidy/VerticalAligner/Line.pm

index 26f382a679fe82c449df040fcde1e1d47b41497f..6c2ddc3200e5d316287a1a4db4db86de7191676c 100644 (file)
@@ -1,6 +1,25 @@
 # Perltidy Change Log
 
-## 2020 12 02
+## 2020 12 07
+
+    - Fixed issue git #47, incorrect welding of anonymous subs. 
+      An incorrect weld format was being made when the --weld-nested-containers option
+      (-wn) was used in to format a function which returns a list of anonymous subs.
+      For example, the following snippet was incorrectly being welded.
+
+    $promises[$i]->then(
+        sub { $all->resolve(@_); () },
+        sub {
+            $results->[$i] = [@_];
+            $all->reject(@$results) if --$remaining <= 0;
+            return ();
+        }
+    );
+
+    This was due to an error introduced in v20201201 related to parsing sub
+    signatures.  Reformatting with the current version will fix the problem.
+
+## 2020 12 01
 
     - This release is being made primarily to make available a several new formatting 
       parameters, in particular -xci, -kbb=s, -kba=s, and -wnxl=s. No significant 
index ad75b5b3ca253c244088bd8d9d3ccdb262282c17..57113b91b4e770cc9820913079270a65bd1e37e3 100755 (executable)
@@ -4560,7 +4560,7 @@ The perltidy binary uses the Perl::Tidy module and is installed when that module
 
 =head1 VERSION
 
-This man page documents perltidy version 20201202
+This man page documents perltidy version 20201207
 
 =head1 BUG REPORTS
 
index 401ad56cbb01ff42fe41bead717b3c4744f7baa9..cb5402f4a1148780f748c991b038b041bbe8f135 100644 (file)
 
 
 <ul id="index">
+  <li><a href="#Issues-fixed-after-release-20201202">Issues fixed after release 20201202</a></li>
   <li><a href="#Issues-fixed-after-release-20201001">Issues fixed after release 20201001</a></li>
   <li><a href="#Issues-fixed-after-release-20200907">Issues fixed after release 20200907</a></li>
   <li><a href="#Open-Issues">Open Issues</a></li>
 </ul>
 
+<h1 id="Issues-fixed-after-release-20201202">Issues fixed after release 20201202</h1>
+
+<dl>
+
+<dt id="Fix-for-issue-git-47"><b>Fix for issue git #47</b></dt>
+<dd>
+
+<p>This issue has to do with the --weld-nested-containers option in the specific case of formatting a function which returns a list of anonymous subs. For example</p>
+
+<pre><code>    $promises[$i]-&gt;then(
+        sub { $all-&gt;resolve(@_); () },
+        sub {
+            $results-&gt;[$i] = [@_];
+            $all-&gt;reject(@$results) if --$remaining &lt;= 0;
+            return ();
+        }
+    );</code></pre>
+
+<p>A bug introduced in v20201202 caused an incorrect welding to occur when the -wn flag was set</p>
+
+<pre><code>    $promises[$i]-&gt;then( sub { $all-&gt;resolve(@_); () },
+        sub {
+        $results-&gt;[$i] = [@_];
+        $all-&gt;reject(@$results) if --$remaining &lt;= 0;
+        return ();
+        } );</code></pre>
+
+<p>This bug has been fixed, and code which has been incorrectly formatted will be correctly formatted with the next release. The bug was a result of a new coding introduced in v20201202 for fixing some issues with parsing sub signatures. Previously they were sometimes parsed the same as prototypes and sometimes as lists, now they are always parsed as lists.</p>
+
+</dd>
+</dl>
+
 <h1 id="Issues-fixed-after-release-20201001">Issues fixed after release 20201001</h1>
 
 <dl>
index f59053974d7e6af6792f896650a5df4f96edf325..6937e365064845af3c92aa5375a7efad8c637a3c 100644 (file)
@@ -1,6 +1,26 @@
 <h1>Perltidy Change Log</h1>
 
-<h2>2020 12 02</h2>
+<h2>2020 12 07</h2>
+
+<pre><code>- Fixed issue git #47, incorrect welding of anonymous subs. 
+  An incorrect weld format was being made when the --weld-nested-containers option
+  (-wn) was used in to format a function which returns a list of anonymous subs.
+  For example, the following snippet was incorrectly being welded.
+
+$promises[$i]-&gt;then(
+    sub { $all-&gt;resolve(@_); () },
+    sub {
+        $results-&gt;[$i] = [@_];
+        $all-&gt;reject(@$results) if --$remaining &lt;= 0;
+        return ();
+    }
+);
+
+This was due to an error introduced in v20201201 related to parsing sub
+signatures.  Reformatting with the current version will fix the problem.
+</code></pre>
+
+<h2>2020 12 01</h2>
 
 <pre><code>- This release is being made primarily to make available a several new formatting 
   parameters, in particular -xci, -kbb=s, -kba=s, and -wnxl=s. No significant 
index 5b0221b236d9ccd2c0c60e054e2b825f79151f14..be01abdd12844841f51a848536f191e1cbb1deb4 100644 (file)
 
 <h1 id="VERSION">VERSION</h1>
 
-<p>This man page documents Perl::Tidy version 20201202</p>
+<p>This man page documents Perl::Tidy version 20201207</p>
 
 <h1 id="LICENSE">LICENSE</h1>
 
index 1a5be9e175681216442507a4b763566af6983c2c..3914d9cc8631840ce50c20104a3d08f9d3d7b746 100644 (file)
 
 <h1 id="VERSION">VERSION</h1>
 
-<p>This man page documents perltidy version 20201202</p>
+<p>This man page documents perltidy version 20201207</p>
 
 <h1 id="BUG-REPORTS">BUG REPORTS</h1>
 
index 5160a4b3038abda0ec4a155cd5df6e3a8d737fe1..e16144aa0501f55fba38c3431c8216e9e7b5fbeb 100644 (file)
@@ -110,7 +110,7 @@ BEGIN {
     # Release version must be bumped, and it is probably past time for a
     # release anyway.
 
-    $VERSION = '20201202';
+    $VERSION = '20201207';
 }
 
 sub DESTROY {
index 9ff8b49eae0c8e095e6d0d6e375289444c5e881d..2895c5e44164243cb26ebbaa0bc7f9d0e58ad63a 100644 (file)
@@ -432,7 +432,7 @@ The module 'Perl::Tidy' comes with a binary 'perltidy' which is installed when t
 
 =head1 VERSION
 
-This man page documents Perl::Tidy version 20201202
+This man page documents Perl::Tidy version 20201207
 
 =head1 LICENSE
 
index def7d724f9a32440742a73b94152ea88f2ec88e2..28a3b33aa3e077d1c9f4842093fdcfb7a1ed57f5 100644 (file)
@@ -7,7 +7,7 @@
 package Perl::Tidy::Debugger;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub new {
 
index 9ad7ae2204c2bda4bd27390aeb6d486e42334554..60449b587e69d9a80cae4c19f70542ce85b879ab 100644 (file)
@@ -7,7 +7,7 @@
 package Perl::Tidy::DevNull;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 sub new   { my $self = shift; return bless {}, $self }
 sub print { return }
 sub close { return }
index cf83d2747dacbe84517758be1a076e2adcafa56b..ceadacbc9111cae33bf7fac8f37a29eaf5cf38c4 100644 (file)
@@ -20,7 +20,7 @@
 package Perl::Tidy::Diagnostics;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub AUTOLOAD {
 
index 63b7fde0faa14f199dbf4869e20a970a12568bbd..47ddc8aafc39007dfa0c48025d180d2db663a333 100644 (file)
@@ -7,7 +7,7 @@
 package Perl::Tidy::FileWriter;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 use constant DEVEL_MODE => 0;
 
index 8160adf5747ef6daf2344fb242ba6dd8cb57d623..a76085955fe95da5443383afc280c4c7ad222665 100644 (file)
@@ -49,7 +49,7 @@ use constant DEVEL_MODE => 0;
 { #<<< A non-indenting brace to contain all lexical variables
 
 use Carp;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 # The Tokenizer will be loaded with the Formatter
 ##use Perl::Tidy::Tokenizer;    # for is_keyword()
index e8413511fc0d42e976e77308401ddc8efefb0d8b..9c85cc1591b6006988207fe8503c441ccdeeee86 100644 (file)
@@ -7,7 +7,7 @@
 package Perl::Tidy::HtmlWriter;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 use File::Basename;
 
index 92665ccaf6603592aa8ca8bc9912273a4060c318..68972bb185c329e3fa1a130f6788564cc1113da5 100644 (file)
@@ -10,7 +10,7 @@ package Perl::Tidy::IOScalar;
 use strict;
 use warnings;
 use Carp;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub AUTOLOAD {
 
index fb782fd344559d6d7a9b06b8992c19d4e46b4c00..bd71fbe687cb164b3e752333e9196deeb81864d8 100644 (file)
@@ -14,7 +14,7 @@ package Perl::Tidy::IOScalarArray;
 use strict;
 use warnings;
 use Carp;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub AUTOLOAD {
 
index 5ded75a1f18bac5e5d1b2636fd0b3f710ffd6978..5453c5839d10c0a9a23835da98dd611e5e48f3bd 100644 (file)
@@ -8,7 +8,7 @@
 package Perl::Tidy::IndentationItem;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 BEGIN {
 
index 02a85f06af9a54035dc23204cff9fc0ffd2b3ffe..f4f15f0c8235898b6758e231848fecf950c21edd 100644 (file)
@@ -12,7 +12,7 @@
 package Perl::Tidy::LineBuffer;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub AUTOLOAD {
 
index 084e145fa9a4def10779af2c5775fe29e93733a9..d25080a3f5c661e7befc3461a456693740ecd226 100644 (file)
@@ -8,7 +8,7 @@
 package Perl::Tidy::LineSink;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub AUTOLOAD {
 
index 8c3e0c39321cb42d2d10b9ca4d3a0c64fca5ee03..7c04c6088b0eca10cd75feb9fe9c81b0e818ba17 100644 (file)
@@ -8,7 +8,7 @@
 package Perl::Tidy::LineSource;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub AUTOLOAD {
 
index d48a973fd667fbe85d7c6e6ba43037ff97c4541a..d6ef73695975fac897f068757bc7a6d9ef8599ac 100644 (file)
@@ -7,7 +7,7 @@
 package Perl::Tidy::Logger;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 sub AUTOLOAD {
 
index 691af758be9b190f7ea6833cabc2060e72c89500..932c88163d853791f91b0defec6db6aa4719fb70 100644 (file)
@@ -21,7 +21,7 @@
 package Perl::Tidy::Tokenizer;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 use Perl::Tidy::LineBuffer;
 use Carp;
index e3219c3132577d21d8777166702f56ac954098e8..70838489a062d0ae246b4a5907e7799426e275c1 100644 (file)
@@ -1,7 +1,7 @@
 package Perl::Tidy::VerticalAligner;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 use Perl::Tidy::VerticalAligner::Alignment;
 use Perl::Tidy::VerticalAligner::Line;
@@ -3008,6 +3008,7 @@ sub delete_null_alignments {
         }    # End loopover lines
         $end_match->();
     }    # End loop over subgroups
+    return;
 } ## end sub delete_null_alignments
 
 sub fat_comma_to_comma {
index e6b0ecabdbded58454a44dd9915e44bad44af2b3..e5a33df1f79b7ce79825c27e8060ea53eec00090 100644 (file)
@@ -10,7 +10,7 @@ use warnings;
 
 { #<<< A non-indenting brace
 
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 #    _column_          # the current column number
 #    _saved_column_    # a place for temporary storage
index a747adb956321ba88f1967e5e0a8dfed1be13328..b7ddcfe9ab328d7928c19c19cc88391c5aad262f 100644 (file)
@@ -8,7 +8,7 @@
 package Perl::Tidy::VerticalAligner::Line;
 use strict;
 use warnings;
-our $VERSION = '20201202';
+our $VERSION = '20201207';
 
 BEGIN {
     my $i = 0;