summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJosh Stone <cuviper@gmail.com>2018-02-05 13:43:19 -0800
committerAlon Zakai <alonzakai@gmail.com>2018-02-05 13:43:19 -0800
commite33d487206e79b5e2fd1382627255a1ea39dad75 (patch)
tree2c09c2c8005790f1428c7e009191fa6ac1e3f137
parent874e89eec8ee4c56ecdb9e6cd68f8366fe983b79 (diff)
downloadbinaryen-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.cpp2
-rw-r--r--src/tools/s2wasm.cpp2
-rw-r--r--src/wasm/wasm-s-parser.cpp4
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);