Skip to content
Commits on May 31, 2011
  1. Remove emit() from roles.

    committed May 29, 2011
    Roles are all about static composition, and events are for dynamic
    composition.  Roles that emit() events blur this distinction, so I'm
    deeming them to be "bad".
    
    Additionally, one of Reflex's prime directives is for consumers to
    decide how they'll handle things.  Implementing "interface" callbacks
    in roles (even default ones) takes this away from the consumer.  The
    current solution is to provide declarative helpers to define the
    defaults.  They also make more explicit that it's the consumer's role
    (har) to define callbacks in all cases.  This is slightly less
    convenient, but it's a lot cleaner.
    
    Also add vim format lines so I stop dealing with whitespace
    inconsistencies.
Commits on Sep 9, 2010
  1. Add agorman's test for run_all() called twice, and fix the problem.

    committed Sep 8, 2010
    Agorman (irc.perl.org #reflex) discovered that run_all() called more
    than once asplodes ugly-like.  He provided a nice test case, which
    I've added to Reflex's tests.  It uses Reflex::POE::Wheel::Run, and
    that might need to be removed to make the test more portable.
Something went wrong with that request. Please try again.