Merge pull request #11 from johnkeeping/catch-by-ref

Catch exceptions by const reference
mateidavid · Nov 5, 2020 · a55d56e · a55d56e
2 parents e90f4a8 + a35cb29
commit a55d56e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/examples/test-strict_fstream.cpp b/examples/test-strict_fstream.cpp
@@ -18,7 +18,7 @@ void test_open(const std::string& stream_class, const std::string& stream_type,
         s_p->open(filename, static_cast< std::ios_base::openmode >(mode));
         exception_thrown = false;
     }
-    catch (std::exception) {}
+    catch (const std::exception &) {}
     std::cout << stream_class << " " << stream_type << " " << (set_fail_bit? "failbit" : "nofailbit") << " "
               << strict_fstream::detail::static_method_holder::mode_to_string(
                   static_cast< std::ios_base::openmode >(mode))

diff --git a/src/strict_fstream.hpp b/src/strict_fstream.hpp
@@ -136,7 +136,7 @@ struct static_method_holder
             is_p->peek();
             peek_failed = is_p->fail();
         }
-        catch (std::ios_base::failure e) {}
+        catch (const std::ios_base::failure &e) {}
         if (peek_failed)
         {
             throw Exception(std::string("strict_fstream: open('")