web commit by http://ethan.betacantrips.com/: another weird thing
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sat, 28 Jul 2007 20:56:09 +0000 (20:56 +0000)
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Sat, 28 Jul 2007 20:56:09 +0000 (20:56 +0000)
doc/bugs/weird_syntax_in_aggregate.pm.mdwn [new file with mode: 0644]

diff --git a/doc/bugs/weird_syntax_in_aggregate.pm.mdwn b/doc/bugs/weird_syntax_in_aggregate.pm.mdwn
new file mode 100644 (file)
index 0000000..2be88cd
--- /dev/null
@@ -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