> On 01.10.2019, at 21:49, Thiede, Christoph <Christoph.Thiede at student.hpi.uni-potsdam.de> wrote:
> 
> Hi,
> 
> No special reason. Each trackInfo item describes one tracks item, so I thought it could be a good idea to set them at once. Equal to #setString:setRuns: or #real:imaginary:, for example.

I see your point. However, these two are more private (as most #set… are). The one for Complex could be even thought of as primitive and maybe this method shouldn't be necessary conceptually.

> If you don't think it improves code quality, I can abstain from it. :)

No problem, I was just wondering. 
In cases where you want to communicate the _caller_ that something should be thought of as a unit, I would probably make a class-side convenience constructor.

Keep it coming :D

Best regards
	-Tobias

> 
> Best,
> Christoph
> Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Tobias Pape <Das.Linux at gmx.de>
> Gesendet: Dienstag, 1. Oktober 2019 20:58:49
> An: squeak-dev at lists.squeakfoundation.org
> Betreff: Re: [squeak-dev] The Inbox: Sound-ct.69.mcz
>  
> Hi
> 
> I don't understand the rationale here.
> Care to explain?
> What's so different with the "dual" setter?
> Best regards
>         -Tobias
> > On 01.10.2019, at 20:25, commits at source.squeak.org wrote:
> > 
> > A new version of Sound was added to project The Inbox:
> > http://source.squeak.org/inbox/Sound-ct.69.mcz
> > 
> > ==================== Summary ====================
> > 
> > Name: Sound-ct.69
> > Author: ct
> > Time: 1 October 2019, 8:25:38.912339 pm
> > UUID: b0d38e4d-1231-494f-8502-e1a631e232b3
> > Ancestors: Sound-eem.66
> > 
> > Add convenience setter for MIDIScore
> > 
> > These values are strongly related and should in most cases be set at once.
> > 
> > =============== Diff against Sound-eem.66 ===============
> > 
> > Item was changed:
> >  ----- Method: MIDIFileReader>>asScore (in category 'chunk reading') -----
> >  asScore
> > 
> >        ^ MIDIScore new
> > +              tracks: tracks trackInfo: trackInfo;
> > -              tracks: tracks;
> > -              trackInfo: trackInfo;
> >                tempoMap: tempoMap;
> >                ticksPerQuarterNote: ticksPerQuarter;
> >                source: stream localName
> >  !
> > 
> > Item was added:
> > + ----- Method: MIDIScore>>tracks:trackInfo: (in category 'accessing') -----
> > + tracks: trackList trackInfo: trackInfoList
> > + 
> > +      self tracks: trackList; trackInfo: trackInfoList.!
> > 
> >