Skip Menu |
 

This queue is for tickets about the Perl-Tidy CPAN distribution.

Report information
The Basics
Id: 124114
Status: resolved
Priority: 0/
Queue: Perl-Tidy

People
Owner: Nobody in particular
Requestors: dugrim [...] gmx.de
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Fat comma alignment does not work in all cases
Date: Wed, 17 Jan 2018 15:33:21 +0100
To: bug-Perl-Tidy [...] rt.cpan.org
From: dugrim <dugrim [...] gmx.de>
Download (untitled) / with headers
text/plain 267b
Example: #!/usr/bin/perl my %h = { a => 2 > 3 ? 1 : 0, bbbb => sub { my $y = "1" }, c => sub { my $z = "2" }, d => 2 > 3 ? 1 : 0, }; Using perl-tidy with default options I would expect all fat commas to be aligned perl 5.16.3 perl-tidy 20180101
Subject: Re: [rt.cpan.org #124114] Fat comma alignment does not work in all cases
Date: Wed, 17 Jan 2018 10:09:31 -0800
To: bug-Perl-Tidy [...] rt.cpan.org
From: Steven Hancock <s7078hancock [...] gmail.com>
Download (untitled) / with headers
text/plain 889b
Thanks, It breaks vertical alignment when the right hand sides are complex and different. I'll look at it. Steve On Wed, Jan 17, 2018 at 6:33 AM, dugrim via RT <bug-Perl-Tidy@rt.cpan.org> wrote: Show quoted text
> Wed Jan 17 09:33:45 2018: Request 124114 was acted upon. > Transaction: Ticket created by dugrim@gmx.de > Queue: Perl-Tidy > Subject: Fat comma alignment does not work in all cases > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: dugrim@gmx.de > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=124114 > > > > Example: > > #!/usr/bin/perl > my %h = { > a => 2 > 3 ? 1 : 0, > bbbb => sub { my $y = "1" }, > c => sub { my $z = "2" }, > d => 2 > 3 ? 1 : 0, > }; > > Using perl-tidy with default options I would expect all fat commas to be > aligned > > perl 5.16.3 > perl-tidy 20180101 > >
This is fixed in version 20180219


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.