<div dir="ltr"><div dir="ltr">Hi Alistair,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 6, 2019 at 10:50 PM Alistair Grant <<a href="mailto:akgrant0710@gmail.com">akgrant0710@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <br>
Hi Eliot,<br>
<br>
On Wed, 6 Mar 2019 at 19:09, Eliot Miranda <<a href="mailto:noreply@github.com" target="_blank">noreply@github.com</a>> wrote:<br>
><br>
><br>
>   Branch: refs/heads/Cog<br>
>   Home:   <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm" rel="noreferrer" target="_blank">https://github.com/OpenSmalltalk/opensmalltalk-vm</a><br>
>   Commit: 2f27a1e1f213943a6f7860713f5d78f15310ac8d<br>
>       <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/2f27a1e1f213943a6f7860713f5d78f15310ac8d" rel="noreferrer" target="_blank">https://github.com/OpenSmalltalk/opensmalltalk-vm/commit/2f27a1e1f213943a6f7860713f5d78f15310ac8d</a><br>
>   Author: Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>><br>
>   Date:   2019-03-06 (Wed, 06 Mar 2019)<br>
><br>
>   Changed paths:<br>
>     M platforms/unix/plugins/FileAttributesPlugin/faSupport.c<br>
>     M platforms/unix/plugins/FileAttributesPlugin/faSupport.h<br>
>     M platforms/win32/plugins/FileAttributesPlugin/faSupport.c<br>
>     M platforms/win32/plugins/FileAttributesPlugin/faSupport.h<br>
>     M src/plugins/FileAttributesPlugin/FileAttributesPlugin.c<br>
>     M src/plugins/SqueakFFIPrims/ARM32FFIPlugin.c<br>
>     M src/plugins/SqueakFFIPrims/SqueakFFIPrims.c<br>
><br>
>   Log Message:<br>
>   -----------<br>
>   CogVM source as per VMMaker.oscog-eem.2525/FileAttributesPlugin.oscog-eem.50<br>
><br>
> Plugins:<br>
><br>
> ThreadedFFIPlugin: Make sure the ARM identifyingPredefinedMacros do not<br>
> confuse 32 & 64 bits.<br>
><br>
> FileAttributesPlugin:<br>
> Simpification and simulaiton of primitiveFileExists & primitivePathMax,<br>
> using the simpler and more efficient methodreturnXXX: protocol.<br>
<br>
You've replaced:<br>
<br>
sqInt faExists(fapath *aFaPath)<br>
{<br>
if (access(faGetPlatPath(aFaPath), F_OK))<br>
return interpreterProxy->falseObject();<br>
else<br>
return interpreterProxy->trueObject();<br>
}<br>
<br>
with:<br>
<br>
#define faExists(aFaPath) access(faGetPlatPath(aFaPath), F_OK)<br>
<br>
But this appears to invert the value returned (access() should be<br>
treated as a predicate function which returns true if the requested<br>
access is denied).<br></blockquote><div><br></div><div>Oops!! Stupid me!  Good catch; thanks.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<br>
Otherwise, this all looks good.<br>
<br>
Thanks,<br>
Alistair<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div></div>