package Method::Generate::Accessor;
use strictures 1;
-use Class::Tiny::_Utils;
-use base qw(Class::Tiny::Object);
+use Moo::_Utils;
+use base qw(Moo::Object);
use Sub::Quote;
use B 'perlstring';
+BEGIN {
+ our $CAN_HAZ_XS =
+ _maybe_load_module('Class::XSAccessor')
+ &&
+ (Class::XSAccessor->VERSION > 1.06)
+ ;
+}
sub generate_method {
my ($self, $into, $name, $spec, $quote_opts) = @_;
die "Must have an is" unless my $is = $spec->{is};
- local $self->{captures} = {};
- my $body = do {
- if ($is eq 'ro') {
- $self->_generate_get($name, $spec)
- } elsif ($is eq 'rw') {
- $self->_generate_getset($name, $spec)
+ if ($is eq 'ro') {
+ $spec->{reader} = $name unless exists $spec->{reader};
+ } elsif ($is eq 'rw') {
+ $spec->{accessor} = $name unless exists $spec->{accessor};
+ } elsif ($is eq 'lazy') {
+ $spec->{init_arg} = undef unless exists $spec->{init_arg};
+ $spec->{reader} = $name unless exists $spec->{reader};
+ $spec->{lazy} = 1;
+ $spec->{builder} ||= '_build_'.$name unless $spec->{default};
+ } elsif ($is ne 'bare') {
+ die "Unknown is ${is}";
+ }
+ my %methods;
+ if (my $reader = $spec->{reader}) {
+ if (our $CAN_HAZ_XS && $self->is_simple_get($name, $spec)) {
+ $methods{$reader} = $self->_generate_xs(
+ getters => $into, $reader, $name
+ );
} else {
- die "Unknown is ${is}";
+ $self->{captures} = {};
+ $methods{$reader} =
+ quote_sub "${into}::${reader}"
+ => $self->_generate_get($name, $spec)
+ => delete $self->{captures}
+ ;
}
- };
- quote_sub
- "${into}::${name}" => ' '.$body."\n",
- $self->{captures}, $quote_opts||{}
- ;
+ }
+ if (my $accessor = $spec->{accessor}) {
+ if (
+ our $CAN_HAZ_XS
+ && $self->is_simple_get($name, $spec)
+ && $self->is_simple_set($name, $spec)
+ ) {
+ $methods{$accessor} = $self->_generate_xs(
+ accessors => $into, $accessor, $name
+ );
+ } else {
+ $self->{captures} = {};
+ $methods{$accessor} =
+ quote_sub "${into}::${accessor}"
+ => $self->_generate_getset($name, $spec)
+ => delete $self->{captures}
+ ;
+ }
+ }
+ if (my $writer = $spec->{writer}) {
+ if (
+ our $CAN_HAZ_XS
+ && $self->is_simple_set($name, $spec)
+ ) {
+ $methods{$writer} = $self->_generate_xs(
+ setters => $into, $writer, $name
+ );
+ } else {
+ $self->{captures} = {};
+ $methods{$writer} =
+ quote_sub "${into}::${writer}"
+ => $self->_generate_set($name, $spec)
+ => delete $self->{captures}
+ ;
+ }
+ }
+ if (my $pred = $spec->{predicate}) {
+ $methods{$pred} =
+ quote_sub "${into}::${pred}" =>
+ ' '.$self->_generate_simple_has('$_[0]', $name)."\n"
+ ;
+ }
+ if (my $cl = $spec->{clearer}) {
+ $methods{$cl} =
+ quote_sub "${into}::${cl}" =>
+ " delete \$_[0]->{${\perlstring $name}}\n"
+ ;
+ }
+ if (my $hspec = $spec->{handles}) {
+ my $asserter = $spec->{asserter} ||= '_assert_'.$name;
+ my @specs = do {
+ if (ref($hspec) eq 'ARRAY') {
+ map [ $_ => $_ ], @$hspec;
+ } elsif (ref($hspec) eq 'HASH') {
+ map [ $_ => ref($hspec->{$_}) ? @{$hspec->{$_}} : $hspec->{$_} ],
+ keys %$hspec;
+ } elsif (!ref($hspec)) {
+ map [ $_ => $_ ], Role::Tiny->methods_provided_by($hspec);
+ } else {
+ die "You gave me a handles of ${hspec} and I have no idea why";
+ }
+ };
+ foreach my $spec (@specs) {
+ my ($proxy, $target, @args) = @$spec;
+ $self->{captures} = {};
+ $methods{$proxy} =
+ quote_sub "${into}::${proxy}" =>
+ $self->_generate_delegation($asserter, $target, \@args),
+ delete $self->{captures}
+ ;
+ }
+ }
+ if (my $asserter = $spec->{asserter}) {
+ $self->{captures} = {};
+ $methods{$asserter} =
+ quote_sub "${into}::${asserter}" =>
+ 'do { '.$self->_generate_get($name, $spec).qq! }||die "Attempted to access '${name}' but it is not set"!,
+ delete $self->{captures}
+ ;
+ }
+ \%methods;
}
sub is_simple_attribute {
my ($self, $name, $spec) = @_;
- return !grep $spec->{$_}, qw(lazy default builder isa trigger);
+ # clearer doesn't have to be listed because it doesn't
+ # affect whether defined/exists makes a difference
+ !grep $spec->{$_},
+ qw(lazy default builder isa trigger predicate);
+}
+
+sub is_simple_get {
+ my ($self, $name, $spec) = @_;
+ !($spec->{lazy} and ($spec->{default} or $spec->{builder}));
+}
+
+sub is_simple_set {
+ my ($self, $name, $spec) = @_;
+ !grep $spec->{$_}, qw(isa trigger);
}
sub _generate_get {
my ($self, $name, $spec) = @_;
my $simple = $self->_generate_simple_get('$_[0]', $name);
- my ($lazy, $default, $builder) = @{$spec}{qw(lazy default builder)};
- return $simple unless $lazy and ($default or $builder);
- 'do { '.$self->_generate_use_default(
- '$_[0]', $name, $spec,
- $self->_generate_simple_has('$_[0]', $name),
- ).'; '.$simple.' }';
+ if ($self->is_simple_get($name, $spec)) {
+ $simple;
+ } else {
+ 'do { '.$self->_generate_use_default(
+ '$_[0]', $name, $spec,
+ $self->_generate_simple_has('$_[0]', $name),
+ ).'; '.$simple.' }';
+ }
}
sub _generate_simple_has {
sub generate_get_default {
my $self = shift;
- local $self->{captures} = {};
+ $self->{captures} = {};
my $code = $self->_generate_get_default(@_);
- return ($code, $self->{captures});
+ ($code, delete $self->{captures});
}
sub _generate_use_default {
}
sub generate_simple_get {
- shift->_generate_simple_get(@_);
+ my ($self, @args) = @_;
+ $self->_generate_simple_get(@args);
}
sub _generate_simple_get {
}
sub _generate_set {
- my ($self, $name, $value, $spec) = @_;
- my $simple = $self->_generate_simple_set('$_[0]', $name, $value);
- my ($trigger, $isa_check) = @{$spec}{qw(trigger isa)};
- return $simple unless $trigger or $isa_check;
- my $code = 'do {';
- if ($isa_check) {
- $code .= ' '.$self->_generate_isa_check($name, '$_[1]', $isa_check).';';
- }
- if ($trigger) {
- my $fire = $self->_generate_trigger($name, '$_[0]', '$value', $trigger);
- $code .=
- ' my $value = '.$simple.'; '.$fire.'; '
- .'$value';
+ my ($self, $name, $spec) = @_;
+ if ($self->is_simple_set($name, $spec)) {
+ $self->_generate_simple_set('$_[0]', $name, '$_[1]');
} else {
- $code .= ' '.$simple;
+ my ($trigger, $isa_check) = @{$spec}{qw(trigger isa)};
+ my $simple = $self->_generate_simple_set('$self', $name, '$value');
+ my $code = "do { my (\$self, \$value) = \@_;\n";
+ if ($isa_check) {
+ $code .=
+ " ".$self->_generate_isa_check($name, '$value', $isa_check).";\n";
+ }
+ if ($trigger) {
+ my $fire = $self->_generate_trigger($name, '$self', '$value', $trigger);
+ $code .=
+ " ".$simple.";\n ".$fire.";\n"
+ ." \$value;\n";
+ } else {
+ $code .= " ".$simple.";\n";
+ }
+ $code .= " }";
+ $code;
}
- $code .= ' }';
- return $code;
}
-
+
sub generate_trigger {
my $self = shift;
- local $self->{captures} = {};
+ $self->{captures} = {};
my $code = $self->_generate_trigger(@_);
- return ($code, $self->{captures});
+ ($code, delete $self->{captures});
}
sub _generate_trigger {
}
sub generate_isa_check {
- my $self = shift;
- local $self->{captures} = {};
- my $code = $self->_generate_isa_check(@_);
- return ($code, $self->{captures});
+ my ($self, @args) = @_;
+ $self->{captures} = {};
+ my $code = $self->_generate_isa_check(@args);
+ ($code, delete $self->{captures});
}
sub _generate_isa_check {
my ($self, $name, $type, $values, $sub) = @_;
if (my $quoted = quoted_from_sub($sub)) {
my $code = $quoted->[1];
- my $at_ = 'local @_ = ('.$values.');';
+ my $at_ = '@_ = ('.$values.');';
if (my $captures = $quoted->[2]) {
my $cap_name = qq{\$${type}_captures_for_${name}};
$self->{captures}->{$cap_name} = \$captures;
- return "do {\n".' '.$at_."\n"
- .Sub::Quote::capture_unroll($cap_name, $captures, 6)
- ." ${code}\n }";
+ Sub::Quote::inlinify(
+ $code, $values, Sub::Quote::capture_unroll($cap_name, $captures, 6)
+ );
+ } else {
+ Sub::Quote::inlinify($code, $values);
}
- return 'do { local @_ = ('.$values.'); '.$code.' }';
+ } else {
+ my $cap_name = qq{\$${type}_for_${name}};
+ $self->{captures}->{$cap_name} = \$sub;
+ "${cap_name}->(${values})";
}
- my $cap_name = qq{\$${type}_for_${name}};
- $self->{captures}->{$cap_name} = \$sub;
- return "${cap_name}->(${values})";
}
sub generate_populate_set {
my $self = shift;
- local $self->{captures} = {};
+ $self->{captures} = {};
my $code = $self->_generate_populate_set(@_);
- return ($code, $self->{captures});
+ ($code, delete $self->{captures});
}
sub _generate_populate_set {
."\n${get_indent})";
($spec->{isa}
? " {\n my \$value = ".$get_value.";\n "
- .$self->_generate_isa_check(
- $name, '$value', $spec->{isa}
- ).";\n"
- .' '.$self->_generate_simple_set($me, $name, '$value').";\n"
- ." }\n"
+ .$self->_generate_isa_check(
+ $name, '$value', $spec->{isa}
+ ).";\n"
+ .' '.$self->_generate_simple_set($me, $name, '$value').";\n"
+ ." }\n"
: ' '.$self->_generate_simple_set($me, $name, $get_value).";\n"
)
.($spec->{trigger}
? ' '
- .$self->_generate_trigger(
- $name, $me, $self->_generate_simple_get($me, $name),
- $spec->{trigger}
- )." if ${test};\n"
+ .$self->_generate_trigger(
+ $name, $me, $self->_generate_simple_get($me, $name),
+ $spec->{trigger}
+ )." if ${test};\n"
: ''
);
} else {
" if (${test}) {\n"
.($spec->{isa}
? " "
- .$self->_generate_isa_check(
- $name, $source, $spec->{isa}
- ).";\n"
+ .$self->_generate_isa_check(
+ $name, $source, $spec->{isa}
+ ).";\n"
: ""
)
." ".$self->_generate_simple_set($me, $name, $source).";\n"
.($spec->{trigger}
- ? " "
- .$self->_generate_trigger(
- $name, $me, $self->_generate_simple_get($me, $name),
- $spec->{trigger}
- ).";\n"
- : ""
+ ? " "
+ .$self->_generate_trigger(
+ $name, $me, $self->_generate_simple_get($me, $name),
+ $spec->{trigger}
+ ).";\n"
+ : ""
)
." }\n";
}
sub generate_simple_set {
my $self = shift;
- local $self->{captures} = {};
+ $self->{captures} = {};
my $code = $self->_generate_simple_set(@_);
- return ($code, $self->{captures});
+ ($code, delete $self->{captures});
}
sub _generate_simple_set {
sub _generate_getset {
my ($self, $name, $spec) = @_;
- q{(@_ > 1 ? }.$self->_generate_set($name, q{$_[1]}, $spec)
- .' : '.$self->_generate_get($name).')';
+ q{(@_ > 1}."\n ? ".$self->_generate_set($name, $spec)
+ ."\n : ".$self->_generate_get($name)."\n )";
+}
+
+sub _generate_delegation {
+ my ($self, $asserter, $target, $args) = @_;
+ my $arg_string = do {
+ if (@$args) {
+ # I could, I reckon, linearise out non-refs here using perlstring
+ # plus something to check for numbers but I'm unsure if it's worth it
+ $self->{captures}{'@curries'} = $args;
+ '@curries, @_';
+ } else {
+ '@_';
+ }
+ };
+ "shift->${asserter}->${target}(${arg_string});";
+}
+
+sub _generate_xs {
+ my ($self, $type, $into, $name, $slot) = @_;
+ Class::XSAccessor->import(
+ class => $into,
+ $type => { $name => $slot }
+ );
+ $into->can($name);
}
1;
projects / gitmo/Moo.git / blobdiff