<div dir="ltr">Pull request submitted (there are a couple of impacted core packages)  As always, ignore all the commits (that's just me keeping in sync with the main repo) and look at the diffs.<div><br></div><div>Thanks,</div><div>Phil</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 26, 2020 at 9:43 AM Juan Vuletich via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</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 Phil,<br>
<br>
Feel free to replace Strings with FileEntries anywhere. It will be a <br>
great contribution. Those Strings are leftovers from before FileEntry.<br>
<br>
I'm sure there are plenty of places in need of cleanup. As we can't <br>
schedule anyone the task of cleaning up the system, we do it in small <br>
unplanned steps, anytime we find an annoyance and feel like cleaning it.<br>
<br>
On 2/25/2020 5:46 AM, Phil B via Cuis-dev wrote:<br>
> Here's something I've been meaning to fix for a while now: we have <br>
> FileList and the related #fileReaderServicesForFile:suffix: methods <br>
> (and in turn the methods they point to) which almost all have 'File' <br>
> in their name but then proceed to take and expect a string filename <br>
> rather than a FileEntry.  Would there be any objection to a changeset <br>
> that renames the vague *File* method names to *FileEntry* and actually <br>
> passes around FileEntry rather than String instances?  If anyone has a <br>
> problem with that and wants to keep using strings, then how about we <br>
> at least rename these methods from *File* to *Filename*?<br>
<br>
Thanks,<br>
<br>
-- <br>
Juan Vuletich<br>
<a href="http://www.cuis-smalltalk.org" rel="noreferrer" target="_blank">www.cuis-smalltalk.org</a><br>
<a href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev" rel="noreferrer" target="_blank">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a><br>
<a href="https://github.com/jvuletich" rel="noreferrer" target="_blank">https://github.com/jvuletich</a><br>
<a href="https://www.linkedin.com/in/juan-vuletich-75611b3" rel="noreferrer" target="_blank">https://www.linkedin.com/in/juan-vuletich-75611b3</a><br>
@JuanVuletich<br>
<br>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div>