Remove "where {1}" from subtype declarations.

This isn't necessary, doesn't do anything, and will interfere with things like
inlining, as the presence of the where clause will make Moose think there's a
real additional constraint on the type.

diff --git a/lib/Catalyst/Engine.pm b/lib/Catalyst/Engine.pm
index 23a1d3f..1e56bdc 100644 (file)
--- a/lib/Catalyst/Engine.pm
+++ b/lib/Catalyst/Engine.pm
@@ -46,8 +46,8 @@ has _response_cb => (
 );
 
 subtype 'Catalyst::Engine::Types::Writer',
-    as duck_type([qw(write close)]),
-    where { 1 };
+    as duck_type([qw(write close)]);
+
 has _writer => (
     is      => 'ro',
     isa     => 'Catalyst::Engine::Types::Writer',

diff --git a/lib/Catalyst/Script/Server.pm b/lib/Catalyst/Script/Server.pm
index 328773c..df61d9d 100644 (file)
--- a/lib/Catalyst/Script/Server.pm
+++ b/lib/Catalyst/Script/Server.pm
@@ -48,8 +48,8 @@ has port => (
 use Moose::Util::TypeConstraints;
 class_type 'MooseX::Daemonize::Pid::File';
 subtype 'Catalyst::Script::Server::Types::Pidfile',
-    as 'MooseX::Daemonize::Pid::File',
-    where { 1 };
+    as 'MooseX::Daemonize::Pid::File';
+
 coerce 'Catalyst::Script::Server::Types::Pidfile', from Str, via {
     try { Class::MOP::load_class("MooseX::Daemonize::Pid::File") }
     catch {