Re: [Pythonsound-devel] Bug fixes against latest CVS

[Paul W. <sli...@ya...>]

On Wed, Oct 10, 2001 at 10:59:10PM +0200, Maurizio Umberto Puxeddu wrote:
> > - changed Aggregate to cache end time; it caches onset and duration,
> > so it seemed consistent to do the same with end.
> Well, I don't know... I mean, it's just adding complexity. Anyway I
> commited it.

Not a big deal - all this cache stuff is just an optimization anyway,
and it's still early to be doing that.

I'd consider making cached_onset and its friends semi-private, e.g.
self._cached_onset. The Aggregate's namespace is getting pretty
crowded, and those are attributes that the user should probably not
touch directly.
 
> > - I assume flatten() is supposed to do the same thing as unwind()?
>
> It makes the Aggregate flat and returns nothing.

Aha, so it's like an in-place unwind(). Done.
 
> Do you have a SourceForge account?

Now that you mention it, I do - I'd forgotten about it!
user ID: 38329
login name: slinkp

Guess I'll have to get more familiar with CVS.
It's silly to be emailing patches like this.

-- 
................    paul winkler   ................
 custom calendars:   http://www.calendargalaxy.com
   A member of ARMS:   http://www.reacharms.com
        home page:  http://www.slinkp.com