Skip to content

Commit

Permalink
NativeCall: explicitly-manage does not need its string arg "is rw"
Browse files Browse the repository at this point in the history
it modifies the string by mixing something in, not via the container.
This should fix a test failure in DBIish, which calls explicitly-manage
on a non-container
  • Loading branch information
moritz committed Sep 27, 2015
1 parent cb25b2f commit db0c46e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/NativeCall.pm
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ role ExplicitlyManagedString {
has $.cstr is rw;
}

multi explicitly-manage(Str $x is rw, :$encoding = 'utf8') is export(:DEFAULT,
multi explicitly-manage(Str $x, :$encoding = 'utf8') is export(:DEFAULT,
:utils) {
$x does ExplicitlyManagedString;
my $class = class CStr is repr('CStr') { method encoding() { $encoding; } };
Expand Down

0 comments on commit db0c46e

Please sign in to comment.