Re: perltidy 20101217 not detecting errors

Replies are listed 'Best First'.
Re^2: perltidy 20101217 not detecting errors by Dandello (Monk) on Feb 07, 2011 at 18:24 UTC
Well, my assumption is that the current version of Perltidy should detect the same issues as the old version - missing brackets, especially - and give me an .ERR file. Since I know the file I'm sending to Tidy has errors because I put them there for testing, I assume I should get an error file. AND, when I process the file I know has errors through Perl::Critic online, it tells me "perltidy had errors!! at line 1, column 1. See page 33 of PBP. Severity: 1" Now, I don't know if this is a genuine bug, since when I checked the Perltidy bug report I didn't see anything related to this issue, but when but when four out of the four machines that have had the newest version of Activestate Perl installed on them all have the same issue... This is the log file for a script I was testing that I know for a fact has a bracket commented out so it would throw an error: `perltidy version 20101217 log file on a MSWin32 system, OLD_PERL_VERSI +ON=5.012002 Windows type is XP/.Net Configuration and command line parameters for this run: -g To find error messages search for 'WARNING' with your editor` [download]	[reply] [d/l]
Re^3: perltidy 20101217 not detecting errors by Anonymous Monk on Feb 07, 2011 at 20:29 UTC
See How do I post a question effectively? and distill your problem down to small code snippert (use Perl::Tidy ....), sample input, expected output, actual received output	[reply]
Re^2: perltidy 20101217 not detecting errors by Dandello (Monk) on Feb 07, 2011 at 21:21 UTC
Just for you, Anonymous Input: `#!/usr/bin/perl use strict; use warnings; my $format = 0; if ( $format == 1 ) { print $format; } else { print 'Oops'; exit;` [download] - including an obvious and intentional error. Run through Perltidy from run/cmd: `perltidy -g C:\testing\test.pl` The -g is the flag for a full log including warnings What I expect: an ERR file from Perltidy telling me I have a missing right curly bracket at or before line 13 and the previous unmatched bracket is at line 10. This file should be in the same directory as the script being checked. No ERR file and the log is this: `perltidy version 20101217 log file on a MSWin32 system, OLD_PERL_VERSI +ON=5.012002 Windows type is XP/.Net Configuration and command line parameters for this run: -g To find error messages search for 'WARNING' with your editor` [download] The previous version of Perltidy gave me the ERR file on exactly this type of problem. Now there isn't one. But it still does a nice job prettying up my code.	[reply] [d/l] [select]
Re^3: perltidy 20101217 not detecting errors by Anonymous Monk on Feb 07, 2011 at 22:01 UTC
Aha, now I see. Consulting http://search.cpan.org/dist/Perl-Tidy/CHANGES I see `- added new flag -it=n or --iterations=n This flag causes perltidy to do n complete iterations. For most purposes the default of n=1 should be satisfactory. H +owever n=2` [download] a quick test with -it=2 generates a .ERR file like the previous version used to You're welcome :)	[reply] [d/l]
Re^4: perltidy 20101217 not detecting errors by Dandello (Monk) on Feb 07, 2011 at 23:30 UTC
Thank You Okay - with that flag it does work the way I was expecting/wanted. I'm not sure this is a positive change on their part. And now there is something searchable for people in the same boat as I was.	[reply]
Re^5: perltidy 20101217 not detecting errors by Anonymous Monk on Feb 08, 2011 at 00:00 UTC


There's more than one way to do things
	PerlMonks