Change PROTOTYPE to issue reference

src/Compilers/CSharp/Portable/Lowering/SyntheticBoundNodeFactory.cs

@@ -1598,7 +1598,7 @@ internal BoundExpression MakeNullCheck(SyntaxNode syntax, BoundExpression rewrit
 
         internal BoundExpression MakeNullableHasValue(SyntaxNode syntax, BoundExpression expression)
         {
-            // PROTOTYPE(param-nullchecking): consider restoring the 'private' accessibility of 'static LocalRewriter.UnsafeGetNullableMethod()'
+            // https://github.com/dotnet/roslyn/issues/58335: consider restoring the 'private' accessibility of 'static LocalRewriter.UnsafeGetNullableMethod()'
             return BoundCall.Synthesized(syntax, expression, LocalRewriter.UnsafeGetNullableMethod(syntax, expression.Type, CodeAnalysis.SpecialMember.System_Nullable_T_get_HasValue, Compilation, Diagnostics));
         }
 
@@ -1673,7 +1673,7 @@ internal BoundExpression RewriteNullableNullEquality(
 
             return result;
         }
-        // PROTOTYPE(BangBang): Re-enable annotations
+        // https://github.com/dotnet/roslyn/issues/58335: Re-enable annotations
 #nullable enable
     }
 }

src/Compilers/CSharp/Portable/Parser/LanguageParser.cs

@@ -11540,7 +11540,7 @@ bool isParenVarCommaSyntax()
                             return true;
                         }
                         // ( x!! , [...]
-                        // PROTOTYPE(BangBang): https://github.com/dotnet/roslyn/pull/46520#discussion_r466650228
+                        // https://github.com/dotnet/roslyn/issues/58335: https://github.com/dotnet/roslyn/pull/46520#discussion_r466650228
                         if (after.Kind == SyntaxKind.ExclamationToken
                             && this.PeekToken(3).Kind == SyntaxKind.ExclamationToken
                             && after.GetTrailingTriviaWidth() == 0

src/Compilers/CSharp/Portable/Symbols/Source/ParameterHelpers.cs

@@ -789,15 +789,15 @@ private static RefKind GetModifiers(SyntaxTokenList modifiers, out SyntaxToken r
             return refKind;
         }
 
-        // PROTOTYPE(param-nullchecking): consider whether we should adjust the set of locations where we call this
+        // https://github.com/dotnet/roslyn/issues/58335: consider whether we should adjust the set of locations where we call this
         internal static void ReportParameterNullCheckingErrors(DiagnosticBag diagnostics, ParameterSymbol parameter)
         {
             if (!parameter.IsNullChecked)
             {
                 return;
             }
             Location location = parameter.Locations.FirstOrNone();
-            // PROTOTYPE(param-nullchecking): can we simplify with the other overload?
+            // https://github.com/dotnet/roslyn/issues/58335: can we simplify with the other overload?
             if (Binder.GetWellKnownTypeMember(parameter.DeclaringCompilation, WellKnownMember.System_ArgumentNullException__ctorString, out UseSiteInfo<AssemblySymbol> useSiteInfo) is null)
             {
                 diagnostics.Add(useSiteInfo.DiagnosticInfo, location);

src/Compilers/CSharp/Test/IOperation/IOperation/IOperationTests_NullCheckedParameters.cs

@@ -2408,7 +2408,7 @@ void M(string x!!) { }
     Statements (0)");
         }
 
-        [Fact(Skip = "PROTOTYPE(param-nullchecking): MakeMemberMissing doesn't work as expected with our method of obtaining Nullable<T>.HasValue in this scenario")]
+        [Fact(Skip = "https://github.com/dotnet/roslyn/issues/58335: MakeMemberMissing doesn't work as expected with our method of obtaining Nullable<T>.HasValue in this scenario")]
         public void TestNullCheckedMethodWithMissingHasValue()
         {
             var source =

src/Compilers/CSharp/Test/Semantic/Semantics/NullCheckedParameterTests.cs

@@ -280,7 +280,7 @@ void M2(__arglist!!)
                     Diagnostic(ErrorCode.ERR_SyntaxError, "!").WithArguments(",", "!").WithLocation(8, 22));
         }
 
-        [Fact(Skip = "PROTOTYPE(BangBang)")]
+        [Fact(Skip = "https://github.com/dotnet/roslyn/issues/58335")]
         public void NullCheckedArgList()
         {
             var source = @"

src/Compilers/Core/Portable/Operations/ControlFlowGraphBuilder.cs

@@ -1481,7 +1481,7 @@ bool visitPossibleUsingDeclarationInLabel(ILabeledOperation labelOperation)
 
             EnterRegion(new RegionBuilder(ControlFlowRegionKind.LocalLifetime, locals: operation.Locals));
 
-            // PROTOTYPE(param-nullchecking): this implementation doesn't handle record primary constructors
+            // https://github.com/dotnet/roslyn/issues/58335: this implementation doesn't handle record primary constructors
             if (operation.SemanticModel!.GetDeclaredSymbol(operation.Syntax) is IMethodSymbol method)
             {
                 VisitNullChecks(operation, method.Parameters);
@@ -1502,7 +1502,7 @@ bool visitPossibleUsingDeclarationInLabel(ILabeledOperation labelOperation)
         {
             StartVisitingStatement(operation);
 
-            // PROTOTYPE(param-nullchecking): do we need to use SemanticModel here?
+            // https://github.com/dotnet/roslyn/issues/58335: do we need to use SemanticModel here?
             var member = operation.SemanticModel!.GetDeclaredSymbol(operation.Syntax);
             Debug.Assert(captureIdForResult is null);
             VisitNullChecks(operation, ((IMethodSymbol)member!).Parameters);
@@ -1518,7 +1518,7 @@ private void VisitNullChecks(IOperation operation, ImmutableArray<IParameterSymb
             {
                 if (param.IsNullChecked)
                 {
-                    // PROTOTYPE(param-nullchecking): do we need to use SemanticModel here?
+                    // https://github.com/dotnet/roslyn/issues/58335: do we need to use SemanticModel here?
                     var check = GenerateNullCheckForParameter(param, operation.Syntax, ((Operation)operation).OwningSemanticModel!);
                     _currentStatement = check;
                     VisitConditional(check, captureIdForResult: null);
@@ -1536,7 +1536,7 @@ private ConditionalOperation GenerateNullCheckForParameter(IParameterSymbol para
             IOperation conditionOp;
             if (ITypeSymbolHelpers.IsNullableType(parameter.Type))
             {
-                // PROTOTYPE(param-nullchecking): is there a better way to get the HasValue symbol here?
+                // https://github.com/dotnet/roslyn/issues/58335: is there a better way to get the HasValue symbol here?
                 // This way doesn't work with compilation.MakeMemberMissing for testing
                 var nullableHasValueProperty = parameter.Type.GetMembers(nameof(Nullable<int>.HasValue)).FirstOrDefault() as IPropertySymbol;
                 var nullableHasValueGet = nullableHasValueProperty?.GetMethod;