<br><br><div class="gmail_quote">On Thu, Nov 26, 2009 at 8:43 AM, David T. Lewis <span dir="ltr"><<a href="mailto:lewis@mail.msen.com">lewis@mail.msen.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
I opened a Mantis bug report for this issue:<br>
<br>
<a href="http://bugs.squeak.org/view.php?id=7421" target="_blank">http://bugs.squeak.org/view.php?id=7421</a><br>
<br>
Dave<br></blockquote><div><br></div><div>I wouldn't bother trying to fix the primitive. Both in VisualWorks and at Teleplace the experience has been the same, getting rid of the primitives for next, nextPut: and atEnd sped things up. The system is simply too complex nowadays for these primitives to pay for themselves because they only cover a small number of cases. The primitives support Array and ByteString bow there are so many different kinds of arrays being streamed over that the cost of primitive failures outweigh the benefits of primitive successes.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5"><br>
On Thu, Nov 26, 2009 at 03:10:52PM +0100, Henrik Johansen wrote:<br>
><br>
> Interpreter >> primitiveNextPut:<br>
> limit := self fetchInteger: StreamReadLimitIndex ofObject: stream.<br>
> atIx := (array bitAnd: AtCacheMask) + AtPutBase.<br>
> (index < limit and: [(atCache at: atIx+AtCacheOop) = array])<br>
><br>
> so basically, it needs ReadLimit to be > 0. (Which is only true for (Read)WriteStreams if created with on:from:to:).<br>
><br>
> Changing the first line to<br>
> limit := self fetchInteger: StreamWriteLimitIndex ofObject: stream.<br>
><br>
> and the primitive now works.<br>
> ws := WriteStream on: (String new: 500000).<br>
> [1 to: 500000 do: [:ix | ws nextPut: $a]] timeToRun<br>
><br>
> before: 130<br>
> after: 33<br>
><br>
> Inserted a counter after the prim call, for the loop above it was always >= 500000, with fix it's >=1.<br>
><br>
> Cheers,<br>
> Henry<br>
><br>
</div></div></blockquote></div><br>