darcs patch: Fix concatenatedTimes in sequence.slate.
Brian Rice
water at tunes.org
Sun Nov 20 14:28:00 PST 2005
Your bug fix is correct.
For some reason, darcs apply wouldn't accept this, so I reconstituted
it myself. I also modified it slightly to use "result capacity"
instead of "result size" since size refers to the number of elements,
and the "c new" call to make results might wind up making an
ExtensibleSequence with 0 size but the necessary capacity.
This change is in the main repository now - Christopher, you will
likely have to unpull your patch and then pull the one I created.
Sorry for the trouble, and thanks for catching this!
On Nov 18, 2005, at 6:43 PM, christophercyll at gmail.com wrote:
> Fri Nov 18 18:35:40 PST 2005 christophercyll at gmail.com
> * Fix concatenatedTimes in sequence.slate.
>
> Before: {4. 5. 6.} concatenatedTimes: 2. ===> {4. 5. 6. Nil. Nil.
> Nil}
> After: {4. 5. 6.} concatenatedTimes: 2. ===> {4. 5. 6. 4. 5. 6}
>
> I think this is the correct fix, but I'm very new to slate.
>
>
> New patches:
>
> [Fix concatenatedTimes in sequence.slate.
> christophercyll at gmail.com**20051119023540
>
> Before: {4. 5. 6.} concatenatedTimes: 2. ===> {4. 5. 6. Nil. Nil.
> Nil}
> After: {4. 5. 6.} concatenatedTimes: 2. ===> {4. 5. 6. 4. 5. 6}
>
> I think this is the correct fix, but I'm very new to slate.
>
> ] {
> hunk ./src/lib/sequence.slate 490
> - 0 below: n by: c size do:
> + 0 below: result size by: c size do:
> }
>
> Context:
--
-Brian
http://tunes.org/~water/brice.vcf
More information about the Slate
mailing list