add FIXME wrt truncation
http://codereview.appspot.com/579340043
---
** [issues:#5799] Fixes for cross-compilation to x86_64-w64-mingw32**
**Status:** Started
**Created:** Thu Feb 27, 2020 04:14 PM UTC by Jonas Hahnfeld
**Last Updated:** Fri Feb 28, 2020 10:07 AM UTC
**Owner:** Jonas Hahnfeld
Fixes for cross-compilation to x86_64-w64-mingw32
1) Define _POSIX_SOURCE for M_PI on mingw
Recently commit 7396a28bde ("Add enabling extension definitions for
`-std=c++11` option") added a few #defines to config.hh to make POSIX
extensions available. This doesn't seem to work in all cases and I
found this change to fix cross-compilation to x86_64-w64-mingw32.
In general I think this method of explicitly spelling out what each
file needs is superior to gathering all of this in config.hh. It serves
as documentation directly in the code and makes sure that the feature
macro is defined before any other header file is included.
2) Improve cast from pointer to integer
When cross-compiling to x86_64-w64-mingw32, 'long' (32 bit) is smaller
than 'void *' (64 bit). This leads to compiler errors because the cast
truncates. After analysis I think that truncation is actually ok here,
so make this explicit by first casting to a suitable integer and then
truncate to long.
Note that 'suitable integer' is quite a mess: Ideally we would use
intptr_t, but it's optional per the C++ standard. So instead rely on
intmax_t and hope that it works.
3) Use host triple when looking for windres
--host=host-type
the type of system on which the package runs. [...]
The previously used $target variable is meant for compilers.
Please refer to section 14.1 Specifying target triplets in the Autoconf
documentation for more details.
http://codereview.appspot.com/579340043
---
Sent from sourceforge.net because [email protected] is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list._______________________________________________
Testlilyissues-auto mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
