[Vm-dev] FloatMathPlugin: patch for squeak 4.19.2 fdlibm.h header

David T. Lewis lewis at mail.msen.com
Sun Dec 13 19:46:32 UTC 2020


On Sun, Dec 13, 2020 at 08:09:27PM +0100, stes at PANDORA.BE wrote:
>  
> 
> I had to run the file through  tr -d '\015' < fdlibm.h
> to delete CR (carriage-return).
> 
> Possibly there are combinations of CR and NL in the file.
> 
> It may also be because of my download of the file.
> 

Yes it was probably just an artifact of the download.


> But besides that - after deleting CR - I had a svn diff,
> and it is close to what I submitted.
> 
> I now get
> 
> bash-4.4$ svn diff
> Index: fdlibm.h
> ===================================================================
> - --- fdlibm.h	(revision 3790)
> +++ fdlibm.h	(working copy)
> @@ -13,11 +13,13 @@
>  /* Sometimes it's necessary to define __LITTLE_ENDIAN explicitly
>     but these catch some common cases. */
>  
> +#ifndef __LITTLE_ENDIAN
>  #if defined(i386) || defined(i486) || \
>  	defined(intel) || defined(x86) || defined(i86pc) || \
>  	defined(__alpha) || defined(__osf__)
>  #define __LITTLE_ENDIAN
>  #endif
> +#endif
>  
>  #ifdef __LITTLE_ENDIAN
> 
> 
> That is OK by me.  This compiles OK now with the SunPRO C compiler.
> 
> I checked both 32bit and 64bit because the Makefile first descends in
> build and then in build64.
> 
> David Stes
> 

I just committed your fix to SVN, revision 3791.

Thanks,
Dave



More information about the Vm-dev mailing list