<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-04-28 7:47 GMT+02:00 Eliot Miranda <span dir="ltr"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br><div dir="auto"><div>Ben,<br></div><div><br>On Apr 27, 2017, at 7:06 PM, Ben Coman <<a href="mailto:btc@openinworld.com" target="_blank">btc@openinworld.com</a>> wrote:<br><br></div><blockquote type="cite"><div><span></span></div></blockquote><blockquote type="cite"><div><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 28, 2017 at 6:16 AM, Eliot Miranda <span dir="ltr"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 27, 2017 at 2:38 PM, Nicolas Cellier <span dir="ltr"><<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmai<wbr>l.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br><div dir="ltr"><div><div>Hi Eliot,<br></div>it's not about my favourite editor, it's about the universally available one:<br><br><a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/57c9ef837f7b0914351c4836734d1df3c880c288" target="_blank">https://github.com/OpenSmallta<wbr>lk/opensmalltalk-vm/commit/57c<wbr>9ef837f7b0914351c4836734d1df3c<wbr>880c288</a><br><br></div>The LF are displayed correctly in Squeak/Pharo and we can use nextLine in replacement of upTo: Character cr, so I fail to see the problem of LF.<br>Could you explain?<br></div></blockquote><div><br></div><div>It's ok.  I give up.  This is another example of the tail wagging the dog, which gets on my nerves.  But getting them to fix their interface is way more difficult than us accommodating their limitations so go ahead.</div></div></div></div></blockquote><div><br></div><div>Its not github!  When looking into Sophie's issue, opening those ST scripts were unreadable </div><div>when opened in `vi` .  It was a pain that I had to break context to hunt for a GUI text app to read them</div></div></div></div></div></blockquote><div><br></div>It's a three liner to wrap vi in a script that converts from CR to LF, invokes vi and then concerts back.  Is it really that problematic?<div><br></div><div><br><blockquote type="cite"><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>CR-only line endings are sooooooo last century, for machines like: Commodore 8-bit machines, Acorn BBC, ZX Spectrum, TRS-80, Apple II family, Oberon, the classic Mac OS up to version 9, MIT Lisp Machine and OS-9 [1].  These dogs died a long time ago ;)</div><div><br></div><div>Is there something particularly Smalltalk related I'm not aware of that makes CR-only line endings preferable?</div><div><br></div><div>[1] <a href="https://en.wikipedia.org/wiki/Newline" target="_blank">https://en.wikipedia.org/wiki/<wbr>Newline</a></div><div><br></div><div>cheers -ben</div><div><br></div></div></div></div></div></blockquote></div></div></blockquote><div><br></div><div>Ben,<br></div><div>There's no need to have three ways to do the same thing, line ending. So Smalltalk has chosen one and stick to it. it was CR.<br></div><div>Thus you will see a message #cr used everywhere in the code base (either sent to Character or to a WriteStream to write an end of line).<br></div><div>Also, carriage return is used thru a CR variable in TextConstants pool dictionary.<br><br></div><div>The wisdom was that connections to/from outside world (external streams like files/sockets...) should do the conversion in/out the Smalltalk space.<br></div><div>But this has been inequally applied, especially in Squeak/Pharo where the zoo of Stream is not so well engineered (it's a mess!).<br><br></div><div>Since the boundary was messy, I've taken the responsibility to introduce the complexity inside the code base by:<br></div><div>- changing display of LF and adding hacks to display a single end of line in case of CR-LF pair<br></div><div>- introduce nextLine et a few other methods to deal with the zoo of line endings<br></div><div>I'm still not completely sure that it was a good idea... Maybe<br></div><br><div>If we want to change our conventions to be more fashioned, we must get rid of cr message in the code base and replace it with something like eol (end of line).<br></div><div>Cuis has changed the CR to LF inside the image.<br><br></div><div>Nicolas<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div><blockquote type="cite"><div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2017-04-27 23:28 GMT+02:00 Eliot Miranda <span dir="ltr"><<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Please revert this.  It is easy to write a wrapper around your favourite editor to map returns.  These scripts should be in CR endings.<br>
<br>
_,,,^..^,,,_ (phone)<br>
<br>
> On Apr 27, 2017, at 12:32 PM, GitHub <<a href="mailto:noreply@github.com" target="_blank">noreply@github.com</a>> wrote:<br>
><br>
>  Branch: refs/heads/Cog<br>
>  Home:   <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm" rel="noreferrer" target="_blank">https://github.com/OpenSmallt<wbr>alk/opensmalltalk-vm</a><br>
>  Commit: 9fd4e371ae0895078fce13ea35c491<wbr>daf0e448e0<br>
>      <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/9fd4e371ae0895078fce13ea35c491daf0e448e0" rel="noreferrer" target="_blank">https://github.com/OpenSmallta<wbr>lk/opensmalltalk-vm/commit/9fd<wbr>4e371ae0895078fce13ea35c491daf<wbr>0e448e0</a><br>
>  Author: Nicolas Cellier <<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmai<wbr>l.com</a>><br>
>  Date:   2017-04-27 (Thu, 27 Apr 2017)<br>
><br>
>  Changed paths:<br>
>    M .gitattributes<br>
>    M image/BuildSpurReader64Image.s<wbr>t<br>
>    M image/<a href="http://BuildSpurTrunk64Image.st" target="_blank">BuildSpurTrunk64Image.st</a><br>
>    M image/BuildSqueakSpurTrunkVMMa<wbr>kerIma<a href="http://ge.st" target="_blank">ge.st</a><br>
>    M image/CompiledMethod-usesAlter<a href="http://nateBytecodeSet.st" target="_blank"><wbr>nateBytecodeSet.st</a><br>
>    M image/<a href="http://FT2Constants.st" target="_blank">FT2Constants.st</a><br>
>    M image/<a href="http://LoadReader.st" target="_blank">LoadReader.st</a><br>
>    M image/<a href="http://LoadSistaSupport.st" target="_blank">LoadSistaSupport.st</a><br>
>    M image/Object-<a href="http://performwithwithwithwithwith.st" target="_blank">performwithwithwi<wbr>thwithwith.st</a><br>
>    M image/<a href="http://RunATestClass.st" target="_blank">RunATestClass.st</a><br>
>    M image/<a href="http://StartReader.st" target="_blank">StartReader.st</a><br>
>    M image/UpdateSqueakTrunkImage.s<wbr>t<br>
><br>
>  Log Message:<br>
>  -----------<br>
>  Use LF instead of CR as image/*.st line ending<br>
><br>
> This is to be able to review/blame/ etc... from GitHub web interface (1 liner with 500+ columns is not tool friendly)<br>
><br>
><br>
>  Commit: c4881946c2bb8b7da7b191489dbf7a<wbr>e180b05f51<br>
>      <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/c4881946c2bb8b7da7b191489dbf7ae180b05f51" rel="noreferrer" target="_blank">https://github.com/OpenSmallta<wbr>lk/opensmalltalk-vm/commit/c48<wbr>81946c2bb8b7da7b191489dbf7ae18<wbr>0b05f51</a><br>
>  Author: Nicolas Cellier <<a href="mailto:nicolas.cellier.aka.nice@gmail.com" target="_blank">nicolas.cellier.aka.nice@gmai<wbr>l.com</a>><br>
>  Date:   2017-04-27 (Thu, 27 Apr 2017)<br>
><br>
>  Changed paths:<br>
>    M platforms/Cross/plugins/Croque<wbr>tPlugin/CroquetPlugin.h<br>
>    M platforms/Cross/plugins/Croque<wbr>tPlugin/TriBoxStub.c<br>
><br>
>  Log Message:<br>
>  -----------<br>
>  Merge branch 'Cog' of <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm" rel="noreferrer" target="_blank">https://github.com/OpenSmallta<wbr>lk/opensmalltalk-vm</a> into Cog<br>
><br>
><br>
> Compare: <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/compare/bfe983b7a720...c4881946c2bb" rel="noreferrer" target="_blank">https://github.com/OpenSmallta<wbr>lk/opensmalltalk-vm/compare/bf<wbr>e983b7a720...c4881946c2bb</a><br>
</blockquote></div><br></div></div></div></div></div></div>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail-m_-1756128816958977645gmail-m_-7379119062949846874gmail_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>
<br></blockquote></div><br></div></div>
</div></blockquote></div></div><br></blockquote></div><br></div></div>