Assert non-null return values from MethodSymbol.Parameters

dotnet · Jun 17, 2021 · 856df7f · 856df7f
1 parent b17b219
commit 856df7f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Compilers/CSharp/Portable/Symbols/Source/SourceDelegateMethodSymbol.cs b/src/Compilers/CSharp/Portable/Symbols/Source/SourceDelegateMethodSymbol.cs
@@ -126,7 +126,8 @@ public sealed override ImmutableArray<ParameterSymbol> Parameters
  {
  get
  {
- return _parameters;
+ Debug.Assert(!_parameters.IsDefault, $"Expected {nameof(InitializeParameters)} prior to accessing this property.");
+ return _parameters.NullToEmpty();
  }
  }
 

diff --git a/src/Compilers/CSharp/Portable/Symbols/Synthesized/SynthesizedGlobalMethodSymbol.cs b/src/Compilers/CSharp/Portable/Symbols/Synthesized/SynthesizedGlobalMethodSymbol.cs
@@ -167,7 +167,8 @@ public override ImmutableArray<ParameterSymbol> Parameters
  {
  get
  {
- if (_parameters.IsEmpty)
+ Debug.Assert(!_parameters.IsDefault, $"Expected {nameof(SetParameters)} prior to accessing this property.");
+ if (_parameters.IsDefault)
  {
  return ImmutableArray<ParameterSymbol>.Empty;
  }