Skip Menu |

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

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

People
Owner: Nobody in particular
Requestors: felipe [...] felipegasper.com
Cc:
AdminCc:

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



Subject: break-at-old-comma-breakpoints still mangles “map”
Date: Mon, 15 Sep 2014 11:19:57 -0500
To: bug-Perl-Tidy [...] rt.cpan.org
From: Felipe Gasper <felipe [...] felipegasper.com>
Hello, ============================ ##### - BEFORE: #tidyoff my %foo = ( alpha => 1, beta => 2, gamma => 3, ); my @bar = map { { number => $_, character => chr $_, padding => (' ' x $_), } } ( 0 .. 32 ); #tidyon ##### - AFTER: my %foo = ( alpha => 1, beta => 2, gamma => 3, ); my @bar = map { { number => $_, character => chr $_, padding => ( ' ' x $_ ), } } ( 0 .. 32 ); ============================ Should “break-at-old-comma-breakpoints” not have left that “map” as it was, without combining everything onto a single line? Thank you! -Felipe Gasper Houston, TX
Subject: Re: [rt.cpan.org #98902] break-at-old-comma-breakpoints still mangles “map”
Date: Sat, 20 Sep 2014 06:55:30 -0700
To: "bug-Perl-Tidy [...] rt.cpan.org" <bug-Perl-Tidy [...] rt.cpan.org>
From: Steven Hancock <s7078hancock [...] gmail.com>
This looks like a conflict with a rule that prefers one-line map blocks. I'll fix it in the next release. Thanks for the report. Steve On Mon, Sep 15, 2014 at 9:20 AM, felipe@felipegasper.com via RT < bug-Perl-Tidy@rt.cpan.org> wrote: Show quoted text
> Mon Sep 15 12:20:11 2014: Request 98902 was acted upon. > Transaction: Ticket created by felipe@felipegasper.com > Queue: Perl-Tidy > Subject: break-at-old-comma-breakpoints still mangles “map” > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: felipe@felipegasper.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=98902 > > > > Hello, > > ============================ > ##### - BEFORE: > > #tidyoff > my %foo = ( > alpha => 1, > beta => 2, > gamma => 3, > ); > > my @bar = map { { > number => $_, > character => chr $_, > padding => (' ' x $_), > } } ( 0 .. 32 ); > #tidyon > > ##### - AFTER: > > my %foo = ( > alpha => 1, > beta => 2, > gamma => 3, > ); > > my @bar = map { { number => $_, character => chr $_, padding => ( ' ' x > $_ ), } } ( 0 .. 32 ); > ============================ > > Should “break-at-old-comma-breakpoints” not have left that “map” as it > was, without combining everything onto a single line? > > Thank you! > > -Felipe Gasper > Houston, TX > >
This is fixed in version 20150815.