Remove PROTOTYPE comments and fix formatting warnings (#34475)

eng/Versions.props

@@ -69,7 +69,6 @@
     <MicrosoftMSXMLVersion>8.0.0.0-alpha</MicrosoftMSXMLVersion>
     <MicrosoftNetCompilersToolsetVersion>3.1.0-beta1-19164-01</MicrosoftNetCompilersToolsetVersion>
     <MicrosoftNetCoreAnalyzersVersion>$(RoslynDiagnosticsNugetPackageVersion)</MicrosoftNetCoreAnalyzersVersion>
-    <!-- PROTOTYPE(DefaultInterfaceImplementation): The MicrosoftNetCoreILAsmVersion was 2.0.0 -->
     <MicrosoftNetCoreILAsmVersion>3.0.0-preview4-27525-72</MicrosoftNetCoreILAsmVersion>
     <MicrosoftNetCoreILAsmVersion>3.0.0-preview4-27525-72</MicrosoftNetCoreILAsmVersion>
     <MicrosoftNetCoreILDasmVersion>3.0.0-preview4-27525-72</MicrosoftNetCoreILDasmVersion>

src/Compilers/CSharp/Portable/Binder/Binder_Expressions.cs

@@ -2928,7 +2928,7 @@ private BoundExpression BindArrayDimension(ExpressionSyntax dimension, Diagnosti
                     if (IsNegativeConstantForArraySize(size))
                     {
                         Error(diagnostics, ErrorCode.ERR_NegativeArraySize, dimension);
-						hasErrors = true;
+                        hasErrors = true;
                     }
                 }
 

src/Compilers/CSharp/Portable/Binder/Semantics/Operators/BinaryOperatorOverloadResolution.cs

@@ -156,7 +156,7 @@ public void BinaryOperatorOverloadResolution(BinaryOperatorKind kind, BoundExpre
                 result.Results.Clear();
                 GetAllBuiltInOperators(kind, left, right, result.Results, ref useSiteDiagnostics);
             }
-            
+
             // SPEC: The overload resolution rules of 7.5.3 are applied to the set of candidate operators to select the best 
             // SPEC: operator with respect to the argument list (x, y), and this operator becomes the result of the overload 
             // SPEC: resolution process. If overload resolution fails to select a single best operator, a binding-time 
@@ -165,9 +165,9 @@ public void BinaryOperatorOverloadResolution(BinaryOperatorKind kind, BoundExpre
             BinaryOperatorOverloadResolution(left, right, result, ref useSiteDiagnostics);
         }
 
-        private bool GetUserDefinedBinaryOperatorsFromInterfaces(BinaryOperatorKind kind, string name, 
+        private bool GetUserDefinedBinaryOperatorsFromInterfaces(BinaryOperatorKind kind, string name,
             TypeSymbol operatorSourceOpt, bool sourceIsInterface,
-            BoundExpression left, BoundExpression right, ref HashSet<DiagnosticInfo> useSiteDiagnostics, 
+            BoundExpression left, BoundExpression right, ref HashSet<DiagnosticInfo> useSiteDiagnostics,
             Dictionary<TypeSymbol, bool> lookedInInterfaces, ArrayBuilder<BinaryOperatorAnalysisResult> candidates)
         {
             Debug.Assert(candidates.Count == 0);

src/Compilers/CSharp/Portable/Binder/Semantics/Operators/UnaryOperatorOverloadResolution.cs

@@ -364,7 +364,7 @@ private bool GetUserDefinedOperators(UnaryOperatorKind kind, BoundExpression ope
             // SPEC: operators is the set provided by the direct base class of T0, or the effective
             // SPEC: base class of T0 if T0 is a type parameter.
 
-            // PROTOTYPE(DefaultInterfaceImplementation): The spec quote should be adjusted to cover operators from interfaces as well.
+            // https://github.com/dotnet/roslyn/issues/34451: The spec quote should be adjusted to cover operators from interfaces as well.
             // From https://github.com/dotnet/csharplang/blob/master/meetings/2017/LDM-2017-06-27.md:
             // - We only even look for operator implementations in interfaces if one of the operands has a type that is an interface or
             // a type parameter with a non-empty effective base interface list.

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

@@ -24,8 +24,8 @@ internal sealed class SourceFieldLikeEventSymbol : SourceEventSymbol
         private readonly SynthesizedFieldLikeEventAccessorSymbol _removeMethod;
 
         internal SourceFieldLikeEventSymbol(SourceMemberContainerTypeSymbol containingType, Binder binder, SyntaxTokenList modifiers, VariableDeclaratorSyntax declaratorSyntax, DiagnosticBag diagnostics)
-            : base(containingType, declaratorSyntax, modifiers, isFieldLike:true, interfaceSpecifierSyntaxOpt:null, 
-                   nameTokenSyntax:declaratorSyntax.Identifier, diagnostics:diagnostics)
+            : base(containingType, declaratorSyntax, modifiers, isFieldLike: true, interfaceSpecifierSyntaxOpt: null,
+                   nameTokenSyntax: declaratorSyntax.Identifier, diagnostics: diagnostics)
         {
             _name = declaratorSyntax.Identifier.ValueText;
 

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

@@ -305,7 +305,7 @@ internal SourceMemberFieldSymbolFromDeclarator(
                 {
                     Binder.CheckFeatureAvailability(declarator, MessageID.IDS_DefaultInterfaceImplementation, diagnostics, ErrorLocation);
                 }
-                else 
+                else
                 {
                     diagnostics.Add(ErrorCode.ERR_InterfacesCantContainFields, ErrorLocation);
                 }

src/Compilers/CSharp/Portable/Symbols/SymbolExtensions.cs

@@ -421,7 +421,7 @@ internal static void GetTypeOrReturnType(this Symbol symbol, out RefKind refKind
 
         internal static bool IsImplementableInterfaceMember(this Symbol symbol)
         {
-            return !symbol.IsStatic && (symbol.IsAbstract || symbol.IsVirtual) && (symbol.ContainingType?.IsInterface ?? false); 
+            return !symbol.IsStatic && (symbol.IsAbstract || symbol.IsVirtual) && (symbol.ContainingType?.IsInterface ?? false);
         }
     }
 }

src/Compilers/CSharp/Portable/Symbols/TypeSymbol.cs

@@ -886,8 +886,8 @@ private static Symbol ComputeImplementationForInterfaceMember(Symbol interfaceMe
                 }
 
                 // WORKAROUND: see comment on method.
-                // PROTOTYPE(DefaultInterfaceImplementation): Is this workaround still relevant? Do we need it when we are looking
-                //                                            for implementations in interfaces?
+                // https://github.com/dotnet/roslyn/issues/34452: Is this workaround still relevant? Do we need it when we are looking
+                //                                                for implementations in interfaces?
                 if (IsExplicitlyImplementedViaAccessors(interfaceMember, currType, out Symbol currTypeExplicitImpl))
                 {
                     // NOTE: may be null.
@@ -944,7 +944,7 @@ private static Symbol ComputeImplementationForInterfaceMember(Symbol interfaceMe
             // update the result (i.e. implicitImpl).
             if (interfaceMember.IsAccessor())
             {
-                // PROTOTYPE(DefaultInterfaceImplementation): Do we need to adjust behavior of this function in any way?
+                // https://github.com/dotnet/roslyn/issues/34453: Do we need to adjust behavior of this function in any way?
                 CheckForImplementationOfCorrespondingPropertyOrEvent((MethodSymbol)interfaceMember, implementingType, implementingTypeIsFromSomeCompilation, ref implicitImpl);
             }
 
@@ -1006,9 +1006,9 @@ private static Symbol FindMostSpecificImplementationInInterfaces(Symbol interfac
             else
             {
                 Debug.Assert(((object)conflict2 == null));
-                // PROTOTYPE(DefaultInterfaceImplementation): We might need to do extra consistency check for events/properties and their 
-                //                                            accessors. Need to verify if what CheckForImplementationOfCorrespondingPropertyOrEvent
-                //                                            is doing should be applicable here as well.
+                // https://github.com/dotnet/roslyn/issues/34453: We might need to do extra consistency check for events/properties and their 
+                //                                                accessors. Need to verify if what CheckForImplementationOfCorrespondingPropertyOrEvent
+                //                                                is doing should be applicable here as well.
             }
 
             return implicitImpl;

src/Compilers/CSharp/Test/Emit/CodeGen/CodeGenIncrementTests.cs

@@ -107,7 +107,7 @@ public void TestIncrementFloat()
             TestIncrementCompilationAndOutput<float>(0, 1);
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/32576")] //  PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/32576
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/32576")]
         [WorkItem(32576, "https://github.com/dotnet/roslyn/issues/32576")]
         public void TestIncrementDecimal()
         {

src/Compilers/CSharp/Test/Emit/CodeGen/CodeGenTests.cs

@@ -9817,7 +9817,7 @@ .locals init (int V_0, //intI
 ");
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/33564
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(538246, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/538246"), WorkItem(543655, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/543655")]
         public void FloatDoubleInfinity()
@@ -10673,7 +10673,7 @@ public static void Main()
         [WorkItem(529827, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/529827")]
         [WorkItem(568494, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/568494")]
         [WorkItem(32576, "https://github.com/dotnet/roslyn/issues/32576")]
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/32576")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/32576
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/32576")]
         public void DecimalLiteral_BreakingChange()
         {
             string source =
@@ -11339,7 +11339,7 @@ .maxstack  5
 }");
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/32576")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/32576
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/32576")]
         [WorkItem(32576, "https://github.com/dotnet/roslyn/issues/32576")]
         [WorkItem(34198, "https://github.com/dotnet/roslyn/issues/34198")]
         public void DecimalBinaryOp_03()
@@ -12906,7 +12906,7 @@ static void Main()
                 Diagnostic(ErrorCode.ERR_ConstOutOfRange, "(decimal)-4e30f").WithArguments("-4E+30", "decimal"));
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/33564
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")]
         public void Bug14064()
         {

src/Compilers/CSharp/Test/Emit/PDB/PDBConstantTests.cs

@@ -462,8 +462,8 @@ IEnumerable<int> M()
 </symbols>");
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/33564
-        // PROTOTYPE(DefaultInterfaceImplementation): Was [ConditionalFact(typeof(WindowsOnly), Reason = ConditionalSkipReason.NativePdbRequiresDesktop)]
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")]
+        // https://github.com/dotnet/roslyn/issues/33564: Was [ConditionalFact(typeof(WindowsOnly), Reason = ConditionalSkipReason.NativePdbRequiresDesktop)]
         [WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")]
         public void LocalConstantsTypes()
         {

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

@@ -895,7 +895,7 @@ static void M()
                 Diagnostic(ErrorCode.ERR_CheckedOverflow, "U64.Min - 2").WithLocation(89, 11));
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/33564
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(528727, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/528727")]
         public void TestConstantNumericConversionsNotOverflow()

src/Compilers/CSharp/Test/Symbol/Microsoft.CodeAnalysis.CSharp.Symbol.UnitTests.csproj

@@ -5,10 +5,7 @@
     <OutputType>Library</OutputType>
     <RootNamespace>Microsoft.CodeAnalysis.CSharp.Symbol.UnitTests</RootNamespace>
     <NoStdLib>true</NoStdLib>
-    <!-- PROTOTYPE(DefaultInterfaceImplementation): TargetFrameworks simply used to inherit $(RoslynPortableTargetFrameworks) -->
     <TargetFrameworks>net472;netcoreapp3.0</TargetFrameworks>
-    <!-- PROTOTYPE(DefaultInterfaceImplementation): Added RuntimeFrameworkVersion property below -->
-    <RuntimeFrameworkVersion Condition="'$(TargetFramework)' == 'netcoreapp3.0'">$(MicrosoftNETCoreRuntimeCoreCLRVersion)</RuntimeFrameworkVersion>
     <RoslynProjectType>UnitTestPortable</RoslynProjectType>
     <RuntimeIdentifiers>$(RoslynPortableRuntimeIdentifiers)</RuntimeIdentifiers>
     <AllowUnsafeBlocks>true</AllowUnsafeBlocks>

src/Compilers/CSharp/Test/Symbol/Symbols/DefaultInterfaceImplementationTests.cs

@@ -17332,10 +17332,10 @@ int I2.P6
                                                  targetFramework: TargetFramework.NetStandardLatest);
             Assert.True(compilation1.Assembly.RuntimeSupportsDefaultInterfaceImplementation);
 
-            // PROTOTYPE(DefaultInterfaceImplementation): The wording "accessor not found in interface member" is somewhat misleading
-            //                                            in this scenario. The accessor is there, but cannot be implemented. Prehaps
-            //                                            the message should be adjusted. Should also check diagnostics for an attempt
-            //                                            to implement other sealed members.
+            // https://github.com/dotnet/roslyn/issues/34455: The wording "accessor not found in interface member" is somewhat misleading
+            //                                                in this scenario. The accessor is there, but cannot be implemented. Prehaps
+            //                                                the message should be adjusted. Should also check diagnostics for an attempt
+            //                                                to implement other sealed members.
             compilation1.VerifyDiagnostics(
                 // (21,9): error CS0550: 'Test1.I1.P3.get' adds an accessor not found in interface member 'I1.P3'
                 //         get {throw null;} 

src/Compilers/CSharp/Test/Syntax/Syntax/SyntaxFactoryTests.cs

@@ -274,7 +274,7 @@ public void TestSeparatedListFactory_DefaultSeparators()
             Assert.Equal("x,y,z", list2.ToString());
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/33564
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(720708, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/720708")]
         public void TestLiteralDefaultStringValues()
@@ -346,7 +346,7 @@ public void TestLiteralDefaultStringValues()
             CheckLiteralToString(decimal.MaxValue, @"79228162514264337593543950335M");
         }
 
-        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")] // PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/33564
+        [ConditionalFact(typeof(WindowsDesktopOnly), Reason = "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")]
         [WorkItem(849836, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/849836")]
         public void TestLiteralToStringDifferentCulture()

src/Compilers/VisualBasic/Test/Emit/CodeGen/CodeGenConstLocal.vb

@@ -179,7 +179,7 @@ End Module
 
         End Sub
 
-        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")> ' PROTOTYPE(DefaultInterfaceImplementation) https://github.com/dotnet/roslyn/issues/33564
+        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")>
         Public Sub TestDoubleConstLocal()
             Dim verifier = CompileAndVerify(

src/Compilers/VisualBasic/Test/Emit/CodeGen/CodeGenForLoops.vb

@@ -326,7 +326,7 @@ End Class
 
         End Sub
 
-        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")> ' PROTOTYPE(DefaultInterfaceImplementation) https://github.com/dotnet/roslyn/issues/33564
+        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")>
         Public Sub ForLoopStepIsFloatNegativeVar()
             Dim TEMP = CompileAndVerify(

src/Compilers/VisualBasic/Test/Emit/CodeGen/CodeGenTests.vb

@@ -277,7 +277,7 @@ End Module
 
         <WorkItem(578074, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/578074")>
         <WorkItem(32576, "https://github.com/dotnet/roslyn/issues/32576")>
-        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/32576")> ' PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/32576
+        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/32576")>
         Public Sub PreserveZeroDigitsInDecimal()
             CompileAndVerify(
 <compilation>
@@ -625,7 +625,7 @@ expectedOutput:=<![CDATA[
         <WorkItem(568494, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/568494")>
         <WorkItem(568520, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/568520")>
         <WorkItem(32576, "https://github.com/dotnet/roslyn/issues/32576")>
-        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/32576")> ' PROTOTYPE(DefaultInterfaceImplementation): https://github.com/dotnet/roslyn/issues/32576
+        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/32576")>
         Public Sub DecimalLiteral_BreakingChange()
 
             CompileAndVerify(
@@ -5262,7 +5262,7 @@ End Class
 ]]>)
         End Sub
 
-        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")> ' PROTOTYPE(DefaultInterfaceImplementation) https://github.com/dotnet/roslyn/issues/33564
+        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(529849, "http://vstfdevdiv:8080/DevDiv2/DevDiv/_workitems/edit/529849")>
         Public Sub ArrayWithTypeCharsWithStaticLocals()
@@ -13821,7 +13821,7 @@ End Module
 ]]>)
         End Sub
 
-        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")> ' PROTOTYPE(DefaultInterfaceImplementation) https://github.com/dotnet/roslyn/issues/33564
+        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(7148, "https://github.com/dotnet/roslyn/issues/7148")>
         Public Sub Issue7148_1()
@@ -13867,7 +13867,7 @@ End Class
 ]]>)
         End Sub
 
-        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")> ' PROTOTYPE(DefaultInterfaceImplementation) https://github.com/dotnet/roslyn/issues/33564
+        <ConditionalFact(GetType(WindowsDesktopOnly), Reason:="https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(33564, "https://github.com/dotnet/roslyn/issues/33564")>
         <WorkItem(7148, "https://github.com/dotnet/roslyn/issues/7148")>
         Public Sub Issue7148_2()