Re: openCOLLADA: Help with GCC6 narrowing conversion

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/15/2016 04:54 PM, Jakub Jelinek wrote:
On Mon, Feb 15, 2016 at 04:46:32PM +0100, Dan Horák wrote:
On Mon, 15 Feb 2016 09:35:03 -0600
Richard Shaw <hobbes1069@xxxxxxxxx> wrote:

On Mon, Feb 15, 2016 at 8:50 AM, Dan Horák <dan@xxxxxxxx> wrote:

On Mon, 15 Feb 2016 08:34:31 -0600
Richard Shaw <hobbes1069@xxxxxxxxx> wrote:

Can someone point me in the right direction? My package
openCOLLADA is FTBFS in rawhide with the following (repeating)
error:


/builddir/build/BUILD/OpenCOLLADA-3335ac164e68b2512a40914b14c74db260e6ff7d/COLLADABaseUtils/src/COLLADABUURI.cpp:57:2:
error: narrowing conversion of '-1' from 'int' to 'char' inside
{ } [-Wnarrowing]
   };

isn't it on ARM? Then you should use explicit "signed char" as char
is unsigned by default on ARM (and other arches), see eg.
http://blog.cdleary.com/2012/11/arm-chars-are-unsigned-by-default/


Thanks Dan, that got it, adding an explicit signed fixed the build.

you are welcome, there will be more packages suffering from the same
problem (I own at least one :-)), sometimes appending "-fsigned-char"
to CFLAGS will be easier.

Note that -fsigned-char changes the ABI, so it might have various
undesirable effects too.

Exactly. Because of this, I -fsigned-char should only be applied as a last resort/work-around to mere program/application packages and not to library packages, IMHO.

Ralf
--
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxxx
http://lists.fedoraproject.org/admin/lists/devel@xxxxxxxxxxxxxxxxxxxxxxx




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux