tests: fix compilation warning in tests/tst-run.cc

It is generally a bad idea to catch exceptions by value, because it
forces an unnecessary copy of the caught value. It's even worse when
the type is polymorphic - in that case the copy will only be a slice
of the original type. This is why recent compilers started to warn
when code catches a polymorphic type by value.

In the specific case fixed here, the warning is spurious, because the
caught exception isn't even used. But it's trivial to fix it, and get
rid of one more warning during test compilation.

Refs #976

Signed-off-by: Nadav Har'El <[email protected]>
Message-Id: <[email protected]>

Author: nyh

tests/tst-run.cc

@@ -44,7 +44,7 @@ int main(int ac, char** av)
         osv::run("/nonexistant.so", 0, nullptr, nullptr);
         // should throw and not get here
         report(false, "Run nonexistant");
-    } catch(osv::launch_error) {
+    } catch(osv::launch_error&) {
         report(true, "Run nonexistant");
     }