Back to 0/1 for true/flase setting for autoflush
Henry Van Styn [Mon, 12 May 2014 21:46:19 +0000 (17:46 -0400)]
From IRC:
<ilmari> I'd rather use 0 than undef for false, really
<vanstyn> ok, then I'll change it back... the reason I changed it to
undef was to match existing code, 'abort'
<vanstyn> I generally prefer 0/1 as well, which is why that is what I
started with

lib/Catalyst.pm

index 6bc7e3d..f9cae7a 100755 (executable)
@@ -1256,7 +1256,7 @@ EOF
     # Turn autoflush back off once setup is finished.
     # TODO: this is being done purely for Static::Simple (legacy API), and has been suggested by
     # mst to be removed and require/update Static::Simple to set this flag itself
-    $class->log->autoflush(undef) if ($class->log->can('autoflush'));
+    $class->log->autoflush(0) if ($class->log->can('autoflush'));
 
     return $class || 1; # Just in case someone named their Application 0...
 }