<div dir="ltr"><div>Hi Levente,</div><div><br></div><div>good catch! Yes that would indeed be a better solution. Would you like me to submit a new version to the inbox or will you just move this to treated and create an according version in trunk directly? :)</div><div><br></div><div>Best,</div><div>Tom<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 4, 2020 at 6:00 PM Levente Uzonyi <<a href="mailto:leves@caesar.elte.hu">leves@caesar.elte.hu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Tom,<br>
<br>
On Sat, 3 Oct 2020, <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> wrote:<br>
<br>
> A new version of Monticello was added to project The Inbox:<br>
> <a href="http://source.squeak.org/inbox/Monticello-tobe.729.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/inbox/Monticello-tobe.729.mcz</a><br>
><br>
> ==================== Summary ====================<br>
><br>
> Name: Monticello-tobe.729<br>
> Author: tobe<br>
> Time: 3 October 2020, 12:57:12.690975 pm<br>
> UUID: cbe1a05f-97e6-4780-9762-824be701f76a<br>
> Ancestors: Monticello-ul.728<br>
><br>
> Include a dot when matching against extensions<br>
><br>
> Packages of extension-less formats (e.g. tonel) will otherwise match for e.g. the MCStReader in MCReader>>#canReadFileNamed: if their name happens to end on an "st", e.g. "Roassal3-Sunburst"<br>
><br>
> =============== Diff against Monticello-ul.728 ===============<br>
><br>
> Item was changed:<br>
>  ----- Method: MCVersionReader class>>canReadFileNamed: (in category 'testing') -----<br>
>  canReadFileNamed: fileName<br>
> +     ^ (fileName endsWith: '.', self extension)!<br>
> -     ^ (fileName endsWith: self extension)!<br>
<br>
This is a reasonable change. But the very same method is implemented by <br>
the superclass MCReader. So, I think this method should be removed and <br>
the change should be applied to MCReader class>>canReadFileNamed:.<br>
What do you think?<br>
<br>
<br>
Levente<br>
<br>
</blockquote></div>