logo
Free, unlimited AI code reviews that run on commit
git-lrc git-lrc GitHub Install Now We'd appreciate a star git-lrc - Free, unlimited AI code reviews that run on commit | Product Hunt git-lrc - Free, unlimited AI code reviews that run on commit | Product Hunt

Perl::Critic::Policy::InputOutput::RequireBracedFileHandleWithPrint - Write "print {$FH} $foo, $bar;"

Affiliation

       This Policy is part of the core Perl::Critic distribution.

Author

       Jeffrey Ryan Thalhammer <jeff@imaginative-software.com>

Configuration

       This Policy is not configurable except for the standard options.

Description

       The "print" and "printf" functions have a unique syntax that supports an optional file handle argument.
       Conway suggests wrapping this argument in braces to make it visually stand out from the other arguments.
       When you put braces around any of the special package-level file handles like "STDOUT", "STDERR", and
       "DATA", you must add the '*' sigil or else it won't compile under use strict 'subs'.

         print $FH   "Mary had a little lamb\n";  #not ok
         print {$FH} "Mary had a little lamb\n";  #ok

         print   STDERR   $foo, $bar, $baz;  #not ok
         print  {STDERR}  $foo, $bar, $baz;  #won't compile under 'strict'
         print {*STDERR}  $foo, $bar, $baz;  #perfect!

Name

       Perl::Critic::Policy::InputOutput::RequireBracedFileHandleWithPrint - Write "print {$FH} $foo, $bar;"
       instead of "print $FH $foo, $bar;".

See Also