Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compiler warnings (linux) #14

Open
gatuin opened this issue Jun 8, 2020 · 1 comment
Open

Compiler warnings (linux) #14

gatuin opened this issue Jun 8, 2020 · 1 comment

Comments

@gatuin
Copy link

gatuin commented Jun 8, 2020

There are compiler warnings being generated when this is included in any build. Builds should be able to complete with no warnings. We have an automated build process and it will mark any build as failed if there are warnings. One of those warnings is a constant that is redefined to a different value - that is a big warning flag for potential bugs.

In general it's not good practice to ignore compiler warnings.

# github.com/jdeng/goheif/libde265
In file included from libde265-all.inl:37:0,
                 from libde265.cc:2:
../../go/pkg/mod/github.com/jdeng/[email protected]/libde265/libde265/slice.cc:2447:0: warning: "MAX_PREFIX" redefined
 #define MAX_PREFIX 64

In file included from libde265-all.inl:13:0,
                 from libde265.cc:2:
../../go/pkg/mod/github.com/jdeng/[email protected]/libde265/libde265/cabac.cc:419:0: note: this is the location of the previous definition
 #define MAX_PREFIX 32

cc1plus: warning: unrecognized command line option ‘-Wno-constant-conversion’
@tagatac
Copy link

tagatac commented Jul 12, 2022

These too:

# github.com/adrium/goheif/libde265
In file included from libde265.cc:2:
In file included from vendor/github.com/adrium/goheif/libde265/libde265-all.inl:45:
vendor/github.com/adrium/goheif/libde265/libde265/x86/sse-motion.cc:3530:68: warning: implicit conversion from 'int' to 'short' changes value from 65535 to -1 [-Wconstant-conversion]
vendor/github.com/adrium/goheif/libde265/libde265/x86/sse-motion.cc:3530:58: warning: implicit conversion from 'int' to 'short' changes value from 65535 to -1 [-Wconstant-conversion]
vendor/github.com/adrium/goheif/libde265/libde265/x86/sse-motion.cc:3530:48: warning: implicit conversion from 'int' to 'short' changes value from 65535 to -1 [-Wconstant-conversion]
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants