Hi Nicolas,

forgive me, I've been too busy :-(.  I think both of these are good and you
should commit.  The next task is to implement FullBlocks along side the V3
embedded blocks.  So as much clean up as possible is appreciated :-)

On Wed, Apr 26, 2017 at 2:58 PM, Nicolas Cellier <
nicolas.cellier.aka.nice at gmail.com> wrote:

> Hi Eliot,
> last two commits in inbox are waiting for your blessing
> cheers
>
> 2017-04-26 23:56 GMT+02:00 <commits at source.squeak.org>:
>
>> Nicolas Cellier uploaded a new version of Collections to project The
>> Inbox:
>> http://source.squeak.org/inbox/Collections-nice.753.mcz
>>
>> ==================== Summary ====================
>>
>> Name: Collections-nice.753
>> Author: nice
>> Time: 26 April 2017, 11:56:06.826092 pm
>> UUID: 5f5b9a99-afa3-40f2-8fb5-a9a6183d10b5
>> Ancestors: Collections-nice.752
>>
>> Nuke obsolete brace constructs used by compiler long long ago.
>>
>> Since encoder v3 (plus closures) there is a dedicated byte code for
>> constructing those arrays dynamically, so the #braceWith: like messages are
>> not sent for a while and won't be used anymore in foreseeable future.
>>
>> =============== Diff against Collections-nice.752 ===============
>>
>> Item was removed:
>> - ----- Method: Array class>>braceWith: (in category 'brace support')
>> -----
>> - braceWith: a
>> -       "This method is used in compilation of brace constructs.
>> -       It MUST NOT be deleted or altered."
>> -
>> -       | array |
>> -       array := self new: 1.
>> -       array at: 1 put: a.
>> -       ^ array!
>>
>> Item was removed:
>> - ----- Method: Array class>>braceWith:with: (in category 'brace
>> support') -----
>> - braceWith: a with: b
>> -       "This method is used in compilation of brace constructs.
>> -       It MUST NOT be deleted or altered."
>> -
>> -       | array |
>> -       array := self new: 2.
>> -       array at: 1 put: a.
>> -       array at: 2 put: b.
>> -       ^ array!
>>
>> Item was removed:
>> - ----- Method: Array class>>braceWith:with:with: (in category 'brace
>> support') -----
>> - braceWith: a with: b with: c
>> -       "This method is used in compilation of brace constructs.
>> -       It MUST NOT be deleted or altered."
>> -
>> -       | array |
>> -       array := self new: 3.
>> -       array at: 1 put: a.
>> -       array at: 2 put: b.
>> -       array at: 3 put: c.
>> -       ^ array!
>>
>> Item was removed:
>> - ----- Method: Array class>>braceWith:with:with:with: (in category
>> 'brace support') -----
>> - braceWith: a with: b with: c with: d
>> -       "This method is used in compilation of brace constructs.
>> -       It MUST NOT be deleted or altered."
>> -
>> -       | array |
>> -       array := self new: 4.
>> -       array at: 1 put: a.
>> -       array at: 2 put: b.
>> -       array at: 3 put: c.
>> -       array at: 4 put: d.
>> -       ^ array!
>>
>> Item was removed:
>> - ----- Method: Array class>>braceWithNone (in category 'brace support')
>> -----
>> - braceWithNone
>> -       "This method is used in compilation of brace constructs.
>> -       It MUST NOT be deleted or altered."
>> -
>> -       ^ self new: 0!
>>
>>
>>
>
>
>
>


-- 
_,,,^..^,,,_
best, Eliot
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20170505/64ce1d1a/attachment.html>