Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

enable almost all compiler warnings on wasm side, disable all warning… #1405

Merged
merged 2 commits into from
Feb 16, 2018

Conversation

moskvanaft
Copy link
Contributor

Rationale: we don't have good tools to debug contracts code yet, let's enable all warnings - maybe they will help us to prevent bugs.

  1. -Weverything warnings enabled for wasm code, some most annoying and useless (in my opinion) warnings manually disabled. We may disable more later.
  2. parameter NOWARNINGS added to cmake macros add_wast_library and add_wast_executable to compile third-party code - musl, libc++
  3. when calling clang to compile to wasm use -isystem to suppress all warnings from musl, libc++ and boost
  4. fixed some annoying warnings in eosiolib that were easy to fix (there are many warnings left, but I am afraid to make massive changes across eosiolib and cause conflicts with other branches)

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

Successfully merging this pull request may close these issues.

2 participants