A little more code tidying

[gitmo/MooseX-SemiAffordanceAccessor.git] / lib / MooseX / SemiAffordanceAccessor / Role / Attribute.pm

diff --git a/lib/MooseX/SemiAffordanceAccessor/Role/Attribute.pm b/lib/MooseX/SemiAffordanceAccessor/Role/Attribute.pm
index 3800bd1..5fca8bd 100644 (file)
--- a/lib/MooseX/SemiAffordanceAccessor/Role/Attribute.pm
+++ b/lib/MooseX/SemiAffordanceAccessor/Role/Attribute.pm
@@ -5,34 +5,28 @@ use warnings;
 
 use Moose::Role;
 
-
-before '_process_options' => sub
-{
+before '_process_options' => sub {
     my $class   = shift;
     my $name    = shift;
     my $options = shift;
 
-    if ( exists $options->{is} &&
-         ! ( exists $options->{reader} || exists $options->{writer} ) )
-    {
-        if ( $options->{is} eq 'ro' )
-        {
+    if ( exists $options->{is}
+        && !( exists $options->{reader} || exists $options->{writer} ) ) {
+        if ( $options->{is} eq 'ro' ) {
             $options->{reader} = $name;
+            delete $options->{is};
         }
-        elsif ( $options->{is} eq 'rw' )
-        {
+        elsif ( $options->{is} eq 'rw' ) {
             $options->{reader} = $name;
 
             my $prefix = 'set';
-            if ( $name =~ s/^_// )
-            {
+            if ( $name =~ s/^_// ) {
                 $prefix = '_set';
             }
 
             $options->{writer} = $prefix . q{_} . $name;
+            delete $options->{is};
         }
-
-        delete $options->{is};
     }
 };
 
@@ -46,11 +40,11 @@ MooseX::SemiAffordanceAccessor::Role::Attribute - Names accessors in a semi-affo
 
 =head1 SYNOPSIS
 
-  Moose::Util::MetaRole::apply_metaclass_roles
-      ( for_class => $p{for_class},
-        attribute_metaclass_roles =>
-        ['MooseX::SemiAffordanceAccessor::Role::Attribute'],
-      );
+  Moose::Util::MetaRole::apply_metaclass_roles(
+      for_class => $p{for_class},
+      attribute_metaclass_roles =>
+          ['MooseX::SemiAffordanceAccessor::Role::Attribute'],
+  );
 
 =head1 DESCRIPTION