gcc 4.7.0 error : 0x80000000 does not fit into int32_t #1659

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@jeremyz
Contributor

jeremyz commented Apr 11, 2012

gcc (GCC) 4.7.0 20120407 (prerelease)

vm/builtin/unpack19.cpp:285:5: error: narrowing conversion of ‘2147483648u’ from ‘unsigned int’ to ‘const int32_t {aka const int}’ inside { } is ill-formed in C++11 [-Werror=narrowing]

@jfirebaugh

This comment has been minimized.

Show comment Hide comment
@jfirebaugh

jfirebaugh Apr 11, 2012

Member

These files are generated from rapa and imported into this repository, and it looks like there's a related PR there already. It uses casts though, so your change might be preferable.

Member

jfirebaugh commented Apr 11, 2012

These files are generated from rapa and imported into this repository, and it looks like there's a related PR there already. It uses casts though, so your change might be preferable.

@dbussink

This comment has been minimized.

Show comment Hide comment
@dbussink

dbussink Apr 11, 2012

Member

Hmm, I actually prefer this fix. I'm going to apply this to rapa, regenerate the files and push them. Therefore closing this pull request.

Member

dbussink commented Apr 11, 2012

Hmm, I actually prefer this fix. I'm going to apply this to rapa, regenerate the files and push them. Therefore closing this pull request.

@dbussink dbussink closed this Apr 11, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment