Skip Menu |

This queue is for tickets about the Moo CPAN distribution.

Report information
The Basics
Id: 106276
Status: resolved
Priority: 0/
Queue: Moo

People
Owner: Nobody in particular
Requestors: Support [...] RoxSoft.co.uk
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: Composite name gives: Identifier too long error
Using Moo::Role->create_class_with_roles I'm getting Eval went very, very wrong: { my $isa_check_for_scheme = ${$_[1]->{"\$isa_check_for_scheme"}}; my $isa_check_for__5fbase = ${$_[1]->{"\$isa_check_for__5fbase"}}; and then 25 pages of Perl source later $$_UNQUOTED = \&Web::ComposableRequest::Base__WITH__Web::ComposableRequest::Role::L10N__AND__Web::ComposableRequest::Role::Session__AND__Web::ComposableRequest::Role::Cookie__AND__Web::ComposableRequest::Role::JSON__AND__Web::ComposableRequest::Role::Static::new } 1; Identifier too long at (eval 210) line 40 If I alter sub _composite_name and reduce the threshold for shortening from 252 to 240 the problem goes away. I think that the extra 12 characters comes from ::SUPER::new which is added to the composite class name of 241 characters and so exceeds the 252 character limit
Created a pull request with a test and a patch https://github.com/moose/Role-Tiny/pull/1
On 2015-08-15 06:09:17, PJFL wrote: Show quoted text
> Created a pull request with a test and a patch > https://github.com/moose/Role-Tiny/pull/1
That PR was closed due to a fix being applied on the Moo side -- can this RT ticket also be closed? (@haarg)
Fixed in Moo 2.001000