<div dir="ltr">Hi Tobias,<div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 6, 2016 at 12:23 AM, Tobias Pape <span dir="ltr"><<a href="mailto:Das.Linux@gmx.de" target="_blank">Das.Linux@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
On 06.07.2016, at 08:17, Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com">eliot.miranda@gmail.com</a>> wrote:<br>
<br>
> On Tue, Jul 5, 2016 at 10:21 PM, Tobias Pape <<a href="mailto:Das.Linux@gmx.de">Das.Linux@gmx.de</a>> wrote:<br>
><br>
> On 06.07.2016, at 01:13, <a href="mailto:commits@source.squeak.org">commits@source.squeak.org</a> wrote:<br>
><br>
> > Eliot Miranda uploaded a new version of System to project The Trunk:<br>
> > <a href="http://source.squeak.org/trunk/System-eem.835.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/System-eem.835.mcz</a><br>
> ><br>
> > ==================== Summary ====================<br>
> ><br>
> > Name: System-eem.835<br>
> > Author: eem<br>
> > Time: 5 July 2016, 6:13:25.980763 pm<br>
> > UUID: 44354df1-cecb-414c-a705-daa84b7388dd<br>
> > Ancestors: System-eem.834<br>
> ><br>
> > Fix reading methods from DataStreams on Spur. Old code used pre-Spur numLits header format, and was not 64-bit aware.<br>
> ><br>
><br>
> Does that mean that pre-Spur methods on datastreams cannot be read in Spur images?<br>
> That would be a bummer…<br>
><br>
> It needs some kind of version stamp on the stream. Point me to some such and we can switch hit.<br>
<br>
</span>There is none.<br>
I would say, we store the data in the format DataStream already know (which happens to be<br>
the Pre-Spur version) and adapt on read. That would ensure backward- and forward-compatibility:<br>
<br>
- We could still read CMs serialized in pre-spur.<br>
- We could also read Spur-CMs in pre-spur images.<br>
<br>
Ok. -- break --<br>
I just realized that what you changed was the read part. so no<br>
problem here. We could change the part to do things differently<br>
when running on spur or non-spur.<br></blockquote><div><br></div><div>The thing is that the two header formats are incompatible. The old one allows up to 2^9-1 literals, the new one up to 2^15-1. So staying with the old format limits us. We need some kind of version stamp. Right now DataStream and ImageSegment are extremely version-dependent and hence restrictive formats. It would be good for someone to revisit them and try and add some kind of versioning information, preferably supporting backward compatibility.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Best regards<br>
<span class="HOEnZb"><font color="#888888"> -Tobias<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
><br>
><br>
> Best regards<br>
> -Tobias<br>
><br>
> > =============== Diff against System-eem.834 ===============<br>
> ><br>
> > Item was changed:<br>
<br>
<br>
<br>
<br>
</div></div></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></div>