diff options
author | Josh Stone <cuviper@gmail.com> | 2018-02-05 13:43:19 -0800 |
---|---|---|
committer | Alon Zakai <alonzakai@gmail.com> | 2018-02-05 13:43:19 -0800 |
commit | e33d487206e79b5e2fd1382627255a1ea39dad75 (patch) | |
tree | 2c09c2c8005790f1428c7e009191fa6ac1e3f137 | |
parent | 874e89eec8ee4c56ecdb9e6cd68f8366fe983b79 (diff) | |
download | binaryen-e33d487206e79b5e2fd1382627255a1ea39dad75.tar.gz binaryen-e33d487206e79b5e2fd1382627255a1ea39dad75.tar.bz2 binaryen-e33d487206e79b5e2fd1382627255a1ea39dad75.zip |
Fix -Wcatch-value from GCC 8 (#1400)
These instances may simply be caught by reference instead.
-rw-r--r-- | src/tools/asm2wasm.cpp | 2 | ||||
-rw-r--r-- | src/tools/s2wasm.cpp | 2 | ||||
-rw-r--r-- | src/wasm/wasm-s-parser.cpp | 4 |
3 files changed, 4 insertions, 4 deletions
diff --git a/src/tools/asm2wasm.cpp b/src/tools/asm2wasm.cpp index 75a2c4d15..3679a86e1 100644 --- a/src/tools/asm2wasm.cpp +++ b/src/tools/asm2wasm.cpp @@ -87,7 +87,7 @@ int main(int argc, const char *argv[]) { [&trapMode](Options *o, const std::string &argument) { try { trapMode = trapModeFromString(argument); - } catch (std::invalid_argument e) { + } catch (std::invalid_argument& e) { std::cerr << "Error: " << e.what() << "\n"; exit(EXIT_FAILURE); } diff --git a/src/tools/s2wasm.cpp b/src/tools/s2wasm.cpp index 32af57dba..f950b9d94 100644 --- a/src/tools/s2wasm.cpp +++ b/src/tools/s2wasm.cpp @@ -92,7 +92,7 @@ int main(int argc, const char *argv[]) { [&trapMode](Options *o, const std::string &argument) { try { trapMode = trapModeFromString(argument); - } catch (std::invalid_argument e) { + } catch (std::invalid_argument& e) { std::cerr << "Error: " << e.what() << "\n"; exit(EXIT_FAILURE); } diff --git a/src/wasm/wasm-s-parser.cpp b/src/wasm/wasm-s-parser.cpp index 5033047b0..5887adfae 100644 --- a/src/wasm/wasm-s-parser.cpp +++ b/src/wasm/wasm-s-parser.cpp @@ -1408,9 +1408,9 @@ Name SExpressionWasmBuilder::getLabel(Element& s) { uint64_t offset; try { offset = std::stoll(s.c_str(), nullptr, 0); - } catch (std::invalid_argument) { + } catch (std::invalid_argument&) { throw ParseException("invalid break offset"); - } catch (std::out_of_range) { + } catch (std::out_of_range&) { throw ParseException("out of range break offset"); } if (offset > nameMapper.labelStack.size()) throw ParseException("invalid label", s.line, s.col); |