From 40cde964d0db0333d6890cc77051c35f5dafafa4 Mon Sep 17 00:00:00 2001 From: joey Date: Sat, 28 Jul 2007 20:56:09 +0000 Subject: [PATCH] web commit by http://ethan.betacantrips.com/: another weird thing --- doc/bugs/weird_syntax_in_aggregate.pm.mdwn | 7 +++++++ 1 file changed, 7 insertions(+) create mode 100644 doc/bugs/weird_syntax_in_aggregate.pm.mdwn diff --git a/doc/bugs/weird_syntax_in_aggregate.pm.mdwn b/doc/bugs/weird_syntax_in_aggregate.pm.mdwn new file mode 100644 index 000000000..2be88cd3b --- /dev/null +++ b/doc/bugs/weird_syntax_in_aggregate.pm.mdwn @@ -0,0 +1,7 @@ + open (IN, "$config{wikistatedir}/aggregate" || + die "$config{wikistatedir}/aggregate: $!"); + +It looks like the intent was "open this file, and die if you can't", +but I'm pretty sure it actually means "open this file and ignore errors +silently". Shouldn't this be `open(IN, $file) || die "$file: $!";` +(i.e. with the parens before the call to `die`)? --Ethan \ No newline at end of file -- 2.26.2