<div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Yes, sorry, after seeking cbc-Interval-related-changes,</div><div>I wanted to clean a bit the inbox where there is too much cruft accumulated.</div><div><br></div><div>I remembered that this change was rejected, but since it was in history, I moved it to trunk.</div><div>Currently, this method is not active despite the ancestry.</div><div>It could have been rejected as a merge conflict, but there is no merge conflict!<br></div><div>Unless the Collections-topa.806 in ancestry was a different package?</div><div>The ancestor ID is the same: 46b95db5-a773-4113-92f0-5ee905404b49</div><div>So it was just manually reverted by Patrick in Collections-pre.818 which is not a simple merge, but a merge + local changes.</div><div>I see no problem in having
Collections-topa.806
in the history, though it's not very usefull.</div><div>Note that 'browse revisions' menu does not mention this branch because it seems that it does not explore each and every branch.</div><div><br></div><div>About UUID, note that anyone doing this cleaning must remember that MC interface is somehow dangerous.</div><div>It marks ancestors with normal font and non ancestors with bold font.<br></div><div>But this is based on ancestor name (the information we easily get from the server).</div><div>Unfortunately, there are several instances of false ancestors (same package name-author-.number with different UUID)</div><div>For example, all the Installer-Core packages currently in inbox are conflicting with the ones in trunk (different UUID, different contents).</div><div><br></div><div>Once we click on the package, we have access to its UUID (which is displayed in the UI).<br></div><div>Maybe the UI should mark it in bold red in order to warn us that this is a false ancestor!</div><div>That would be helpful.<br></div><div><br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mer. 13 févr. 2019 à 11:40, Levente Uzonyi <<a href="mailto:leves@caesar.elte.hu">leves@caesar.elte.hu</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Okay, it sneaked into the ancestry with Collections-pre.818. That's <br>
actually bad, and should be reverted.<br>
<br>
Levente<br>
<br>
On Wed, 13 Feb 2019, Levente Uzonyi wrote:<br>
<br>
> I thought we agreed not to integrate this. See <br>
> <a href="http://forum.world.st/The-Trunk-Collections-topa-806-mcz-td5084658.html" rel="noreferrer" target="_blank">http://forum.world.st/The-Trunk-Collections-topa-806-mcz-td5084658.html</a><br>
><br>
> Levente<br>
><br>
> On Wed, 13 Feb 2019, <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> wrote:<br>
><br>
>> Nicolas Cellier uploaded a new version of Collections to project The Trunk:<br>
>> <a href="http://source.squeak.org/trunk/Collections-topa.806.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/Collections-topa.806.mcz</a><br>
>><br>
>> ==================== Summary ====================<br>
>><br>
>> Name: Collections-topa.806<br>
>> Author: topa<br>
>> Time: 12 September 2018, 3:28:40.687052 pm<br>
>> UUID: 46b95db5-a773-4113-92f0-5ee905404b49<br>
>> Ancestors: Collections-cmm.805<br>
>><br>
>> Fix separators to include U+00A0 (no break space)<br>
>><br>
>> Thanks Ron!<br>
>><br>
>> =============== Diff against Collections-cmm.805 ===============<br>
>><br>
>> Item was changed:<br>
>> ----- Method: Character class>>separators (in category 'instance <br>
> creation') -----<br>
>> separators<br>
>> + "Answer a collection of space-like separator characters.<br>
>> + Note that we do not consider spaces in >8bit code points yet.<br>
>> + "<br>
>> - "Answer a collection of the standard ASCII separator characters."<br>
>> <br>
>> + ^ #(9 "tab"<br>
>> - ^ #(32 "space"<br>
>> - 13 "cr"<br>
>> - 9 "tab"<br>
>> 10 "line feed"<br>
>> + 12 "form feed"<br>
>> + 13 "cr"<br>
>> + 32 "space"<br>
>> + 160 "non-breaking space, see Unicode Z general category")<br>
>> + collect: [:v | Character value: v] as: String<br>
>> + " To be considered:<br>
>> + 16r1680 OGHAM SPACE MARK<br>
>> + 16r2000 EN QUAD<br>
>> + 16r2001 EM QUAD<br>
>> + 16r2002 EN SPACE<br>
>> + 16r2003 EM SPACE<br>
>> + 16r2004 THREE-PER-EM SPACE<br>
>> + 16r2005 FOUR-PER-EM SPACE<br>
>> + 16r2006 SIX-PER-EM SPACE<br>
>> + 16r2007 FIGURE SPACE<br>
>> + 16r2008 PUNCTUATION SPACE<br>
>> + 16r2009 THIN SPACE<br>
>> + 16r200A HAIR SPACE<br>
>> + 16r2028 LINE SEPARATOR<br>
>> + 16r2029 PARAGRAPH SEPARATOR<br>
>> + 16r202F NARROW NO-BREAK SPACE<br>
>> + 16r205F MEDIUM MATHEMATICAL SPACE<br>
>> + 16r3000 IDEOGRAPHIC SPACE<br>
>> + "!<br>
>> - 12 "form feed")<br>
>> - collect: [:v | Character value: v] as: String!<br>
>><br>
>> Item was changed:<br>
>> + (PackageInfo named: 'Collections') postscript: 'CharacterSet cleanUp: <br>
> false.'!<br>
>> - (PackageInfo named: 'Collections') postscript: 'Character <br>
> initializeClassificationTable'!<br>
><br>
><br>
<br>
</blockquote></div>