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::ProhibitOneArgSelect - Never write "select($fh)".

Affiliation

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

Author

       Graham TerMarsch <graham@howlingfrog.com>

Configuration

       This Policy is not configurable except for the standard options.

Description

       Conway discourages the use of a raw select() when setting autoflushes.  We'll extend that further by
       simply prohibiting the one-argument form of select() entirely; if you really need it you should know
       when/where/why that is.  For performing autoflushes, Conway recommends the use of "IO::Handle" instead.

         select((select($fh), $|=1)[0]);     # not ok
         select $fh;                         # not ok

          use IO::Handle;
          $fh->autoflush();                   # ok
          *STDOUT->autoflush();               # ok

Name

       Perl::Critic::Policy::InputOutput::ProhibitOneArgSelect - Never write "select($fh)".

See Also

       IO::Handle.

See Also