just use overload twice

mojolicious · Nov 2, 2014 · 4efe43a · 4efe43a
1 parent 2551b57
commit 4efe43a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/Changes b/Changes
@@ -1,6 +1,6 @@
 
 5.57  2014-11-02
-  - Deprecated stringify support in Mojo::Collection.
+  - Deprecated stringication support in Mojo::Collection.
   - Deprecated Mojo::Collection::pluck in favor of Mojo::Collection::map.
   - Deprecated Mojo::DOM::val.
   - Improved map method in Mojo::Collection to be able to call methods.

diff --git a/lib/Mojo/Collection.pm b/lib/Mojo/Collection.pm
@@ -9,13 +9,11 @@ use Mojo::Util 'deprecated';
 use Scalar::Util 'blessed';
 
 # DEPRECATED in Tiger Face!
-use overload
-  bool => sub {1},
-  '""' => sub {
-  deprecated 'Stringify support in Mojo::Collection is DEPRECATED';
+use overload '""' => sub {
+  deprecated 'Stringification support in Mojo::Collection is DEPRECATED';
   shift->join("\n");
-  },
-  fallback => 1;
+};
+use overload bool => sub {1}, fallback => 1;
 
 our @EXPORT_OK = ('c');
 
@@ -78,6 +76,7 @@ sub new {
   return bless [@_], ref $class || $class;
 }
 
+# DEPRECATED in Tiger Face!
 sub pluck {
   deprecated
     'Mojo::Collection::pluck is DEPRECATED in favor of Mojo::Collection::map';