Remove default case for unhandled enum values

Code consistency in switch statement

Author: snarang181

clang-tools-extra/clang-doc/BitcodeReader.cpp

@@ -315,14 +315,13 @@ static llvm::Error parseRecord(const Record &R, unsigned ID,
 
 static llvm::Error parseRecord(const Record &R, unsigned ID,
                                llvm::StringRef Blob, CommentInfo *I) {
+  llvm::SmallString<16> KindStr;
   switch (ID) {
-  case COMMENT_KIND: {
-    llvm::SmallString<16> KindStr;
+  case COMMENT_KIND:
     if (llvm::Error Err = decodeRecord(R, KindStr, Blob))
       return Err;
     I->Kind = stringToCommentKind(KindStr);
     return llvm::Error::success();
-  }
   case COMMENT_TEXT:
     return decodeRecord(R, I->Text, Blob);
   case COMMENT_NAME:

clang-tools-extra/clang-doc/HTMLGenerator.cpp

@@ -678,10 +678,19 @@ static std::unique_ptr<HTMLNode> genHTML(const CommentInfo &I) {
     return std::make_unique<TextNode>(I.Text);
   }
 
-  // For now, no handling — fallthrough.
-  default:
+  // For now, return nullptr for unsupported comment kinds
+  case CommentKind::CK_InlineCommandComment:
+  case CommentKind::CK_HTMLStartTagComment:
+  case CommentKind::CK_HTMLEndTagComment:
+  case CommentKind::CK_ParamCommandComment:
+  case CommentKind::CK_TParamCommandComment:
+  case CommentKind::CK_VerbatimBlockComment:
+  case CommentKind::CK_VerbatimBlockLineComment:
+  case CommentKind::CK_VerbatimLineComment:
+  case CommentKind::CK_Unknown:
     return nullptr;
   }
+  llvm_unreachable("Unhandled CommentKind");
 }
 
 static std::unique_ptr<TagNode> genHTML(const std::vector<CommentInfo> &C) {