divmod-users team mailing list archive

[Laurens Van Houtven <_@xxxxxx>]

Cool,
https://code.launchpad.net/~lvh/divmod.org/storeless-conform-1118498/+merge/148490


On Thu, Feb 14, 2013 at 4:12 PM, Tristan Seligmann
<mithrandi@xxxxxxxxxxxxx>wrote:

> On Thu, Feb 14, 2013 at 3:16 PM, Laurens Van Houtven <_@xxxxxx> wrote:
> > While powerUp does indeed not work with IPowerupIndirector,
> inMemoryPowerup
> > does, and then you can in fact get that thing back from powerupsFor
> (since
> > it checks in-memory ones first). I'm not sure if that's a bug, but at
> least
> > the current implementation passes the test suite.
>
> If it is a bug, it's an existing bug in inMemoryPowerUp so fixing it
> would be out of scope for this change anyway. But I don't think it's
> too important; nothing /bad/ happens when you have a powerup for
> IPowerupIndirector, it's just that nothing useful happens either so if
> you're trying to power something up for that interface you probably
> screwed up.
>
> > Should I fire a merge proposal?
>
> +1
> --
> mithrandi, i Ainil en-Balandor, a faer Ambar
>



-- 
cheers
lvh