<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-11-21 17:29 GMT+01:00 Levente Uzonyi <span dir="ltr"><<a href="mailto:leves@elte.hu" target="_blank">leves@elte.hu</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I checked how the implementation actually works, and I came to the conclusion that using a separate class might be a good idea, because there's no easy way to tell if an Array returned by #tempAt: is a temp vector with some temporaries or just a regular array in a temporary variable. This means that returning the wrapped objects would probably require a lot more changes.<br>
<br>
I'd still refrain from the additional changes which would introduce some overhead in the methods you listed. Why?<br>
For #become:, I understand the reason why you'd want it to add the check, but there are so many other cases to handle (e.g.: "true become: false") if you want to make #become: safer.<br>
For immutability, I don't see the reason for the restriction. What if I, as the programmer, want to do a little hack, and overwrite the value of a temporary variable of a given block. That should work IMHO. So making the array immutable is fine, making the user unable to make it mutable is not.</blockquote><div><br></div><div>Well the problem is that if you make a temp vector immutable, remote temp accessing use unchecked temp vector access that mutate the temp vector even if it is immutable. So basically at:put: would fail on the temp vector but access though the remote temp bytecode would still mutate it.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
Levente</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Sat, 21 Nov 2015, Levente Uzonyi wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Wouldn't it be a better solution to change #tempAt: to return the wrapped object instead of the temp vector?<br>
<br>
Levente<br>
<br>
</blockquote>
</div></div></blockquote></div><br></div></div>