Another unreachable `return' that is not needed

The compilers we use now do not care, but coverity complains.

diff --git a/src/insets/InsetListingsParams.cpp b/src/insets/InsetListingsParams.cpp
index f37566138f86635502bf162ccba3290dc709a0b1..bc1b4cebf5bd3bab7901a098138afebae73dc67d 100644 (file)
--- a/src/insets/InsetListingsParams.cpp
+++ b/src/insets/InsetListingsParams.cpp
@@ -687,9 +687,6 @@ docstring ParValidator::validate(string const & name,
                        return bformat(_("Parameters starting with '%1$s': %2$s"),
                                                                from_utf8(name), from_utf8(matching_names));
        }
-       // this stifles a warning but upsets coverity
-       // coverity[UNREACHABLE]
-       return docstring();
 }