Another forgotten 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 bc1b4cebf5bd3bab7901a098138afebae73dc67d..4b90ce7e91128495817fed52aa140ae2f5e1819c 100644 (file)
--- a/src/insets/InsetListingsParams.cpp
+++ b/src/insets/InsetListingsParams.cpp
@@ -223,9 +223,6 @@ docstring ListingsParam::validate(string const & par) const
                        return bformat(_("Try one of %1$s."), from_utf8(info_));
                else
                        return bformat(_("I guess you mean %1$s."), from_utf8(matching_names));
-               // this stifles a warning but upsets coverity
-               // coverity[UNREACHABLE]
-               return docstring();
        }
        case SUBSETOF:
                if (par2.empty() && !onoff_) {