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