Skip to content

Don't use %zx because it's unsupported on mingw64 - #578

Merged
xor-gate merged 6 commits into
stlink-org:masterfrom
oli-obk:master
Mar 24, 2017
Merged

xor-gate merged 6 commits into
stlink-org:masterfrom
oli-obk:master

Conversation

@oli-obk

@oli-obk oli-obk commented Mar 24, 2017

Copy link
Copy Markdown
Contributor

Also check that no warnings happen on travis

cc #569

Also check that no warnings happen on travis
@xor-gate

Copy link
Copy Markdown
Contributor

Looks good i'm only not 100% sure if clang or other strict compiler will eat this. You should prefer adding a size to the cast so:

(unsigned) becomes (unsigned int)

Then we are good to go.

@oli-obk

oli-obk commented Mar 24, 2017

Copy link
Copy Markdown
Contributor Author

Ok... so after this PR:

  • travis will check gcc and clang on both linux and os-x
  • all compilations must be warning free
  • O2 is used instead of O3 optimizations (this was intended by the CMakeLists.txt, but never actually called due to elseif() not doing the same as else()

i'm only not 100% sure if clang or other strict compiler will eat this.

clang accepted it, but I still made the change.

Maybe there's a warning we can enable to enforce this?

@xor-gate
xor-gate merged commit 78ced6a into stlink-org:master Mar 24, 2017
@xor-gate

Copy link
Copy Markdown
Contributor

Thank you for this improvement and understanding!

@xor-gate

Copy link
Copy Markdown
Contributor

I'm not sure, probably GCC or some analyser have warned me some day about no defined typed, assuming int.

@xor-gate xor-gate added this to the v1.3.2 milestone Mar 24, 2017
@Nightwalker-87 Nightwalker-87 linked an issue Apr 9, 2020 that may be closed by this pull request
6 tasks done
@Nightwalker-87 Nightwalker-87 linked an issue Apr 10, 2020 that may be closed by this pull request
@stlink-org stlink-org locked as resolved and limited conversation to collaborators Apr 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inappropriate driver is taken on using v1.3.0 Windows MinGW64 executables have problems

3 participants