Revert "Use GetValueOrDefault() for nullable PSLanguageMode (#13849)" (#15980)

adityapatwardhan · web-flow · commit ab5e1779a3b3 · 2021-08-23T17:20:29.000-07:00
diff --git a/src/System.Management.Automation/engine/ExternalScriptInfo.cs b/src/System.Management.Automation/engine/ExternalScriptInfo.cs
@@ -242,7 +242,7 @@ private static ScriptBlock ParseScriptContents(Parser parser, string fileName, s
             // If we are in ConstrainedLanguage mode but the defining language mode is FullLanguage, then we need
             // to parse the script contents in FullLanguage mode context.  Otherwise we will get bogus parsing errors
             // such as "Configuration keyword not allowed".
-            if (definingLanguageMode.GetValueOrDefault() == PSLanguageMode.FullLanguage)
+            if (definingLanguageMode.HasValue && (definingLanguageMode == PSLanguageMode.FullLanguage))
             {
                 var context = LocalPipeline.GetExecutionContextFromTLS();
                 if ((context != null) && (context.LanguageMode == PSLanguageMode.ConstrainedLanguage))

Original file line number	Diff line number	Diff line change
`@@ -242,7 +242,7 @@ private static ScriptBlock ParseScriptContents(Parser parser, string fileName, s`
`242`	`242`	`// If we are in ConstrainedLanguage mode but the defining language mode is FullLanguage, then we need`
`243`	`243`	`// to parse the script contents in FullLanguage mode context. Otherwise we will get bogus parsing errors`
`244`	`244`	`// such as "Configuration keyword not allowed".`
`245`		`- if (definingLanguageMode.GetValueOrDefault() == PSLanguageMode.FullLanguage)`
	`245`	`+ if (definingLanguageMode.HasValue && (definingLanguageMode == PSLanguageMode.FullLanguage))`
`246`	`246`	`{`
`247`	`247`	`var context = LocalPipeline.GetExecutionContextFromTLS();`
`248`	`248`	`if ((context != null) && (context.LanguageMode == PSLanguageMode.ConstrainedLanguage))`