Hi Eliot,<br>
<br>
thanks for the speed-up! Would you mind to have a short look at Files-ct.198? I think there might be a regression in this patch for Windows. :-)<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><font color="#808080"><i>Sent from </i></font><font color="#808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2022-07-13T21:27:01-07:00, eliot.miranda@gmail.com wrote:<br>
<br>
> Much smaller improvement on Windows but still nice to have. Old: '10,100<br>
> per second. 98.6 microseconds per run. 0 % GC time.' New: '14,700 per<br>
> second. 67.8 microseconds per run. 0 % GC time.'<br>
> <br>
> (paths used '\no\where\at\all', '\Users\eliot\Desktop\not there'<br>
> FileDirectory default)<br>
> <br>
> On Wed, Jul 13, 2022 at 9:09 PM <commits at source.squeak.org> wrote:<br>
> <br>
> > Eliot Miranda uploaded a new version of Files to project The Trunk:<br>
> > http://source.squeak.org/trunk/Files-eem.195.mcz<br>
> ><br>
> > ==================== Summary ====================<br>
> ><br>
> > Name: Files-eem.195<br>
> > Author: eem<br>
> > Time: 13 July 2022, 9:09:36.427827 pm<br>
> > UUID: 7ccbfb5c-e5d3-4911-9497-09fdc484d09c<br>
> > Ancestors: Files-eem.194<br>
> ><br>
> > Occasionally FileDirectory>>exists fails.  The old implementation seems<br>
> > unreliable, but it's also slow.  So if possible, implement it above<br>
> > primLookupEntryIn:name:<br>
> ><br>
> ><br>
> > Old:<br>
> > | foo bar baz |<br>
> > foo := FileDirectory on: '/no/where/at/all'.<br>
> > bar := FileDirectory on: '/Users/eliot/Desktop/not there'.<br>
> > baz := FileDirectory default.<br>
> > [foo exists. bar exists. baz exists] bench '8,070 per second. 124<br>
> > microseconds per run. 0 % GC time.'<br>
> ><br>
> > New:<br>
> > | foo bar baz |<br>
> > foo := FileDirectory on: '/no/where/at/all'.<br>
> > bar := FileDirectory on: '/Users/eliot/Desktop/not there'.<br>
> > baz := FileDirectory default.<br>
> > [foo exists. bar exists. baz exists] bench '73,700 per second. 13.6<br>
> > microseconds per run. 0.64 % GC time.'<br>
> ><br>
> > =============== Diff against Files-eem.194 ===============<br>
> ><br>
> > Item was changed:<br>
> >   ----- Method: FileDirectory>>exists (in category 'testing') -----<br>
> >   exists<br>
> > +       "Answer whether the directory exists"<br>
> > +       ^self class currentDirectoryNickname<br>
> > +               ifNotNil:<br>
> > +                       [:me|<br>
> > +                       (self primLookupEntryIn: pathName asVmPathName<br>
> > name: me asVmPathName)<br>
> > +                               ifNil: [false]<br>
> > +                               ifNotNil: [:maybeErr| maybeErr ~~<br>
> > #primFailed]]<br>
> > +               ifNil:<br>
> > +                       [(self primLookupEntryIn: pathName asVmPathName<br>
> > index: 1) ~~ #badDirectoryPath]!<br>
> > - "Answer whether the directory exists"<br>
> > -<br>
> > -       | result |<br>
> > -       result := self primLookupEntryIn: pathName asVmPathName index: 1.<br>
> > -       ^ result ~= #badDirectoryPath<br>
> > - !<br>
> ><br>
> ><br>
> ><br>
> <br>
> -- <br>
> _,,,^..^,,,_<br>
> best, Eliot<br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220713/14903653/attachment.html><br>
> <br>
>