Thread: [HACKERS] use strict in all Perl programs
Here is a patch to add 'use strict' to all Perl programs (that I could find), or move it to the right place where it was already there. I think that is a pretty standard thing to do nowadays. I tried testing the changes in pgcheckdefines, but it just spits out nonsense before and after. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
Attachment
On Sat, Dec 31, 2016 at 3:07 PM, Peter Eisentraut <peter.eisentraut@2ndquadrant.com> wrote: > Here is a patch to add 'use strict' to all Perl programs (that I could > find), or move it to the right place where it was already there. I > think that is a pretty standard thing to do nowadays. > > I tried testing the changes in pgcheckdefines, but it just spits out > nonsense before and after. What about adding as well "use warnings"? That's standard in all the TAP tests. -- Michael
On 12/31/16 1:34 AM, Michael Paquier wrote: > On Sat, Dec 31, 2016 at 3:07 PM, Peter Eisentraut > <peter.eisentraut@2ndquadrant.com> wrote: >> Here is a patch to add 'use strict' to all Perl programs (that I could >> find), or move it to the right place where it was already there. I >> think that is a pretty standard thing to do nowadays. committed that > What about adding as well "use warnings"? That's standard in all the TAP tests. 'use strict' can be statically checked using perl -c, but 'use warnings' is run-time behavior, so one would have to extensively test the involved programs. Some cursory checking already reveals that this is going to need to more investigation. So in principle yes, but maybe later. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
On 1/5/17 12:37 PM, Peter Eisentraut wrote: > On 12/31/16 1:34 AM, Michael Paquier wrote: >> On Sat, Dec 31, 2016 at 3:07 PM, Peter Eisentraut >> <peter.eisentraut@2ndquadrant.com> wrote: >>> Here is a patch to add 'use strict' to all Perl programs (that I could >>> find), or move it to the right place where it was already there. I >>> think that is a pretty standard thing to do nowadays. > > committed that > >> What about adding as well "use warnings"? That's standard in all the TAP tests. > > 'use strict' can be statically checked using perl -c, but 'use warnings' > is run-time behavior, so one would have to extensively test the involved > programs. Some cursory checking already reveals that this is going to > need to more investigation. So in principle yes, but maybe later. With regard to warnings, I prefer to use: use warnings FATAL => qw(all); This transforms all warnings into errors rather than just printing a message to stderr, which is very easy to miss among the other output. -- -David david@pgmasters.net
On Fri, Jan 6, 2017 at 11:13 PM, David Steele <david@pgmasters.net> wrote: > With regard to warnings, I prefer to use: > > use warnings FATAL => qw(all); > > This transforms all warnings into errors rather than just printing a message > to stderr, which is very easy to miss among the other output. Interesting. A couple of warnings have slipped a couple of times in some TAP tests like those of pg_rewind, so it could be useful to switch to that at least for the tests by detault. -- Michael