Ticket #776 (closed defect: fixed)
PerlTidy needs -nse and -nst options
| Reported by: | misterwhipple | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | Perl Tidy | Version: | trunk |
| Keywords: | tidy perltidy plugin | Cc: |
Description
Summary: If a user's .perltidyrc contains either the -se or -st options (or just the -pbp option, which implies the other two), this causes Padre::Plugin::PerlTidy? to issue an error.
Steps To Duplicate:
- Add the -st option to .perltidyrc.
- With a Perl file open in Padre, select Plugins->PerlTidy?->"Tidy the Active Document" on the Padre menu.
Symptoms: A popup dialog has the title "PerlTidy? Error" and the text "You may not specify a destination array and -st together".
Workaround: Don't add -se or -st to .perltidyrc. If you add -pbp, also add -nst and -nse.
Fix: In Padre::Plugin::PerlTidy?.pm, add the options -nst and -nse were Perl::Tidy::perltidy() is called (in the _tidy and _export subs). A patch against trunk 9372 is attached.
Attachments
Change History
Changed 3 years ago by misterwhipple
- Attachment Padre-PerlTidy-nsenst.patch added
Changed 3 years ago by misterwhipple
- Attachment Padre-PerlTidy-nsenst-2.patch added
Corrects the conflict created by my uploading a patch with an error.
comment:1 Changed 3 years ago by misterwhipple
My first patch had an error, and it got committed before I uploaded the correction. I'll upload another patch, against trunk 9374, to resolve.
Immunize Padre::Plugin::PerlTidy? against -st option in .perltidyrc