Made a small review ;) On Wed, Nov 15, 2017 at 10:34 AM, Alistair Grant <akgrant0...@gmail.com> wrote:
> On 14 November 2017 at 22:36, Alistair Grant <akgrant0...@gmail.com> > wrote: > > On 10 November 2017 at 22:30, Alistair Grant <akgrant0...@gmail.com> > wrote: > >> On 9 November 2017 at 22:12, Norbert Hartl <norb...@hartl.name> wrote: > >>> is there a good explanation why > >>> > >>> ‚foo‘ asFileReference parent basename > >>> > >>> gives ‚/‘ ? > >> > >> I'd like to think about this a bit more, but at first glance the > >> problem seems to be that Path>>basename was written only taking > >> Absolute paths in to consideration. > >> > >> So RelativePath>>basename needs to be added: > >> > >> basename > >> "Returns the base of the basename, > >> i.e. > >> foo/gloops.taz basename is 'gloops.taz' > >> . basename is '.'" > >> self size == 0 > >> "the current directory" > >> ifTrue: [ ^ '.']. > >> ^ self at: self size > > > > Fogbugz 20693: https://pharo.fogbugz.com/f/cases/20693/Incorrect- > basename-of-empty-relative-path > > > > I've got the code changes done, but am having troubles with Iceberg > > (which I'll post about if I can't get it working). > > PR #481 submitted (the Iceberg problems were pre-keyboard :-)) > > Thanks, > Alistair > > -- Guille Polito Research Engineer Centre de Recherche en Informatique, Signal et Automatique de Lille CRIStAL - UMR 9189 French National Center for Scientific Research - *http://www.cnrs.fr <http://www.cnrs.fr>* *Web:* *http://guillep.github.io* <http://guillep.github.io> *Phone: *+33 06 52 70 66 13