Add a test for duplicate catch clauses in throw matchers, fix a couple of nitpicks.

googlemock/include/gmock/gmock-matchers.h

@@ -4738,7 +4738,6 @@ class ExceptionMatcherImpl {
   ExceptionMatcherImpl(Matcher<const Err&> matcher)
       : matcher_(std::move(matcher)) {}
 
- public:
   void DescribeTo(::std::ostream* os) const {
     *os << "throws an exception of type " << GetTypeName<Err>();
     if (matcher_.GetDescriber() != nullptr) {
@@ -4775,7 +4774,7 @@ class ExceptionMatcherImpl {
       *listener << "with description \"" << err.what() << "\"";
       return false;
     } catch (...) {
-      *listener << "throws an exception of some other type";
+      *listener << "throws an exception of an unknown type";
       return false;
     }
     *listener << "does not throw any exception";

googlemock/test/gmock-matchers_test.cc

@@ -8139,6 +8139,12 @@ TEST(ThrowsTest, Examples) {
           Property(&std::runtime_error::what, HasSubstr("message"))));
 }
 
+TEST(ThrowsTest, DoesNotGenerateDuplicateCatchClauseWarning) {
+  EXPECT_THAT(
+      []() { throw std::exception(); },
+      Throws<std::exception>());
+}
+
 TEST(ThrowsTest, Describe) {
   Matcher<void (*)()> matcher = Throws<std::runtime_error>();
   std::stringstream ss;