<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Sep 13, 2018, 12:30  <<a href="mailto:commits@source.squeak.org">commits@source.squeak.org</a>> wrote:<br></div><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.807.mcz" rel="noreferrer noreferrer" target="_blank">http://source.squeak.org/inbox/Collections-nice.807.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: Collections-nice.807<br>
Author: nice<br>
Time: 13 September 2018, 9:29:57.057799 pm<br>
UUID: f4686e6e-1820-4c3f-841d-c1904914c147<br>
Ancestors: Collections-topa.806<br>
<br>
Let isSeparator be consistent with Character separators after introduction of nbsp. <br>
<br>
The performance cost should be marginal.<br>
<br>
Note: some (CharacterSet cleanUp: false) might be necessary in postscript, unless this is included in the release process...<br><br>Please include the postscript. There are still some of us that keep images alive for years (or decades) and just update through releases instead of reloading.  Thanks,<br></blockquote></div></div><div dir="auto">CBC</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
=============== Diff against Collections-topa.806 ===============<br>
<br>
Item was changed:<br>
  ----- Method: Character>>isSeparator (in category 'testing') -----<br>
  isSeparator<br>
        "Answer whether the receiver is one of the separator characters--space, <br>
+       cr, tab, line feed, or form feed and nbsp."<br>
-       cr, tab, line feed, or form feed."<br>
<br>
        | integerValue |<br>
+       (integerValue := self asInteger) > 32 ifTrue: [ ^integerValue = 160 "non breaking space" ].<br>
-       (integerValue := self asInteger) > 32 ifTrue: [ ^false ].<br>
        integerValue<br>
                caseOf: {<br>
                        [ 32 "space" ] -> [ ^true ].<br>
                        [ 9 "tab" ] -> [ ^true ].<br>
                        [ 13 "cr"] -> [ ^true ].<br>
                        [ 10 "line feed" ] -> [ ^true ].<br>
                        [ 12 "form feed"] -> [ ^true ] }<br>
                otherwise: [ ^false  ]!<br>
<br>
<br>
</blockquote></div></div></div>