Re: pmchat

[merlyn]

in reply to pmchat

Brief code review item...

It's bad form to mix prompting with <>. Use <STDIN> instead. The <> operator was designed for pipe filters, looking at @ARGV (and therefore the command line) to determine whether to read from files or read from standard input. Since your program isn't wanting that feature (you really need the input to be from the terminal), your use of <> is misleading.

-- Randal L. Schwartz, Perl hacker