summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>2007-07-28 20:56:09 +0000
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>2007-07-28 20:56:09 +0000
commit40cde964d0db0333d6890cc77051c35f5dafafa4 (patch)
tree9cab0da310f503c5eb277d996efab427f26d15e2
parent3bbf5eb28aa22fe993e7f97c583a1c4d64ccc8e5 (diff)
web commit by http://ethan.betacantrips.com/: another weird thing
-rw-r--r--doc/bugs/weird_syntax_in_aggregate.pm.mdwn7
1 files changed, 7 insertions, 0 deletions
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