[Cuis-dev] FileList annoyance

[Juan Vuletich]

Hi Phil,

Feel free to replace Strings with FileEntries anywhere. It will be a 
great contribution. Those Strings are leftovers from before FileEntry.

I'm sure there are plenty of places in need of cleanup. As we can't 
schedule anyone the task of cleaning up the system, we do it in small 
unplanned steps, anytime we find an annoyance and feel like cleaning it.

On 2/25/2020 5:46 AM, Phil B via Cuis-dev wrote:
> Here's something I've been meaning to fix for a while now: we have 
> FileList and the related #fileReaderServicesForFile:suffix: methods 
> (and in turn the methods they point to) which almost all have 'File' 
> in their name but then proceed to take and expect a string filename 
> rather than a FileEntry.  Would there be any objection to a changeset 
> that renames the vague *File* method names to *FileEntry* and actually 
> passes around FileEntry rather than String instances?  If anyone has a 
> problem with that and wants to keep using strings, then how about we 
> at least rename these methods from *File* to *Filename*?

Thanks,

-- 
Juan Vuletich
www.cuis-smalltalk.org
https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev
https://github.com/jvuletich
https://www.linkedin.com/in/juan-vuletich-75611b3
@JuanVuletich