<div dir="ltr">Hi Nicolas,<div><br></div><div>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 :-)</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 26, 2017 at 2:58 PM, Nicolas Cellier <span dir="ltr"><<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi Eliot,<br></div>last two commits in inbox are waiting for your blessing<br></div>cheers<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2017-04-26 23:56 GMT+02:00  <span dir="ltr"><<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Nicolas Cellier uploaded a new version of Collections to project The Inbox:<br>
<a href="http://source.squeak.org/inbox/Collections-nice.753.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/inbox<wbr>/Collections-nice.753.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: Collections-nice.753<br>
Author: nice<br>
Time: 26 April 2017, 11:56:06.826092 pm<br>
UUID: 5f5b9a99-afa3-40f2-8fb5-a9a618<wbr>3d10b5<br>
Ancestors: Collections-nice.752<br>
<br>
Nuke obsolete brace constructs used by compiler long long ago.<br>
<br>
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.<br>
<br>
=============== Diff against Collections-nice.752 ===============<br>
<br>
Item was removed:<br>
- ----- Method: Array class>>braceWith: (in category 'brace support') -----<br>
- braceWith: a<br>
-       "This method is used in compilation of brace constructs.<br>
-       It MUST NOT be deleted or altered."<br>
-<br>
-       | array |<br>
-       array := self new: 1.<br>
-       array at: 1 put: a.<br>
-       ^ array!<br>
<br>
Item was removed:<br>
- ----- Method: Array class>>braceWith:with: (in category 'brace support') -----<br>
- braceWith: a with: b<br>
-       "This method is used in compilation of brace constructs.<br>
-       It MUST NOT be deleted or altered."<br>
-<br>
-       | array |<br>
-       array := self new: 2.<br>
-       array at: 1 put: a.<br>
-       array at: 2 put: b.<br>
-       ^ array!<br>
<br>
Item was removed:<br>
- ----- Method: Array class>>braceWith:with:with: (in category 'brace support') -----<br>
- braceWith: a with: b with: c<br>
-       "This method is used in compilation of brace constructs.<br>
-       It MUST NOT be deleted or altered."<br>
-<br>
-       | array |<br>
-       array := self new: 3.<br>
-       array at: 1 put: a.<br>
-       array at: 2 put: b.<br>
-       array at: 3 put: c.<br>
-       ^ array!<br>
<br>
Item was removed:<br>
- ----- Method: Array class>>braceWith:with:with:wit<wbr>h: (in category 'brace support') -----<br>
- braceWith: a with: b with: c with: d<br>
-       "This method is used in compilation of brace constructs.<br>
-       It MUST NOT be deleted or altered."<br>
-<br>
-       | array |<br>
-       array := self new: 4.<br>
-       array at: 1 put: a.<br>
-       array at: 2 put: b.<br>
-       array at: 3 put: c.<br>
-       array at: 4 put: d.<br>
-       ^ array!<br>
<br>
Item was removed:<br>
- ----- Method: Array class>>braceWithNone (in category 'brace support') -----<br>
- braceWithNone<br>
-       "This method is used in compilation of brace constructs.<br>
-       It MUST NOT be deleted or altered."<br>
-<br>
-       ^ self new: 0!<br>
<br>
<br>
</blockquote></div><br></div>
</div></div><br><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>
</div>