From 2c952fe8505fc427cdfe30ed4b40343f0f5f39bc Mon Sep 17 00:00:00 2001
From: Jake Bailey <5341706+jakebailey@users.noreply.github.com>
Date: Tue, 21 Mar 2023 16:04:06 -0700
Subject: [PATCH] Remove redundant throws of Debug.assertNever (#53405)

---
 src/compiler/checker.ts                 | 8 ++++----
 src/jsTyping/jsTyping.ts                | 2 +-
 src/services/classifier.ts              | 2 +-
 src/services/codefixes/requireInTs.ts   | 2 +-
 src/services/refactors/extractSymbol.ts | 2 +-
 5 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
index eefaed24674..825d339cfd3 100644
--- a/src/compiler/checker.ts
+++ b/src/compiler/checker.ts
@@ -4736,7 +4736,7 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
             }
         }
         else {
-            throw Debug.assertNever(name, "Unknown entity name kind.");
+            Debug.assertNever(name, "Unknown entity name kind.");
         }
         Debug.assert((getCheckFlags(symbol) & CheckFlags.Instantiated) === 0, "Should never get an instantiated symbol here.");
         if (!nodeIsSynthesized(name) && isEntityName(name) && (symbol.flags & SymbolFlags.Alias || name.parent.kind === SyntaxKind.ExportAssignment)) {
@@ -17040,7 +17040,7 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
                     links.resolvedType = getTypeFromTypeNode(node.type);
                     break;
                 default:
-                    throw Debug.assertNever(node.operator);
+                    Debug.assertNever(node.operator);
             }
         }
         return links.resolvedType;
@@ -33962,7 +33962,7 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
             case SyntaxKind.JsxSelfClosingElement:
                 return resolveJsxOpeningLikeElement(node, candidatesOutArray, checkMode);
         }
-        throw Debug.assertNever(node, "Branch in 'resolveSignature' should be unreachable.");
+        Debug.assertNever(node, "Branch in 'resolveSignature' should be unreachable.");
     }
 
     /**
@@ -47701,7 +47701,7 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
                     currentKind = DeclarationMeaning.SetAccessor;
                     break;
                 default:
-                    throw Debug.assertNever(prop, "Unexpected syntax kind:" + (prop as Node).kind);
+                    Debug.assertNever(prop, "Unexpected syntax kind:" + (prop as Node).kind);
             }
 
             if (!inDestructuring) {
diff --git a/src/jsTyping/jsTyping.ts b/src/jsTyping/jsTyping.ts
index 1278b8bff5b..f31e07f196e 100644
--- a/src/jsTyping/jsTyping.ts
+++ b/src/jsTyping/jsTyping.ts
@@ -467,6 +467,6 @@ function renderPackageNameValidationFailureWorker(typing: string, result: NameVa
         case NameValidationResult.Ok:
             return Debug.fail(); // Shouldn't have called this.
         default:
-            throw Debug.assertNever(result);
+            Debug.assertNever(result);
     }
 }
diff --git a/src/services/classifier.ts b/src/services/classifier.ts
index 9377a9e4f59..b5012a3cf94 100644
--- a/src/services/classifier.ts
+++ b/src/services/classifier.ts
@@ -656,7 +656,7 @@ function getClassificationTypeName(type: ClassificationType): ClassificationType
         case ClassificationType.jsxAttribute: return ClassificationTypeNames.jsxAttribute;
         case ClassificationType.jsxText: return ClassificationTypeNames.jsxText;
         case ClassificationType.jsxAttributeStringLiteralValue: return ClassificationTypeNames.jsxAttributeStringLiteralValue;
-        default: return undefined!; // TODO: GH#18217 throw Debug.assertNever(type);
+        default: return undefined!; // TODO: GH#18217 Debug.assertNever(type);
     }
 }
 
diff --git a/src/services/codefixes/requireInTs.ts b/src/services/codefixes/requireInTs.ts
index 0667683d1ff..37781a5f586 100644
--- a/src/services/codefixes/requireInTs.ts
+++ b/src/services/codefixes/requireInTs.ts
@@ -67,7 +67,7 @@ interface Info {
 function getInfo(sourceFile: SourceFile, program: Program, pos: number): Info | undefined {
     const { parent } = getTokenAtPosition(sourceFile, pos);
     if (!isRequireCall(parent, /*checkArgumentIsStringLiteralLike*/ true)) {
-        throw Debug.failBadSyntaxKind(parent);
+        Debug.failBadSyntaxKind(parent);
     }
 
     const decl = cast(parent.parent, isVariableDeclaration);
diff --git a/src/services/refactors/extractSymbol.ts b/src/services/refactors/extractSymbol.ts
index e353589a07c..151b64e9756 100644
--- a/src/services/refactors/extractSymbol.ts
+++ b/src/services/refactors/extractSymbol.ts
@@ -1001,7 +1001,7 @@ function getDescriptionForFunctionLikeDeclaration(scope: FunctionLikeDeclaration
         case SyntaxKind.SetAccessor:
             return `'set ${scope.name.getText()}'`;
         default:
-            throw Debug.assertNever(scope, `Unexpected scope kind ${(scope as FunctionLikeDeclaration).kind}`);
+            Debug.assertNever(scope, `Unexpected scope kind ${(scope as FunctionLikeDeclaration).kind}`);
     }
 }
 function getDescriptionForClassLikeDeclaration(scope: ClassLikeDeclaration): string {