From: Henry Van Styn <vanstyn@intellitree.com>
Date: Mon, 12 May 2014 21:46:19 +0000 (-0400)
Subject: Back to 0/1 for true/flase setting for autoflush
X-Git-Tag: 5.90065~3
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=catagits%2FCatalyst-Runtime.git;a=commitdiff_plain;h=eb0d3cdea253b57559d190821a4f11b6265661ab

Back to 0/1 for true/flase setting for autoflush

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
---

diff --git a/lib/Catalyst.pm b/lib/Catalyst.pm
index 6bc7e3d..f9cae7a 100755
--- a/lib/Catalyst.pm
+++ b/lib/Catalyst.pm
@@ -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...
 }