src/Compilers/VisualBasic/Portable/Parser/Parser.vb (27 lines): - line 960: 'TODO - davidsch - In C++ code there is a call to GreedilyParseColonSeparatedStatements. Why? - line 1103: 'TODO Move all of this code to ParseIdentifier - line 1215: 'TODO - Move the check below ExecutableStatementContext.ProcessStatement - line 1259: 'TODO - Remove when select is fully implemented - line 1765: 'TODO - Consider moving the error to the Declaration context - line 1883: 'TODO - Compare this method with IsDeclarationStatement. - line 1966: 'TODO - davidsch - The error checking here looks like parser doing semantics. The grammar allows - line 2187: 'TODO - For better error recovery consider adding a resync here for - line 2326: ' TODO - Consider improving the handling of implicit line continuations. - line 2514: 'TODO - davidsch - This used to call ParseInitializer which checked for DotToken before calling ParseAssignmentInitializer - line 2611: 'TODO - davidsch - Dev10 error is on .Name? - line 2662: ' TODO - In Dev 10 error is on the xmlName - line 3083: 'TODO - - line 3507: 'TODO - davidsch - line 3865: 'TODO - davidsch Can ParseFunctionOrDelegateDeclaration and - line 3879: ' TODO - davidsch - Why do ParseFunctionDeclaration and ParseSubDeclaration have - line 3959: 'TODO - davidsch - line 3995: 'TODO - davidsch - What should be created here? For now use + as a canonical operator - line 4022: 'TODO - davidsch - Why does operator resync here with different condition than Sub and Function? Seems like these should be consistent. - line 4184: ' TODO - GreenSepList knows its error count. Expose it instead of recomputing it. - line 4286: ' TODO - Which keyword SUB or FUNCTION? - line 4288: ' TODO - Consider adding this as another case of VerifyExpectedToken - line 4546: ' TODO - Bug 889301 - Dev10 does a resync here when there is an error. That prevents ERRID_InvalidParameterSyntax below from - line 4615: 'TODO - Move these checks to Binder_Utils.DecodeParameterModifiers - line 4713: ' TODO - Move these errors (ERRID.ERR_DefaultValueForNonOptionalParamout, ERRID.ERR_ObsoleteOptionalWithoutValue) of the parser. - line 5503: ' TODO: this function is so complex (n^2 loop?) it times out in CC verifier. - line 6212: 'TODO - These should be removed. Checks should be in binding. src/Compilers/VisualBasic/Portable/Syntax/Syntax.xml (24 lines): - line 57: General TODO: - line 67: - line 351: - line 415: - line 1215: TODO: Describe how these map! - line 2349: - line 2389: - line 2422: - line 3771: TODO: Describe how these map! - line 4089: TODO: Describe how these map! - line 4285: TODO: Describe how these map! - line 4312: TODO: Describe how these map! - line 4938: - line 5460: - line 5504: - line 5539: - line 5563: - line 5570: - line 7338: - line 9109: - line 9128: - line 9193: - line 9208: src/EditorFeatures/CSharpTest/TodoComment/TodoCommentTests.cs (23 lines): - line 21: var code = @"// [|TODO:test|]"; - line 29: var code = @"// [|TODO test|]"; - line 53: var code = @"// ""TODO test"""; - line 61: var code = @"// Hello TODO test"; - line 69: var code = @"/// [|TODO test|]"; - line 77: var code = @"#if DEBUG // [|TODO test|]"; - line 85: var code = @"#if DEBUG /// [|TODO test|]"; - line 93: var code = @"#region // TODO test"; - line 101: var code = @"#endregion // [|TODO test|]"; - line 109: var code = @"// [|TODO test |]"; - line 117: var code = @"/* [|TODO: hello |]*/"; - line 125: var code = @"/** [|TODO: hello |]*/"; - line 134: /* [|TODO: hello |] - line 135: [|TODO: hello |] - line 136: [|TODO: hello |] - line 137: * [|TODO: hello |] - line 138: [|TODO: hello |]*/"; - line 147: /** [|TODO: hello |] - line 148: [|TODO: hello |] - line 149: [|TODO: hello |] - line 150: * [|TODO: hello |] - line 151: [|TODO: hello |]*/"; - line 161: /// [|TODO : test |] src/Compilers/VisualBasic/Portable/Parser/ParseStatement.vb (17 lines): - line 80: ' TODO - Which keyword in this case? - line 225: 'TODO - What kind of statement should be generated? Bad Exit would make most sense. - line 502: ' TODO - Is an END IF with missing tokens the right way to model this? - line 508: ' TODO - Is an END IF with missing tokens the right way to model this? - line 617: 'TODO - In Dev 10 only syntax error was reported. Now - line 640: ' TODO - Handle case where controlVariable is OK but current token is not equals. Need to resync in that case too - line 659: ' TODO - consider removing redundant syntax error message. - line 669: 'TODO - davidsch - Why is newline allowed after '=' but not after 'to'? - line 679: 'TODO - ERRID_ExpectedExpression here? - line 778: 'TODO - davidsch - I have kept this code as-is but it seems that it would be better to parse the common prefix of - line 925: 'TODO - Not calling ResyncAt may cause different errors from Dev10. - line 1127: 'TODO - Are there other built-in types that should not through this path?, i.e - line 1375: 'TODO - Rename ParseKeywordExpression and share with other statements that follow this pattern - line 1410: 'TODO - rename ParseObsoleteAssignment - line 1612: 'TODO - Does the Mid need to be modeled as a contextual keyword? Right now it is kept as an identifier. - line 1768: 'TODO - See if all of the simple statements that take only one keyword can be merged into one parsing function - line 1807: 'TODO - not resyncing here may cause errors to differ from Dev10. src/Compilers/Core/Portable/MetadataReader/PEModule.cs (15 lines): - line 618: // TODO: Consider if we should cache the result (not the IEnumerable, but the actual values). - line 903: // TODO (tomat) report error: - line 913: // TODO (tomat) report error: - line 919: // TODO (tomat) report error: - line 1382: // TODO: error checking offset in range - line 2303: // TODO - Support nested types. - line 2316: // TODO - Support nested types. - line 2384: // TODO: Check validity of the typeSpec handle. - line 2387: // TODO: error checking offset in range - line 2532: // TODO (tomat): report an error? - line 2709: // TODO: Error checking: Throw an error if the table entry cannot be found - line 2726: // TODO: Error checking: Throw an error if the table entry cannot be found - line 2769: // TODO (tomat): report error: - line 2791: // TODO (tomat): report error: - line 3067: // TODO: remove, API should be provided by MetadataReader src/Compilers/VisualBasic/Portable/Semantics/TypeInference/TypeArgumentInference.vb (14 lines): - line 133: ' TODO: Dev10 has two locations to track type inferred so far. - line 203: ' TODO: Dev10 has two locations to track type inferred so far. - line 387: ' TODO: Should we simply exit the loop for RELEASE build? - line 522: ' TODO: Not sure if this is applicable to Roslyn, need to try this out when all required features are available. - line 529: ' TODO: We are setting inference level before - line 887: ' TODO: Should we use InferredType or CandidateInferredType here? It looks like Dev10 is using the latter, - line 1226: ' TODO: Consider using location for the type declaration. - line 1313: ' TODO: It looks like this check can give us a false positive. For example, - line 1535: ' TODO: Would it make sense to continue through other type arguments even if inference failed for - line 1898: ' TODO: Do we need to continue even if we already have a matching base class? - line 2058: ' TODO: Doesn't this make the inference algorithm order dependent? For example, if we were to - line 2108: ' TODO: Why does it make sense to continue here? It looks like we can infer something from - line 2321: ' TODO: Should we exit the loop? For some reason Dev10 keeps going. - line 2338: ' TODO: Should we exit the loop? For some reason Dev10 keeps going. src/Compilers/VisualBasic/Portable/Binding/Binder_Expressions.vb (14 lines): - line 1184: ' TODO Modules?? - line 1458: 'TODO: We need to do other expression reclassifications here. - line 1790: ' TODO: Check base expressions for properties if digThroughProperty==true. - line 2073: ' TODO: Address array initializers, they might need to change type. - line 2166: ' TODO: Is this reachable? Check and add tests - line 2268: ' TODO: Find a test case for ERRID_UseOfKeywordOutsideClass1 - line 2271: ' TODO: Check for closures - line 2288: ' TODO: Find a test case for ERRID_UseOfKeywordOutsideClass1 - line 2580: ' TODO (tomat): host object binding - line 3109: 'TODO: Deal with errors reported by BindTypeArguments. Should we adjust hasError? - line 3170: ' TODO: Check if this is a constant field with missing or bad value and report an error. - line 3583: 'TODO: What should we do if count is 0? Can we get in a situation like this? - line 3592: ' TODO: Should we set HasError flag if any of the BindTypeSyntax calls report errors? - line 4371: ' TODO - What expression should be generated in this case? Note, the parser already generates a syntax error. src/Compilers/Core/Portable/PEWriter/MetadataWriter.cs (14 lines): - line 982: // TODO: error - line 986: // TODO: special treatment for global fields and methods. Object model support would be nice. - line 1721: // TODO: we can precalculate the exact size of IL stream - line 1753: // TODO (https://github.com/dotnet/roslyn/issues/3905): - line 1852: // TODO (https://github.com/dotnet/roslyn/issues/3905): - line 2003: // TODO: attributes on member reference entries 6 - line 2009: // TODO: declarative security entries 8 - line 2013: // TODO: standalone signature entries 11 - line 2015: // TODO: type spec entries 13 - line 2017: // TODO: this.AddCustomAttributesToTable(assembly.Files, 16); - line 2018: // TODO: exported types 17 - line 2019: // TODO: this.AddCustomAttributesToTable(assembly.Resources, 18); - line 2905: // TODO: consider parallelizing these (local signature tokens can be piped into IL serialization & debug info generation) - line 3662: // TODO: xml for older platforms src/EditorFeatures/VisualBasicTest/TodoComment/TodoCommentTests.vb (13 lines): - line 18: Dim code = ' [|TODO:test|] - line 25: Dim code = ' [|TODO test|] - line 46: Dim code = ' "TODO test" - line 53: Dim code = ' Hello TODO test - line 60: Dim code = ''' [|TODO test|] - line 67: Dim code = #If DEBUG Then ' [|TODO test|] - line 74: Dim code = #If DEBUG Then ''' [|TODO test|] - line 81: Dim code = #Region ' [|TODO test |] - line 88: Dim code = #End Region ' [|TODO test |] - line 95: Dim code = ' [|TODO test |] - line 102: Dim code = REM [|TODO test |] - line 109: Dim code = #If Debug Then REM [|TODO test |] - line 118: ''' [|TODO : test |] src/Compilers/CSharp/Portable/Compiler/ClsComplianceChecker.cs (11 lines): - line 514: // TODO: it would be nice to report this on the base type clause. - line 525: // TODO: it would be nice to report this on the base type clause. - line 553: // TODO: it would be nice to report this on the constraint clause. - line 597: // TODO: it would be nice to report for each bad argument, but currently it's pointless since they - line 613: // TODO: it would be nice to report for each bad argument, but currently it's pointless since they - line 631: // TODO: it would be nice to report for each bad argument, but currently it's pointless since they - line 885: // TODO: Shouldn't we somehow reference the conflicting member? Dev11 doesn't. - line 910: // TODO: Shouldn't we somehow reference the conflicting member? Dev11 doesn't. - line 957: // TODO: There's no public equivalent of Symbol.DeclaringCompilation. - line 965: // TODO: it would be nice to be more precise than this: we only want to - line 1053: /// TODO: This should really have a separate error code - it's logically separate and requires explanation. src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_BinaryOperator.cs (11 lines): - line 70: // TODO: We generate every one of these as "temp = x; T.false(temp) ? temp : T.&(temp, y)" even - line 71: // TODO: when x has no side effects. We can optimize away the temporary if there are no side effects. - line 1054: // TODO: (This TODO applies throughout this file, not just to this method.) - line 1055: // TODO: We might be storing a constant to this temporary that we could simply inline. - line 1056: // TODO: (There are other expressions that can be safely moved around other than constants - line 1057: // TODO: as well -- for example a boxing conversion of a constant int to object.) - line 1058: // TODO: Build a better temporary-storage management system that decides whether or not - line 1059: // TODO: to store a temporary. - line 1234: // TODO: There are expressions other than constants that have no side effects - line 1235: // TODO: or elidable side effects. - line 1800: // TODO: If the expression has no side effects then it can be optimized away here as well. src/Compilers/CSharp/Portable/Parser/Lexer.cs (11 lines): - line 1229: // TODO: See below, it may be desirable to mark this token - line 1242: // TODO: See below, it may be desirable to mark this token - line 1274: // TODO: See below, it may be desirable to mark this token - line 1305: // TODO: Change to Int64.TryParse when it supports NumberStyles.AllowBinarySpecifier (inline this method into GetValueUInt32/64) - line 3144: // TODO: Take the < by itself, I guess? - line 3544: // TODO: Could put a big switch here for common cases - line 3563: // TODO: which is the right one? - line 3574: // TODO: which is the right one? - line 3579: // TODO: There is a lot of duplication between attribute text, CDATA text, and comment text. - line 3745: //TODO: Dev11 allows C# comments and newlines in cref trivia (DevDiv #530523). - line 3801: //TODO: Dev11 allows C# comments and newlines in cref trivia (DevDiv #530523). src/Compilers/VisualBasic/Portable/Lowering/SyntheticBoundNodeFactory.vb (10 lines): - line 209: ' TODO: unroll loop and add diagnostics for failure - line 210: ' TODO: should we use GetBaseProperty() to ensure we generate a call to the overridden method? - line 211: ' TODO: replace this with a mechanism that uses WellKnownMember instead of string names. - line 218: ' TODO: unroll loop and add diagnostics for failure - line 408: ' TODO: use WellKnownMembers instead of strings. - line 418: ' TODO: use WellKnownMembers instead of strings. - line 419: ' TODO: share code with SynthesizedImplementation(...) - line 485: ' TODO: add diagnostics for when things fall apart - line 558: ' TODO: (EXPRTREE) These error codes are probably not correct. Fix them. - line 826: ' TODO: add diagnostics for when things fall apart src/Workspaces/Core/Portable/Execution/AbstractReferenceSerializationService.cs (9 lines): - line 110: // TODO: think a way to support this or a way to deal with this kind of situation. - line 172: // TODO: remove this kind of host specific knowledge from common layer. - line 198: // TODO: think a way to support this or a way to deal with this kind of situation. - line 333: // TODO: what I should do with documentation provider? it is not exposed outside - line 348: // TODO: deal with xml document provider properly - line 582: // TODO: make reader be able to read byte[] chunk - line 600: // TODO: any way to avoid allocating byte array here? - line 684: // TODO: doc comment provider is a bit wierd. - line 690: // TODO: properly implement this src/EditorFeatures/VisualBasicTest/EditAndContinue/RudeEditTopLevelTests.vb (9 lines): - line 3180: ' TODO bug https://github.com/dotnet/roslyn/issues/2504 - line 3396: ' TODO: We could check that the types and order of b and c haven't changed. - line 3414: ' TODO: We could check that the types and order of b and c haven't changed. - line 3432: ' TODO: We could check that the types and order of b and c haven't changed. - line 3448: ' TODO: We could check that the types and order of b and c haven't changed. - line 3464: ' TODO: We could check that the types and order of b and c haven't changed. - line 3554: ' TODO: the type didn't really change, we can allow this - line 3963: ' TODO (bug https//github.com/dotnet/roslyn/issues/2504): - line 4409: ' TODO: we could detect that the type haven't changed and allow this src/Compilers/VisualBasic/Portable/CodeGen/EmitConversion.vb (9 lines): - line 148: 'TODO: used - line 213: 'TODO: used - line 284: 'TODO: Dev10 CodeGenerator::GenerateDirectCast does some optimization for - line 325: 'TODO: is this needed for widening conversions? - line 376: ' TODO: Do we still need this checks or CLR is smarter now? - line 433: 'TODO: compute this (note: returning true is safe - false would just enable optimizations) - line 438: 'TODO: compute this (note: returning true is safe - false would just enable optimizations) - line 443: 'TODO: compute this (note: returning true is safe - false would just enable optimizations) - line 456: 'TODO: used src/Features/Core/Portable/EditAndContinue/AbstractEditAndContinueAnalyzer.cs (8 lines): - line 340: // TODO: newTree.HasErrors? - line 897: // TODO: if the member isn't a field/property we should return empty span. - line 907: // TODO: VB field multi-initializers break this - line 1595: // TODO (tomat): use BinarySearch - line 2198: // TODO: scripting - line 2593: // TODO: the compiler should expose TypeLayout property on INamedTypeSymbol - line 2774: // TODO (bug https://github.com/dotnet/roslyn/issues/2504) - line 2883: // TODO (bug https://github.com/dotnet/roslyn/issues/2504) src/Compilers/VisualBasic/Portable/Scanner/ScannerXml.vb (8 lines): - line 101: ' TODO: this will not consume trailing trivia - line 328: ' TODO: the entity could be whitespace, do we want to report it as WS? - line 394: 'TODO: error recovery - how can we do this? - line 426: 'TODO: error recovery. We cannot do this. - line 522: ' TODO: we cannot do the following - line 754: ' TODO: review - line 876: ' TODO: this trivia is used only in XmlDoc. May split the function? - line 1066: 'TODO - Fix ScanXmlNCName to conform to XML spec instead of old loose scanning. src/Compilers/VisualBasic/Portable/Scanner/Scanner.vb (8 lines): - line 255: ' TODO: this could be vastly simplified if we could ignore line continuations. - line 350: ' TODO: should this be a Require? - line 1239: ' TODO: do we allow widechars in keywords? - line 1248: ' TODO: do we allow widechars in keywords? - line 1257: ' TODO: do we allow widechars in keywords? - line 1495: ' TODO: make sure we get good perf here - line 1629: ' TODO: this should be provable? - line 1632: ' TODO: consider interning. src/Compilers/VisualBasic/Portable/Binding/Binder_Statements.vb (8 lines): - line 1704: 'TODO: override in MethodBodySemanticModel similarly to BindVariableDeclaration. - line 2392: ' TODO: Should try to report ERR_NameNotEvent2 without making notParenthesized an RValue, - line 3139: ' TODO: update the types to the ones that get really used by the rewriter. In case of using an index of an user - line 3744: ' TODO: consider special casing strings and one dimensional arrays in the semantic info - line 4504: ' TODO: Dev10 shows the error on the symbol name, not the whole declaration. See bug 10720 - line 4506: ' TODO: we could suppress this warning if the type does not implement IDisposable and is not late bound. - line 4523: ' TODO: add late binding, see statementsemantics.cpp lines 6765++ - line 4574: ' TODO: if this constraint is a type parameter as well, dig into its constraints as well. src/Features/CSharp/Portable/EditAndContinue/CSharpEditAndContinueAnalyzer.cs (8 lines): - line 170: // TODO: The logic is similar to BreakpointSpans.TryCreateSpanForVariableDeclaration. Can we abstract it? - line 574: // TODO: Consider mapping an expression body to an equivalent statement expression or return statement and vice versa. - line 1162: // TODO: - line 1833: // TODO: - line 1896: // TODO: - line 2139: // TODO: - line 2694: // TODO: We currently don't support switching from auto-props to properties with accessors and vice versa. - line 3082: // TODO: changes around suspension points (foreach, lock, using, etc.) src/Compilers/VisualBasic/Portable/Parser/BlockContexts/BlockContext.vb (7 lines): - line 155: 'TODO - Remove dependency on Parser - line 403: ' TODO: this will add an error to the statement. Perhaps duplicating it - line 473: 'TODO - davidsch - line 479: 'TODO - davidsch - line 631: 'TODO - davidsch make these expected error message names consistent. Some are EndXXExpected and others are ExpectedEndXX - line 652: 'TODO rename this enum for consistency ERRID_MissingEndProperty - line 665: 'TODO rename this enum for consistency ERRID_MissingEndProperty src/Compilers/VisualBasic/Portable/Binding/Binder_SelectCase.vb (7 lines): - line 598: ' TODO: RecommendSwitchTable for Range clause. - line 599: ' TODO: If we decide to implement it we will need to - line 600: ' TODO: add heuristic to determine when the range is - line 601: ' TODO: big enough to prefer IF lists over SWITCH table. - line 602: ' TODO: We will also need to add logic in the emitter - line 603: ' TODO: to iterate through ConstantValues in a range. - line 647: ' TODO: beneficial over IF lists (as per a threshold on size ratio) src/Compilers/Core/Portable/SynthesizedLocalKind.cs (7 lines): - line 110: // VB TODO: - line 112: // VB TODO: - line 114: // VB TODO: - line 116: // VB TODO: - line 137: StateMachineReturnValue = AsyncMethodReturnValue, // TODO VB: why do we need this in iterators? - line 181: // VB TODO: XmlInExpressionLambda locals are always lifted and must have distinct names. - line 229: /// TODO: Avoid using lambdas and display classes for implementation of relaxation stubs and remove this kind. src/EditorFeatures/Core/Implementation/CodeFixes/CodeFixService.cs (7 lines): - line 202: // TODO (https://github.com/dotnet/roslyn/issues/4932): Don't restrict CodeFixes in Interactive - line 235: // TODO (https://github.com/dotnet/roslyn/issues/4932): Don't restrict CodeFixes in Interactive - line 281: // TODO: Can we share code between similar lambdas that we pass to this API in BatchFixAllProvider.cs, CodeFixService.cs and CodeRefactoringService.cs? - line 434: // TODO (https://github.com/dotnet/roslyn/issues/4932): Don't restrict CodeFixes in Interactive - line 471: // TODO: Can we share code between similar lambdas that we pass to this API in BatchFixAllProvider.cs, CodeFixService.cs and CodeRefactoringService.cs? - line 518: // TODO (https://github.com/dotnet/roslyn/issues/4932): Don't restrict CodeFixes in Interactive - line 649: // TODO (https://github.com/dotnet/roslyn/issues/4932): Don't restrict CodeFixes in Interactive src/Compilers/VisualBasic/Portable/CodeGen/EmitStatement.vb (7 lines): - line 293: ' TODO: this local can be released when we can release named locals. - line 313: ' TODO: parts of the following code is common with AssignmentExpression - line 820: 'TODO: is this to fold value? Same in C#? - line 1012: ' TODO: For now we use IF lists if we encounter - line 1013: ' TODO: BoundRangeCaseClause, we should not reach here. - line 1240: 'TODO: can we free any locals here? Perhaps nameless temps? - line 1363: 'TODO: releasing locals with name NYI. src/Compilers/CSharp/Portable/Binder/Binder_Expressions.cs (7 lines): - line 1954: // TODO: report more specific diagnostics here for failed method group conversions - line 3886: // TODO: If/when we have a way to version warnings, we should add a warning for this. - line 3908: // TODO: distinct error code for collection initializers? (Dev11 doesn't have one.) - line 3997: // TODO: distinct error code for collection initializers? (Dev11 doesn't have one.) - line 5574: // TODO: we should check if such a using directive would actually help, or if there is already one in scope. - line 6678: // TODO: We're creating an extra copy of the properties array in BindIndexerOrIndexedProperty - line 7170: // TODO: need a special ERR for this. src/Compilers/VisualBasic/Portable/CodeGen/EmitExpression.vb (7 lines): - line 465: ' TODO: check the ctor signature, and recover more gracefully from failure to - line 618: 'TODO: For static field access this may require ..ctor to run. Is this a side-effect? - line 655: 'TODO: this may require ..ctor to run. Is this a side-effect? - line 662: 'TODO: access to a field on this/base has no side-effects. - line 915: 'TODO: there must be more non-null cases. - line 972: 'TODO: it is ok to use Call with final methods. - line 1576: ' TODO: revise? src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.cs.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: Uvolněte nespravované prostředky (nespravované objekty) a přepište metodu Finalize() níže. - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: Přepište metodu Finalize(), jenom pokud metoda Dispose(disposing As Boolean) výše obsahuje kód pro uvolnění nespravovaných prostředků. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: Zrušte komentář následujícího řádku, pokud se výše přepisuje metoda Finalize(). src/Compilers/VisualBasic/Portable/Parser/ParseXml.vb (6 lines): - line 748: 'TODO - Consider whether the current element should be closed or create a missing start tag to match this dangling end tag. - line 787: 'TODO - Don't add an error if the unexpectedSyntax already has errors. - line 1489: ' TODO: The following is simplified, need to be revised - line 1832: 'TODO - handle whitespace between begin and name. This check is also needed for xml document declaration - line 1837: 'TODO - name has to allow :. Dev10 puts a fully qualified name here. - line 1989: ' TODO: is this ok? src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.tr.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: yönetilmeyen kaynakları (yönetilmeyen nesneleri) serbest bırakın ve aşağıda Finalize() ifadesini geçersiz kılın. - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: Finalize() ifadesini yalnızca yukarıdaki Dispose(disposing As Boolean) yönetilmeyen kaynakları serbest bırakacak koda sahipse geçersiz kılın. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: Finalize() yukarıda geçersiz kılınırsa aşağıdaki satırın açıklamasını kaldırın. src/Compilers/VisualBasic/Portable/Parser/ParseExpression.vb (6 lines): - line 169: ' TODO (DevDiv workitem 966425): Replace exception name test with a type test once the type - line 828: ' TODO: having "0 To 10" style arguments would also point to array init - line 851: ' TODO: this should be optional - line 887: ' TODO: this should actually go on the next token ( "With" ) - line 901: ' TODO: this should actually go on the next token ( "With" ) - line 1740: 'TODO - Review all errors in Dev10 ParseStatementLambda src/Features/VisualBasic/Portable/EditAndContinue/StatementSyntaxComparer.vb (6 lines): - line 104: ' TODO: avoid allocation of closure - line 112: ' TODO: avoid allocation of closure - line 223: ' TODO: AwaitExpression - line 636: ' TODO: Parameters are not mapped? - line 715: ' TODO: - line 718: ' TODO: src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.zh-Hant.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: 釋放 Unmanaged 資源 (Unmanaged 物件) 並覆寫下方的 Finalize()。 - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: 只有當上方的 Dispose(disposing As Boolean) 具有要釋放 Unmanaged 資源的程式碼時,才覆寫 Finalize()。 - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: 覆寫上列 Finalize() 時,取消下行的註解狀態。 src/Compilers/VisualBasic/Portable/Symbols/Source/SourceMethodSymbol.vb (6 lines): - line 32: ' TODO (tomat): should be private - line 36: ' TODO (tomat): should be private - line 616: ' TODO: Associated property/event not implemented. - line 641: ' TODO (tomat): NotOverridable? - line 700: ' TODO (tomat) : NotOverridable? - line 2143: ' TODO: The check for Locations is to rule out cases such as implicit parameters src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.pt-BR.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: liberar recursos não gerenciados (objetos não gerenciados) e substituir um Finalize() abaixo. - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: substituir Finalize() somente se Dispose(disposing As Boolean) acima tiver o código para liberar recursos não gerenciados. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: remover marca de comentário da linha a seguir se Finalize() for substituído acima. src/EditorFeatures/CSharpTest/SignatureHelp/AttributeSignatureHelpProviderTests.cs (6 lines): - line 354: // TODO: Bug 12319: Enable tests for script when this is fixed. - line 485: // TODO: Bug 12319: Enable tests for script when this is fixed. - line 512: // TODO: Bug 12319: Enable tests for script when this is fixed. - line 539: // TODO: Bug 12319: Enable tests for script when this is fixed. - line 564: // TODO: Bug 12319: Enable tests for script when this is fixed. - line 589: // TODO: Bug 12319: Enable tests for script when this is fixed. src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.zh-Hans.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: 释放未托管资源(未托管对象)并在以下内容中替代 Finalize()。 - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: 仅当以上 Dispose(disposing As Boolean)拥有用于释放未托管资源的代码时才替代 Finalize()。 - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: 如果在以上内容中替代了 Finalize(),则取消注释以下行。 src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.ko.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: 관리되지 않는 리소스(관리되지 않는 개체)를 해제하고 아래의 Finalize()를 재정의합니다. - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: 위의 Dispose(disposing As Boolean)에 관리되지 않는 리소스를 해제하기 위한 코드가 있는 경우에만 Finalize()를 재정의합니다. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: 위의 Finalize()가 재정의된 경우 다음 줄의 주석 처리를 제거합니다. src/VisualStudio/Core/Def/Implementation/ProjectSystem/VisualStudioWorkspaceImpl.cs (6 lines): - line 847: // TODO: do we need save argument for CloseDocument? - line 874: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 - line 878: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 - line 946: // TODO: This is a very roundabout way to update the context - line 1106: // TODO: meditate about the flags we can pass to this and decide what is most appropriate for Roslyn - line 1246: // TODO: Move this out to DocumentProvider. As is, this depends on being able to src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.cs.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: Uvolněte nespravované prostředky (nespravované objekty) a přepište finalizační metodu níže. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: Přepište finalizační metodu, jenom pokud metoda Dispose(bool disposing) výše obsahuje kód pro uvolnění nespravovaných prostředků. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: Zrušte komentář následujícího řádku, pokud se výše přepisuje finalizační metoda. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.tr.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: yönetilmeyen kaynakları (yönetilmeyen nesneleri) serbest bırakın ve aşağıda bir sonlandırıcıyı geçersiz kılın. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: yalnızca yukarıdaki Dispose(bool disposing) ifadesi yönetilmeyen kaynakları serbest bırakacak koda sahipse sonlandırıcıyı geçersiz kılın. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: sonlandırıcı yukarıda geçersiz kılınırsa aşağıdaki satırın açıklamasını kaldırın. src/Interactive/HostTest/InteractiveHostTests.cs (6 lines): - line 683: // TODO: We should report an error that assembly named 'a' was already loaded with different content. - line 715: // TODO: We should report an error that assembly named 'c' was already loaded with different content. - line 749: // TODO: We should report an error that assembly named 'c' was already loaded with different content. - line 752: Assert.Equal("TODO: error", ReadErrorOutputToEnd().Trim()); - line 756: //// TODO (987032): - line 1205: // TODO (https://github.com/dotnet/roslyn/issues/7976): delete this src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.fr.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: libérer les ressources non managées (objets non managés) et remplacer Finalize() ci-dessous. - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: remplacer Finalize() seulement si la fonction Dispose(disposing As Boolean) ci-dessus a du code pour libérer les ressources non managées. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: supprimer les marques de commentaire pour la ligne suivante si Finalize() est remplacé ci-dessus. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.de.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: nicht verwaltete Ressourcen (nicht verwaltete Objekte) freigeben und Finalizer weiter unten überschreiben. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: Finalizer nur überschreiben, wenn Dispose(bool disposing) weiter oben Code für die Freigabe nicht verwalteter Ressourcen enthält. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: Auskommentierung der folgenden Zeile aufheben, wenn der Finalizer weiter oben überschrieben wird. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.ru.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: освободить неуправляемые ресурсы (неуправляемые объекты) и переопределить ниже метод завершения. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: переопределить метод завершения, только если Dispose(bool disposing) выше включает код для освобождения неуправляемых ресурсов. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: раскомментировать следующую строку, если метод завершения переопределен выше. src/Compilers/VisualBasic/Portable/Symbols/Source/SourceMemberContainerTypeSymbol.vb (6 lines): - line 1787: ' TODO: revise - line 2604: ' TODO (tomat): should be added to the initializers - line 2788: ' TODO: does it need to be deterministic? - line 2797: 'TODO: anything to do here? - line 3531: ' TODO: this is O(N^2), maybe this can be improved. - line 3821: ' TODO: maybe rewrite these diagnostics to if/elseifs to report just one diagnostic per src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.ja.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: アンマネージ リソース (アンマネージ オブジェクト) を解放し、下のファイナライザーをオーバーライドします。 - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: 上の Dispose(bool disposing) にアンマネージ リソースを解放するコードが含まれる場合にのみ、ファイナライザーをオーバーライドします。 - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: 上のファイナライザーがオーバーライドされる場合は、次の行のコメントを解除してください。 src/Compilers/VisualBasic/Portable/Semantics/Conversions.vb (6 lines): - line 417: ' TODO: It looks like Dev10 MethodConversionKinds for return are badly named because - line 1399: ' TODO: May need to do classification in a special way when ExprIsOperandOfConditionalBranch==True, - line 2299: 'TODO: From an anonymous delegate type generated for a lambda method reclassification to any delegate type with an identical signature. - line 3228: 'TODO: - line 4012: ''' TODO: It looks like Dev10 MethodConversionKinds for return are badly named because - line 4186: ' TODO: Take custom modifiers into account, if needed. src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_LateBindingHelpers.vb (6 lines): - line 294: 'TODO: Dev11 does GetobjectValue here is it needed? Should array initialization do it in general? - line 374: 'TODO: should we suppress object clone here? Dev11 does not. - line 445: ' TODO: should we suppress object clone here? Dev11 does not suppress. - line 517: ' TODO: May need a special case for parameters. - line 596: ' TODO: May need a special case for parameters. - line 916: ' TODO: src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.pl.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: Zwolnić niezarządzane zasoby (niezarządzane obiekty) i przesłonić poniższy finalizator. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: Przesłonić finalizator tylko w sytuacji, gdy powyższa metoda Dispose(bool disposing) zawiera kod służący do zwalniania niezarządzanych zasobów. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: Usunąć komentarz z poniższego wiersza, jeśli finalizator został przesłonięty powyżej. src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.it.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: liberare risorse non gestite (oggetti non gestiti) ed eseguire sotto l'override di Finalize(). - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: eseguire l'override di Finalize() solo se Dispose(disposing As Boolean) include il codice per liberare risorse non gestite. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: rimuovere il commento dalla riga seguente se è stato eseguito l'override di Finalize(). src/VisualStudio/Core/Def/Implementation/EditAndContinue/VsENCRebuildableProjectImpl.cs (6 lines): - line 172: // TODO: Remove once https://github.com/dotnet/roslyn/issues/16657 is addressed - line 484: // TODO: This is a workaround for a debugger bug in which not all projects exit the break state. - line 545: // TODO: This is a workaround for a debugger bug. - line 623: // TODO (tomat): - line 851: // TODO (https://github.com/dotnet/roslyn/issues/1204): this check should be unnecessary. - line 1297: // TODO: CustomDebugInfoReader should throw InvalidDataException src/EditorFeatures/VisualBasicTest/EditAndContinue/RudeEditStatementTests.vb (6 lines): - line 43: ' TODO: consider adding location distance between nodes that match into consideration when - line 760: ' TODO: test GroupBy with known matches across CRVs (coming from active statement tracking) - line 2420: ' TODO allow creating a new leaf closure - line 3411: ' TODO: better location - line 4426: ' TODO "Function(a) x + x0" is matched with "Function(a) y1 + x0", hence we report more errors. - line 5603: ' TODO (bug 1212) should report no error src/Compilers/VisualBasic/Portable/Semantics/OverloadResolution.vb (6 lines): - line 240: ' TODO: Consider building this bitmap lazily, on demand. - line 1075: 'TODO: Where does this fit? - line 1674: ''' TODO: Update indexes of other overload method resolution rules - line 2131: ' TODO: Should probably handle GroupTypeInferenceLambda too. - line 2292: ' TODO: Should probably handle GroupTypeInferenceLambda too. - line 3289: ' TODO: Do we need to do more thorough check for error types here, i.e. dig into generics, src/Compilers/VisualBasic/Portable/Compilation/MethodCompiler.vb (6 lines): - line 208: ' TODO: revise to use a loop instead of a recursion - line 567: ' TODO: any flow analysis for initializers? - line 1162: '' TODO: add filtering as follows - line 1575: ' TODO (https://github.com/dotnet/roslyn/issues/2846): This will need to change for member initializers in partial class. - line 1618: stateMachineTypeNameOpt:=stateMachineTypeOpt?.Name, ' TODO: remove or update AddedOrChangedMethodInfo - line 1792: ' TODO (tomat): report errors if not available src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.pt-BR.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: liberar recursos não gerenciados (objetos não gerenciados) e substituir um finalizador abaixo. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: substituir um finalizador somente se Dispose(bool disposing) acima tiver o código para liberar recursos não gerenciados. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: remover marca de comentário da linha a seguir se o finalizador for substituído acima. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.es.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: libere los recursos no administrados (objetos no administrados) y reemplace el siguiente finalizador. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: reemplace un finalizador solo si el anterior Dispose(bool disposing) tiene código para liberar los recursos no administrados. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: quite la marca de comentario de la siguiente línea si el finalizador se ha reemplazado antes. src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.pl.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: zwolnij niezarządzane zasoby (niezarządzane obiekty) i przesłoń poniższą metodę Finalize(). - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: przesłoń metodę Finalize() tylko w sytuacji, gdy powyższa metoda Dispose(disposing As Boolean) ma kod umożliwiający zwolnienie niezarządzanych zasobów. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: usuń komentarz z poniższego wiersza, jeśli powyższa metoda Finalize() została przesłonięta. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.ko.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: 관리되지 않는 리소스(관리되지 않는 개체)를 해제하고 아래의 종료자를 재정의합니다. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: 위의 Dispose(bool disposing)에 관리되지 않는 리소스를 해제하는 코드가 포함되어 있는 경우에만 종료자를 재정의합니다. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: 위의 종료자가 재정의된 경우 다음 코드 줄의 주석 처리를 제거합니다. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.zh-Hant.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: 釋放 Unmanaged 資源 (Unmanaged 物件) 並覆寫下方的完成項。 - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: 僅當上方的 Dispose(bool disposing) 具有會釋放 Unmanaged 資源的程式碼時,才覆寫完成項。 - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: 如果上方的完成項已被覆寫,即取消下行的註解狀態。 src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.es.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: libere los recursos no administrados (objetos no administrados) y reemplace Finalize() a continuación. - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: reemplace Finalize() solo si el anterior Dispose(disposing As Boolean) tiene código para liberar recursos no administrados. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: quite la marca de comentario de la siguiente línea si Finalize() se ha reemplazado antes. src/Compilers/CSharp/Portable/FlowAnalysis/PreciseAbstractFlowPass.cs (6 lines): - line 246: // TODO: There are no scenarios involving a zero-length span - line 1656: // TODO: should events be handled specially too? - line 1698: // TODO: should events be handled specially too? - line 1784: // TODO: In an expression such as - line 2060: // TODO: should we also specially handle events? - line 2388: // TODO: in a struct constructor, "this" is not initially assigned. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.fr.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: libérer les ressources non managées (objets non managés) et remplacer un finaliseur ci-dessous. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: remplacer un finaliseur seulement si la fonction Dispose(bool disposing) ci-dessus a du code pour libérer les ressources non managées. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: supprimer les marques de commentaire pour la ligne suivante si le finaliseur est remplacé ci-dessus. src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.de.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: nicht verwaltete Ressourcen (nicht verwaltete Objekte) freigeben und Finalize() weiter unten überschreiben. - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: Finalize() nur überschreiben, wenn Dispose(disposing As Boolean) weiter oben Code zur Bereinigung nicht verwalteter Ressourcen enthält. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: Auskommentierung der folgenden Zeile aufheben, wenn Finalize() oben überschrieben wird. src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.zh-Hans.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: 释放未托管的资源(未托管的对象)并在以下内容中替代终结器。 - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: 仅当以上 Dispose(bool disposing) 拥有用于释放未托管资源的代码时才替代终结器。 - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: 如果在以上内容中替代了终结器,则取消注释以下行。 src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.ru.xlf (6 lines): - line 1496: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1497: TODO: освободить неуправляемые ресурсы (неуправляемые объекты) и переопределить ниже Finalize(). - line 1501: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1502: TODO: переопределить Finalize(), только если Dispose(disposing As Boolean) выше имеет код для освобождения неуправляемых ресурсов. - line 1511: TODO: uncomment the following line if Finalize() is overridden above. - line 1512: TODO: раскомментировать следующую строку, если Finalize() переопределен выше. src/EditorFeatures/CSharpTest/EditAndContinue/TopLevelEditingTests.cs (6 lines): - line 4637: // TODO: bug https://github.com/dotnet/roslyn/issues/2504 - line 5391: // TODO (tomat): diagnostic should point to the field initializer - line 5404: // TODO (tomat): diagnostic should point to the property initializer - line 5428: // TODO (tomat): diagnostic should point to the property initializer - line 6681: // TODO (bug https://github.com/dotnet/roslyn/issues/2504): - line 7042: // TODO: we can allow the move since the property doesn't have a backing field src/Features/CSharp/Portable/xlf/CSharpFeaturesResources.it.xlf (6 lines): - line 251: TODO: free unmanaged resources (unmanaged objects) and override a finalizer below. - line 252: TODO: liberare risorse non gestite (oggetti non gestiti) ed eseguire sotto l'override di un finalizzatore. - line 256: TODO: override a finalizer only if Dispose(bool disposing) above has code to free unmanaged resources. - line 257: TODO: eseguire l'override di un finalizzatore solo se Dispose(bool disposing) include il codice per liberare risorse non gestite. - line 266: TODO: uncomment the following line if the finalizer is overridden above. - line 267: TODO: rimuovere il commento dalla riga seguente se è stato eseguito l'override del finalizzatore. src/Features/VisualBasic/Portable/xlf/VBFeaturesResources.ja.xlf (6 lines): - line 1495: TODO: free unmanaged resources (unmanaged objects) and override Finalize() below. - line 1496: TODO: アンマネージ リソース (アンマネージ オブジェクト) を解放し、下の Finalize() をオーバーライドします。 - line 1500: TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources. - line 1501: TODO: 上の Dispose(disposing As Boolean) にアンマネージ リソースを解放するコードが含まれる場合にのみ Finalize() をオーバーライドします。 - line 1510: TODO: uncomment the following line if Finalize() is overridden above. - line 1511: TODO: 上の Finalize() がオーバーライドされている場合は、次の行のコメントを解除してください。 src/VisualStudio/VisualBasic/Impl/ProjectSystemShim/VisualBasicProject.vb (5 lines): - line 129: 'TODO: report error - line 155: ' TODO: implement - line 207: ' TODO: implement - line 349: ' TODO: implement - line 363: ' TODO: implement src/Compilers/CSharp/Portable/Symbols/Source/SourceAssemblySymbol.cs (5 lines): - line 455: // TODO: - line 551: //TODO: cache - line 1126: // TODO: This list used to include AssemblyOperatingSystemAttribute and AssemblyProcessorAttribute, - line 1545: /// TODO: We should replace methods GetSourceDecodedWellKnownAttributeData and GetNetModuleDecodedWellKnownAttributeData with - line 2542: // TODO: localize these strings src/Compilers/VisualBasic/Portable/Parser/BlockContexts/DeclarationContext.vb (5 lines): - line 142: 'TODO - davidsch - ParseSpecifierDeclaration reports ERRID_ExpectedDeclaration but ParsePropertyOrEventProcedureDefinition reports ERRID_Syntax. - line 206: 'TODO - Does parser create Constructors? Verify that code was ported correctly. - line 343: ' TODO: need to add all executable statements that ParseDeclarationStatement handle as unexpected executables - line 443: 'TODO - davidsch - handle regions - line 456: 'TODO src/Compilers/VisualBasic/Portable/Compilation/VisualBasicCompilation.vb (5 lines): - line 734: ' TODO: look for return statements - line 1146: ' TODO: This comparison probably will change to compiler command line order, or at least needs - line 1147: ' TODO: to be resolved. See bug 8520. - line 1802: ' TODO: See CSharpCompilation.GetTypeByReflectionType - line 2296: ' TODO (tomat): XML doc comments diagnostics src/Scripting/CSharpTest/ObjectFormatterTests.cs (5 lines): - line 274: // TODO: better overflow handling - line 874: // TODO (DevDiv #173210): Should show Fixture.Method - line 898: // TODO (DevDiv #173210): Should show Fixture.Method - line 922: // TODO (DevDiv #173210): Should show Fixture.Method - line 1020: // TODO (DevDiv #173210): Should show ParametersFixture.Method(ref char) src/EditorFeatures/Core.Wpf/Preview/PreviewFactoryService.cs (5 lines): - line 313: // TODO: We could use ITextBufferCloneService instead here to clone the original buffer. - line 340: // TODO: We could use ITextBufferCloneService instead here to clone the original buffer. - line 374: // TODO: We could use ITextBufferCloneService instead here to clone the original buffer. - line 457: // TODO: We could use ITextBufferCloneService instead here to clone the original buffer. - line 473: // TODO: Why aren't we attaching conflict / warning annotations here like we do for regular documents above? src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_Conversion.cs (5 lines): - line 472: // TODO: Consider doing constant folding for default parameter value conversion. - line 1386: // TODO: how do we distinguish from normal and lifted conversions here? - line 1405: // TODO: what about nullable? - line 1420: // TODO: what about nullable? - line 1477: // TODO: distinguish between normal and lifted conversions here src/Compilers/CSharp/Portable/Binder/Binder_Lookup.cs (5 lines): - line 208: // TODO: Diagnose ambiguity problems here, and conflicts between non-method and method? Or is that - line 218: // TODO: generalize to other result kinds and/or candidate counts? - line 253: // TODO: optimize lookup (there might be many interactions in the chain) - line 1543: // TODO: we need tests - line 1544: // TODO: optimize lookup (there might be many interactions in the chain) src/Compilers/VisualBasic/Portable/Binding/Binder_Invocation.vb (5 lines): - line 2278: ' TODO: Dev10 uses the location of the type parameter or argument that - line 2482: ' TODO: Do we need to do more thorough check for error types here, i.e. dig into generics, - line 2552: ' TODO: Do we need to do more thorough check for error types here, i.e. dig into generics, - line 2831: ' TODO: Fields of MarshalByRef object are passed via temp. - line 3055: ' TODO: revise (add warning/error?) src/Compilers/VisualBasic/Portable/Binding/Binder_Lookup.vb (5 lines): - line 252: ' TODO maybe this condition needs to be adjusted to be shown in cases of e.g. inaccessible properties - line 741: ' TODO: Add field support in the C# and VB member comparers and then - line 906: ' TODO (tomat): extension methods - line 1942: ' TODO (tomat): import aliases - line 1951: ' TODO (tomat): extension methods src/Compilers/CSharp/Portable/Lowering/SyntheticBoundNodeFactory.cs (5 lines): - line 193: // TODO: we used to have an invariant that a bound node factory was tied to a - line 287: return Call(receiverOpt, property.GetMethod); // TODO: should we use property.GetBaseProperty().GetMethod to ensure we generate a call to the overridden method? - line 560: // TODO: add diagnostics for when things fall apart - line 567: // TODO: add diagnostics for when things fall apart - line 928: // TODO: add diagnostics for when things fall apart src/EditorFeatures/CSharpTest/EditAndContinue/StatementEditingTests.cs (5 lines): - line 2018: // TODO: allow creating a new leaf closure - line 4361: // TODO: better diagnostics - identify a1 that causes the capture vs. a1 that doesn't - line 4569: // TODO: "a => x + x0" is matched with "a => y1 + x0", hence we report more errors. - line 4997: // TODO: allow creating a new leaf closure - line 7769: // TODO: better location (the variable, not the from clause) src/Compilers/CSharp/Portable/CodeGen/Optimizer.cs (5 lines): - line 57: //TODO: run other optimizing passes here. - line 141: //TODO: perf. This can be simplified to not use a query. - line 604: // TODO: an interesting case arises in unused x[i]++ and ++x[i] : - line 1060: // TODO: for some reason receiver could be not null even if method is static... - line 1154: // TODO: do we need to save/restore old context here? src/VisualStudio/InteractiveServices/Interactive/CommonVsUtils.cs (5 lines): - line 16: // TODO (tomat): this needs to be polished and tested - line 47: // TODO: Report error - line 54: // TODO: Report error - line 60: // TODO: Report error - line 68: // TODO: Report error src/Compilers/CSharp/Portable/Symbols/NonMissingModuleSymbol.cs (4 lines): - line 85: // TODO (tomat): we should report an error/warning for all unified references, not just the first one. - line 111: ownerAssembly.Name, // TODO (tomat): should rather be MetadataReference.Display for the corresponding reference - line 127: ownerAssembly.Name, // TODO (tomat): should rather be MetadataReference.Display for the corresponding reference - line 130: dependentAssembly.Name, // TODO (tomat): should rather be MetadataReference.Display for the corresponding reference src/Compilers/VisualBasic/Portable/Parser/ParseQuery.vb (4 lines): - line 138: 'TODO: this is very different from original implementation. - line 281: ' TODO: Merge with ParseFromControlVars. The two methods are almost identical. - line 687: ' TODO: semantics - line 1259: ' TODO: this should be done in semantics. src/Features/Core/Portable/xlf/FeaturesResources.es.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: elimine el estado administrado (objetos administrados). - line 1092: TODO: set large fields to null. - line 1093: TODO: configure los campos grandes en nulos. src/Compilers/VisualBasic/Portable/Symbols/Source/SourceAssemblySymbol.vb (4 lines): - line 164: ' TODO: This list used to include AssemblyOperatingSystemAttribute and AssemblyProcessorAttribute, - line 548: ''' TODO: We should replace methods GetSourceDecodedWellKnownAttributeData and GetNetModuleDecodedWellKnownAttributeData with - line 796: ' TODO: cache - line 1363: 'TODO need to figure out the right behavior when command line and src/Compilers/VisualBasic/Portable/Symbols/UnsupportedMetadataTypeSymbol.vb (4 lines): - line 22: ' TODO: Do we want to do anything with the "explanation". C# uses it in error messages - line 23: ' TODO: and it seems worth no losing it in cases. - line 24: ' TODO: If so, we need to go back and fill in cases that call the parameterless - line 25: ' TODO: constructor. src/Features/Core/Portable/xlf/FeaturesResources.de.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: verwalteten Zustand (verwaltete Objekte) entsorgen. - line 1092: TODO: set large fields to null. - line 1093: TODO: große Felder auf Null setzen. src/Features/Core/Portable/xlf/FeaturesResources.ko.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: 관리되는 상태(관리되는 개체)를 삭제합니다. - line 1092: TODO: set large fields to null. - line 1093: TODO: 큰 필드를 null로 설정합니다. src/Workspaces/Core/Portable/Formatting/Engine/AbstractFormatEngine.cs (4 lines): - line 18: // TODO : two alternative design possible for formatting engine - line 360: // TODO : figure out a way to run alignment operations in parallel. probably find - line 396: // TODO: find out a way to apply anchor operation concurrently if possible - line 496: // TODO : need to revisit later for the case where line and space operations src/Features/Core/Portable/xlf/FeaturesResources.it.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: eliminare lo stato gestito (oggetti gestiti). - line 1092: TODO: set large fields to null. - line 1093: TODO: impostare campi di grandi dimensioni su Null. src/Compilers/VisualBasic/Portable/Lowering/AsyncRewriter/AsyncRewriter.AsyncMethodToClassRewriter.Await.vb (4 lines): - line 25: ' TODO: We may have problems in case - line 26: ' TODO: the awaiter is being mutated in the code flow below; revise and get rid - line 27: ' TODO: of local *or* make sure mutable awaiter works - line 182: ' TODO: POSTPROCESS ASSIGNMENT src/Features/Core/Portable/xlf/FeaturesResources.pt-BR.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: descartar estado gerenciado (objetos gerenciados). - line 1092: TODO: set large fields to null. - line 1093: TODO: definir campos grandes como nulos. src/Compilers/VisualBasic/Portable/Compilation/SemanticModel.vb (4 lines): - line 980: ' TODO: Understand cases where we don't have a conversion node better. - line 1021: ' TODO: Understand the case patched by TODO in GetSemanticInfoForNode and create a better fix. - line 1290: 'TODO: Will the below work correctly even in the case where M is a parameterless method that returns - line 1943: ' TODO: What happens here if options has LookupOptions.AllMethodsOfAnyArity bit set? src/Features/Core/Portable/xlf/FeaturesResources.fr.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: supprimer l'état managé (objets managés). - line 1092: TODO: set large fields to null. - line 1093: TODO: définir les champs de grande taille avec la valeur Null. src/Compilers/CSharp/Portable/Emitter/EditAndContinue/CSharpSymbolMatcher.cs (4 lines): - line 490: // TODO: LambdaFrame has alpha renamed type parameters, should we rather fix that? - line 677: // TODO: Test with overloads (from PE base class?) that have modifiers. - line 730: // TODO: Test with overloads (from PE base class?) that have modifiers. - line 751: // TODO: Test with overloads (from PE base class?) that have modifiers. src/VisualStudio/Core/Def/Implementation/CodeLens/RemoteCodeLensReferencesService.cs (4 lines): - line 35: // TODO: send telemetry on session - line 59: // TODO: send telemetry on session - line 83: // TODO: send telemetry on session - line 107: // TODO: send telemetry on session src/EditorFeatures/CSharpTest/CodeActions/IntroduceVariable/IntroduceVariableTests.cs (4 lines): - line 2879: var d = [|true|] // TODO: comment - line 2891: var d = V // TODO: comment - line 2911: : [|2|]; // TODO: comment - line 2923: : V; // TODO: comment src/Features/Core/Portable/xlf/FeaturesResources.cs.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: Uvolněte spravovaný stav (spravované objekty). - line 1092: TODO: set large fields to null. - line 1093: TODO: Nastavte velká pole na hodnotu null. src/Workspaces/CSharp/Portable/Rename/CSharpRenameRewriterLanguageService.cs (4 lines): - line 608: // TODO: we can't use escaped unicode characters in xml doc comments, so we need to pass the valuetext as text as well. - line 986: // TODO: handle Dispose for using statement and Add methods for collection initializers. - line 1021: // TODO: support other implicitly used methods like dispose - line 1025: // TODO: partial methods currently only show the location where the rename happens as a conflict. src/Compilers/VisualBasic/Portable/Symbols/Source/ImplementsHelper.vb (4 lines): - line 450: ' TODO: Perhaps give ERR_CantImplementNonVirtual3 like Dev10. But, this message seems more - line 451: ' TODO: confusing than useful, so for now, just treat it like a method that doesn't exist. - line 492: ' TODO: If implementing event, check that delegate types are consistent, or maybe set the delegate type. See Dev10 compiler - line 493: ' TODO: in ImplementsSemantics.cpp, Bindable::BindImplements. src/Scripting/Core/Hosting/ObjectFormatter/CommonObjectFormatter.Visitor.cs (4 lines): - line 78: // TODO (https://github.com/dotnet/roslyn/issues/6689): remove this - line 222: // TODO (tomat): we should not use recursion - line 409: string v = FormatWithEmbeddedExpressions(lengthLimit, debuggerDisplay.Value, obj) ?? string.Empty; // TODO: ? - line 437: if ((array = value as Array) != null) // TODO (tomat): n-dim arrays src/Features/Core/Portable/xlf/FeaturesResources.zh-Hant.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: 處置 Managed 狀態 (Managed 物件)。 - line 1092: TODO: set large fields to null. - line 1093: TODO: 將大型欄位設為 null。 src/Compilers/CSharp/Portable/Parser/Blender.cs (4 lines): - line 42: // TODO: Consider implementing NormalizedChangeCollection for TextSpan. the real - line 50: // TODO: this is a temporary measure to prevent individual change spans from - line 116: // max. TODO: 1 token lookahead seems a bit too optimistic. Increase if needed. - line 119: // check if change is not after the end. TODO: there should be an assert somewhere about src/Compilers/CSharp/Portable/Parser/DocumentationCommentParser.cs (4 lines): - line 12: // TODO: The Xml parser recognizes most commonplace XML, according to the XML spec. - line 109: // TODO: It is possible that a non-literal gets in here. ]]>, specifically. Is that ok? - line 615: // TODO: It is possible that a non-literal gets in here. <, specifically. Is that ok? - line 733: // TODO: It is possible that a non-literal gets in here. --, specifically. Is that ok? src/EditorFeatures/VisualBasicTest/RemoveUnnecessaryImports/RemoveUnnecessaryImportsTests.vb (4 lines): - line 660: ' TODO: Enable testing in script when it comes online - line 812: End Class|]") 'TODO (tomat): modules not yet supported in script - line 827: End Class|]") 'TODO (tomat): modules not yet supported in script - line 858: ' TODO: Enable script context testing. src/Compilers/VisualBasic/Portable/Binding/Binder_Attributes.vb (4 lines): - line 263: ' TODO: we may want to fix the behavior of the Lookup result to contain more than one e.g. inaccessible symbol. - line 306: ' TODO - Split ReportOverloadResolutionFailureAndProduceBoundNode into two methods. One that does error reporting and one that - line 614: ' TODO: revise - line 622: ' TODO: check/revise src/EditorFeatures/VisualBasic/LineCommit/ContainingStatementInfo.MatchingStatementsVisitor.vb (4 lines): - line 44: ' TODO: evilness around ending multiple statements at once with a single "next" - line 49: ' TODO: evilness around ending multiple statements at once with a single "next" - line 54: ' TODO: just use Yield Return once we have it in VB - line 91: ' TODO: just use Yield Return once we have it in VB src/Compilers/Core/Portable/PEWriter/Members.cs (4 lines): - line 198: { // TODO: add custom modifiers - line 245: /// TODO: use instead. - line 251: /// TODO: use instead. - line 582: /// TODO: Ideally we would expose INamespace on INamespaceTypeDefinition. Right now we can only get the qualified namespace name. src/Compilers/CSharp/Portable/Symbols/Source/SourceMemberMethodSymbol.cs (4 lines): - line 282: // TODO: if this lock ever encloses a potential call to Debugger.NotifyOfCrossThreadDependency, - line 395: // TODO (tomat): sealed - line 407: // TODO (tomat): sealed? - line 427: // TODO (tomat): sealed? src/Features/Core/Portable/xlf/FeaturesResources.pl.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: wyczyść stan zarządzany (obiekty zarządzane). - line 1092: TODO: set large fields to null. - line 1093: TODO: ustaw wartość null dla dużych pól. src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/BatchFixAllProvider.cs (4 lines): - line 123: // TODO: Wrap call to ComputeFixesAsync() below in IExtensionManager.PerformFunctionAsync() so that - line 193: // TODO: Wrap call to ComputeFixesAsync() below in IExtensionManager.PerformFunctionAsync() so that - line 371: // TODO: Handle added/removed documents - line 372: // TODO: Handle changed/added/removed additional documents src/Features/Core/Portable/xlf/FeaturesResources.ja.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: マネージ状態を破棄します (マネージ オブジェクト)。 - line 1092: TODO: set large fields to null. - line 1093: TODO: 大きなフィールドを null に設定します。 src/Compilers/CSharp/Portable/Compilation/CSharpSemanticModel.cs (4 lines): - line 760: // TODO: Should we support positions that are not in existing constructor initializers? - line 1762: // TODO: Should parenthesized expression really not have symbols? At least for C#, I'm not sure that - line 1881: // TODO: Should parenthesized expression really not have symbols? At least for C#, I'm not sure that - line 3197: // TODO: src/Compilers/CSharp/Portable/Parser/Lexer_StringLiteral.cs (4 lines): - line 280: // TODO: scan the contents (perhaps using ScanInterpolatedStringLiteralContents) to reconstruct any lexical - line 628: // TODO: after the colon this has no special meaning. - line 632: // TODO: after the colon this has no special meaning. - line 636: // TODO: after the colon this has no special meaning. src/Features/VisualBasic/Portable/EditAndContinue/VisualBasicEditAndContinueAnalyzer.vb (4 lines): - line 2035: ' TODO: DllImport/Declare? - line 2425: ' TODO (tomat): We could be smarter and consider the following syntax changes to be legal: - line 2578: ' TODO (tomat): We can support this - line 3002: ' TODO: changes around suspension points (foreach, lock, using, etc.) src/Compilers/Core/Portable/PEWriter/MetadataWriter.PortablePdb.cs (4 lines): - line 79: // TODO: the compiler should produce a scope for each debuggable method - line 158: // TODO: BlobEncoder.LocalConstantSignature - line 346: writer.WriteCompressedInteger(CodedIndex.TypeDefOrRefOrSpec(GetTypeHandle(import.TargetTypeOpt))); // TODO: index in release build - line 381: // TODO: cache? src/Features/Core/Portable/xlf/FeaturesResources.zh-Hans.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: 释放托管状态(托管对象)。 - line 1092: TODO: set large fields to null. - line 1093: TODO: 将大型字段设置为 null。 src/Features/Core/Portable/xlf/FeaturesResources.tr.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: yönetilen durumu (yönetilen nesneleri) bırakın. - line 1092: TODO: set large fields to null. - line 1093: TODO: büyük alanları null olarak ayarlayın. src/Compilers/CSharp/Portable/Lowering/StateMachineRewriter/MethodToStateMachineRewriter.cs (4 lines): - line 762: // TODO (move to AsyncMethodToStateMachineRewriter, this is not applicable to iterators) - line 896: //TODO: It seems we may capture more than needed here. - line 898: // TODO: Why don't we drop "this" while lowering if method is static? - line 912: // TODO: fix up the type of the resulting node to be the base type src/Features/Core/Portable/xlf/FeaturesResources.ru.xlf (4 lines): - line 1087: TODO: dispose managed state (managed objects). - line 1088: TODO: освободить управляемое состояние (управляемые объекты). - line 1092: TODO: set large fields to null. - line 1093: TODO: задать большим полям значение NULL. src/VisualStudio/Core/Impl/CodeModel/ExternalElements/ExternalCodeFunction.cs (3 lines): - line 46: // TODO: Verify VB implementation - line 96: // TODO: What if ReturnType is null? - line 115: // TODO: Verify VB implementation src/EditorFeatures/Core.Wpf/InlineRename/InlineRenameSession.OpenTextBufferManager.cs (3 lines): - line 349: // TODO: why does the following line hang when uncommented? - line 476: // TODO: Add these to the unresolvable conflict counts in the dashboard - line 595: // TODO: Track all spans at once src/Workspaces/Core/Desktop/Workspace/CommandLineProject.cs (3 lines): - line 25: // TODO (tomat): the method may throw all sorts of exceptions. - line 152: // TODO (tomat): what should be the assemblyName when compiling a netmodule? Should it be /moduleassemblyname - line 164: // TODO (https://github.com/dotnet/roslyn/issues/4967): src/Compilers/CSharp/Portable/Symbols/NamedTypeSymbol.cs (3 lines): - line 34: // TODO: How should anonymous types be represented? One possible options: have an - line 386: // TODO: Probably should provide similar accessors for static constructor, destructor, - line 387: // TODO: operators, conversions. src/Scripting/Core/Hosting/ObjectFormatter/ObjectFormatterHelpers.cs (3 lines): - line 52: // TODO (tomat): which assembly should we look at for dd attributes? - line 76: // TODO (tomat): which assembly should we look at for proxy attributes? - line 96: // TODO: Unify NoPIA interfaces src/Compilers/VisualBasic/Portable/Binding/Binder_Operators.vb (3 lines): - line 53: ' TODO: Add rewrite for Nullable. - line 361: ' TODO: I believe the IsTrue is just an optimization to prevent Nullable from unnecessary bubbling up the tree. - line 919: ' TODO: Add handling for TryCast, similar to DirectCast src/Compilers/VisualBasic/Portable/Analysis/FlowAnalysis/DataFlowPass.vb (3 lines): - line 1964: Throw ExceptionUtilities.UnexpectedValue(expr.Kind) ' TODO: is this reachable at all? - line 2082: Visit(localDecl) ' TODO: do we really need that? - line 2094: Visit(localDecl) ' TODO: do we really need that? src/Features/Core/Portable/EditAndContinue/EditSession.cs (3 lines): - line 97: // TODO: small dict, pool? - line 101: // TODO: small dict, pool? - line 231: // TODO (https://github.com/dotnet/roslyn/issues/1204): src/Workspaces/VisualBasic/Portable/Rename/VisualBasicRenameRewriterLanguageService.vb (3 lines): - line 762: ' TODO: handle Dispose for using statement and Add methods for collection initializers. - line 857: ' TODO: support other implicitly used methods like dispose - line 877: ' TODO: Partial methods currently only show the location where the rename happens As a conflict. src/Compilers/VisualBasic/Portable/Symbols/NamedTypeSymbol.vb (3 lines): - line 433: ' TODO: Is this correct for VB Module? - line 446: ' TODO: Is this correct for VB Module? - line 1161: ' TODO (tomat): src/Compilers/Core/Portable/ReferenceManager/CommonReferenceManager.Binding.cs (3 lines): - line 920: // TODO: figure out if we need to be able to resolve this ambiguity. - line 925: // TODO: need to make sure we error if such assembly declares local type in source. - line 932: // TODO: C# seems to pick the first one (but produces warnings when looking up predefined types). src/VisualStudio/Core/Impl/CodeModel/FileCodeModel_Refactoring.cs (3 lines): - line 19: // TODO: Support options - line 65: // TODO: Implement! - line 70: // TODO: Implement! src/Scripting/Core/Hosting/CommonMemberFilter.cs (3 lines): - line 18: // TODO (https://github.com/dotnet/roslyn/issues/23101): investigate which submission frames *can* be excluded - line 31: // TODO (https://github.com/dotnet/roslyn/issues/5250): look for other types indicating that we're in Roslyn code - line 38: // TODO (tomat): we don't want to include awaiter helpers, shouldn't they be marked by DebuggerHidden in FX? src/Compilers/Core/Portable/Syntax/InternalSyntax/SyntaxNodeCache.cs (3 lines): - line 43: // TODO: remove when done tweaking this cache. - line 161: // TODO: should use slotCount - line 171: // TODO: should use slotCount src/Compilers/CSharp/Portable/Symbols/Compilation_WellKnownMembers.cs (3 lines): - line 16: // TODO: (tomat) translated 1:1 from VB, might need adjustments - line 17: // TODO: (tomat) can we share more with VB? - line 153: // TODO: should GetTypeByMetadataName rather return a missing symbol? src/Compilers/VisualBasic/Portable/VisualBasicCompilationOptions.vb (3 lines): - line 263: ' TODO: implement (only called from VBI) https://github.com/dotnet/roslyn/issues/5854 - line 974: ' TODO: add check for - line 1091: deterministic:=False,' TODO: fix this src/Scripting/VisualBasic/VisualBasicScriptCompiler.vb (3 lines): - line 43: ' TODO: remember these per options instance so we don't need to reparse each submission - line 44: ' TODO: get imports out of compilation??? https://github.com/dotnet/roslyn/issues/5854 - line 57: ' TODO report Diagnostics src/Compilers/VisualBasic/Portable/Errors/MessageProvider.vb (3 lines): - line 230: ' TODO: Add an error code for CompileCancelled - line 398: Return ERRID.ERR_UnableToOpenResourceFile1 'TODO: refine (DevDiv #12914) - line 410: Return ERRID.ERR_UnableToOpenResourceFile1 'TODO: refine (DevDiv #12914) src/Features/VisualBasic/Portable/VBFeaturesResources.Designer.vb (3 lines): - line 2986: ''' Looks up a localized string similar to TODO: free unmanaged resources (unmanaged objects) and override Finalize() below.. - line 2996: ''' Looks up a localized string similar to TODO: override Finalize() only if Dispose(disposing As Boolean) above has code to free unmanaged resources.. - line 3006: ''' Looks up a localized string similar to TODO: uncomment the following line if Finalize() is overridden above.. src/Compilers/VisualBasic/Portable/Binding/Binder_AnonymousTypes.vb (3 lines): - line 272: ' TODO: when Dev10 reports ERR_BadOrCircularInitializerReference (BC36555) ?? - line 407: ' TODO: revise - line 454: ' TODO: revise and maybe report a special error message src/Compilers/VisualBasic/Portable/Symbols/Retargeting/RetargetingSymbolTranslator.vb (3 lines): - line 306: ' TODO: report better error? - line 503: ' TODO: Do we need to check constraints on type parameters as well? - line 816: ' TODO: if it is no longer missing in the target assembly, then we can resolve it here. src/Compilers/CSharp/Portable/Symbols/Retargeting/RetargetingSymbolTranslator.cs (3 lines): - line 340: // TODO: report better error? - line 579: // TODO: Do we need to check constraints on type parameters as well? - line 753: // TODO: if it is a missing symbol error but no longer missing in the target assembly, then we can resolve it here. src/Workspaces/Core/Portable/Execution/Serializer_Asset.cs (3 lines): - line 27: // TODO: refactor this part in its own abstraction (Bits) that has multiple sub types - line 72: // TODO: once compiler team adds ability to serialize compilation options to ObjectWriter directly, we won't need this. - line 95: // TODO: once compiler team adds ability to serialize parse options to ObjectWriter directly, we won't need this. src/Scripting/Core/Hosting/AssemblyLoader/InteractiveAssemblyLoader.cs (3 lines): - line 284: // TODO: We shouldn't block this on CoreCLR. - line 300: // TODO: Stop using reflection once ModuleVersionId property once is available in Core contract. - line 320: // TODO: Desktop FX only src/VisualStudio/Core/Impl/CodeModel/ExternalElements/ExternalCodeEvent.cs (3 lines): - line 60: // TODO: Verify VB implementation - line 66: // TODO: Verify VB implementation - line 103: // TODO: Verify this with VB implementation src/Compilers/VisualBasic/Portable/Scanner/Directives.vb (3 lines): - line 212: SyntaxKind.EnableWarningDirectiveTrivia, 'TODO: Add support for processing #Enable and #Disable - line 397: ' TODO - Is the following comment still relevant? How should the error be reported? - line 431: ' TODO - Is the following comment still relevant? How should the error be reported? src/Scripting/CSharpTest.Desktop/InteractiveSessionTests.cs (3 lines): - line 296: // TODO (https://github.com/dotnet/roslyn/issues/6456): - line 316: // TODO (https://github.com/dotnet/roslyn/issues/6456): - line 461: // TODO: enable this with our AssemblyLoader: src/Compilers/VisualBasic/Portable/CommandLine/VisualBasicCommandLineParser.vb (3 lines): - line 1362: pdbChecksumAlgorithm:=HashAlgorithmName.SHA256) ' TODO: set from /checksumalgorithm (see https://github.com/dotnet/roslyn/issues/24735) - line 1577: ' TODO: localize ? - line 1588: ' TODO: localize ? src/Compilers/VisualBasic/Portable/Analysis/FlowAnalysis/ControlFlowPass.vb (3 lines): - line 86: ' TODO: uncomment the following line - line 97: ' TODO: uncomment the following line - line 107: ' TODO: uncomment the following line src/Features/Core/Portable/EditAndContinue/RudeEditDiagnosticDescriptors.cs (3 lines): - line 9: // TODO (tomat): cleanup and localize - line 76: // TODO: change the error message to better explain what's going on - line 100: title: message, // TODO: come up with real titles. src/Compilers/VisualBasic/Portable/Parser/ParseScan.vb (3 lines): - line 21: 'TODO - This applies ot both IsContinuableEOL and PeekPastStatementTerminator and any code that peeks through an EOL. - line 34: 'TODO - This is really peekToken skipping optional statementterminator - line 245: ' TODO: note that queries sometimes use very large lists of resync tokens. src/VisualStudio/InteractiveServices/Interactive/VsResetInteractive.cs (3 lines): - line 99: // TODO: Revert this back to using dynamic for web projects, since they have copies of these interfaces. - line 136: // TODO (tomat): project Scripts dir - line 166: #if TODO // TODO: This shouldn't directly depend on GAC, rather we should have some kind of "reference simplifier". src/Compilers/VisualBasic/Portable/Symbols/SubstitutedNamedType.vb (3 lines): - line 405: ' TODO - Perf - line 418: ' TODO - Perf - line 746: ' TODO: Remove this code once we are confident that it is really unreachable. src/EditorFeatures/VisualBasicTest/TypeInferrer/TypeInferrerTests.vb (3 lines): - line 499: ' TODO: review this - line 511: ' TODO: review this - line 523: ' TODO: review this src/Features/VisualBasic/Portable/Completion/KeywordRecommenders/Declarations/InKeywordRecommender.vb (3 lines): - line 23: ' TODO: make this API less ugly in the parser - line 26: ' TODO: improve this - line 34: ' TODO: figure out if this is the parse tree not acting correctly here. Why is this a SyntaxNonTerminal? src/Compilers/VisualBasic/Portable/Symbols/Source/SourceNamespaceSymbol.vb (3 lines): - line 520: ' TODO: do we really want to omit these warnings and display a new one after each fix? - line 529: ' TODO: report "related location" rather than including path in the message: - line 536: ' TODO: once the declarations are sorted, one might cache the filename if the first declaration matches the case. src/Workspaces/Core/Portable/Workspace/Solution/TextDocumentState.cs (3 lines): - line 54: // TODO: optimize this so that we only re-calcuate checksums that are actually changed - line 166: // TODO: Adjust this behavior in the future if we add support for non-text additional files - line 211: // TODO: Adjust this behavior in the future if we add support for non-text additional files src/Interactive/EditorFeatures/Core/Extensibility/Interactive/InteractiveEvaluator.cs (3 lines): - line 264: // TODO: To support CoreCLR we need to query the remote process for TPA list and pass it to the resolver. - line 320: // TODO (tomat): Is there a better way to remove mapping between buffer and document in REPL? - line 520: // TODO (https://github.com/dotnet/roslyn/issues/4725) src/Compilers/VisualBasic/Portable/Parser/BlockContexts/PropertyBlockContext.vb (3 lines): - line 64: ' TODO - In Dev10, the code tries to report ERRID_PropertyMemberSyntax. This would occur prior to auto properties - line 114: ' TODO - Consider changing the kind to AutoProperty. For now auto properties are just PropertyStatement - line 152: 'TODO - In Dev10 resources there is an unused resource ERRID.ERR_NewExpandedProperty for the new case. src/Compilers/VisualBasic/Portable/Syntax/LambdaUtilities.vb (3 lines): - line 77: ' TODO: handle peers - line 615: ''' TODO: Could we avoid generating proper lambdas for these? - line 694: ' TODO: EE expression src/Features/Core/Portable/SolutionCrawler/WorkCoordinator.cs (3 lines): - line 222: // TODO: add telemetry that record how much it takes to process an event (max, min, average and etc) - line 443: // TODO: Async version for GetXXX methods? - line 486: // TODO: why solution changes return Project not ProjectId but ProjectChanges return DocumentId not Document? src/Compilers/VisualBasic/Portable/Symbols/Source/OverrideHidingHelper.vb (3 lines): - line 166: ' TODO: only do this check if CheckOverrideMember didn't find an error? - line 632: ' TODO: When we support virtual events, that might change. - line 694: ' TODO: When we support virtual events, that might change. src/Workspaces/Remote/Core/Services/RoslynServices.cs (2 lines): - line 11: /// TODO: change all these services to WorkspaceServices - line 15: // TODO: probably need to split this to private and public services src/Compilers/CSharp/Portable/Symbols/ReferenceManager.cs (2 lines): - line 289: // TODO: https://github.com/dotnet/roslyn/issues/9004 - line 1025: // TODO (tomat): src/Workspaces/Core/Portable/Workspace/Solution/ProjectInfo.cs (2 lines): - line 440: // TODO: figure out a way to send version info over as well - line 451: // TODO: once CompilationOptions, ParseOptions, ProjectReference, MetadataReference, AnalyzerReference supports src/Compilers/VisualBasic/Portable/CodeGen/EmitAddress.vb (2 lines): - line 338: ' TODO: The condition is applicable only when LocalSymbol.IsReadOnly - line 352: 'TODO: we have to do the following to separate real locals src/Compilers/VisualBasic/Portable/Lowering/LambdaRewriter/LambdaRewriter.Analysis.vb (2 lines): - line 227: ' TODO: this could be precomputed and stored by analysis phase - line 518: 'TODO: synthetic gotos should be marked as compiler generated. src/Interactive/Features/Interactive/Core/InteractiveHost.Service.cs (2 lines): - line 158: // TODO (tomat): we should share the copied files with the host - line 581: // TODO (tomat): this is also done in CommonInteractiveEngine, perhaps we can pass the parsed command lines to here? src/Compilers/CSharp/Portable/Binder/Semantics/OverloadResolution/OverloadResolution.cs (2 lines): - line 230: if (members.Count > 50) // TODO: fine-tune this value - line 3207: // TODO (tomat): the spec wording isn't final yet src/Compilers/VisualBasic/Portable/Binding/Binder_Symbols.vb (2 lines): - line 111: ' TODO: Why do we get here with non-generic type and, - line 921: ' TODO: Dev10 squiggles type arguments for this error, but Roslyn will squiggle the whole type name. src/Compilers/CSharp/Portable/Binder/Binder.QueryUnboundLambdaState.cs (2 lines): - line 35: public override MessageID MessageID { get { return MessageID.IDS_FeatureQueryExpression; } } // TODO: what is the correct ID here? - line 41: // TODO: improved diagnostics for query expressions src/Compilers/Core/Portable/InternalUtilities/TextKeyedCache.cs (2 lines): - line 52: // TODO: Need to tweak the size with more scenarios. - line 89: // TODO: consider whether a counter is random enough src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_NullableHelpers.vb (2 lines): - line 78: ' TODO: in most cases right operand does not change value of the left one - line 338: ''' TODO: Perhaps belong to a different file src/Compilers/CSharp/Portable/Symbols/AssemblySymbol.cs (2 lines): - line 956: // TODO: Roslyn does not yet give an error in other circumstances whereby a strong-named assembly - line 957: // TODO: references a weak-named assembly. src/Workspaces/VisualBasic/Portable/Formatting/Engine/Trivia/VisualBasicTriviaFormatter.vb (2 lines): - line 220: ' TODO : make document comment to be formatted by structured trivia formatter as well. - line 245: ' TODO : make document comment to be formatted by structured trivia formatter as well. src/Compilers/VisualBasic/Portable/Analysis/FlowAnalysis/AbstractFlowPass.vb (2 lines): - line 265: ''' TODO: Need to try and get rid of this property - line 735: ' TODO: Other placeholder kinds? src/Compilers/VisualBasic/Portable/Operations/VisualBasicOperationFactory_Methods.vb (2 lines): - line 186: ' TODO: figure out if this is true: - line 194: ' TODO: figure our if this is true: src/Compilers/CSharp/Portable/Emitter/Model/PEModuleBuilder.cs (2 lines): - line 22: // TODO: Need to estimate amount of elements for this map and pass that value to the constructor. - line 142: // C# currently doesn't emit compilation level imports (TODO: scripting). src/Compilers/VisualBasic/Portable/Symbols/AnonymousTypes/SynthesizedSymbols/AnonymousTypeOrDelegateTemplateSymbol.vb (2 lines): - line 221: ' TODO - Perf - line 227: ' TODO - Perf src/Compilers/CSharp/Portable/BoundTree/BoundNodes.xml (2 lines): - line 639: TODO: it appears in C#, variable's extent (life time) never escapes its scope. - line 1113: src/VisualStudio/Core/Impl/CodeModel/InternalElements/CodeStruct.cs (2 lines): - line 65: // TODO: Fix -- C# uses the current node, not the symbol. - line 72: // TODO: Fix -- C# will actually allow the user to set an abstract modifier on a struct. VB throws E_NOTIMPL src/VisualStudio/Core/Impl/CodeModel/AbstractCodeModelService.cs (2 lines): - line 493: // TODO: (tomescht) make this testable through unit tests. - line 837: // TODO: Support vsCMTypeRef2? It doesn't appear that Dev10 C# does... src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_BinaryOperators.vb (2 lines): - line 237: ' TODO: Recheck - line 1104: 'TODO: when not initializing right temp, can use And. (fewer branches) src/EditorFeatures/VisualBasicTest/Completion/CompletionProviders/AbstractVisualBasicCompletionProviderTests.vb (2 lines): - line 46: ' TODO: Re-enable these when interactive is back in the product. - line 64: ' TODO: Re-enable these when interactive is back in the product. src/Workspaces/VisualBasic/Portable/LanguageServices/VisualBasicSyntaxFactsService.vb (2 lines): - line 466: ' TODO: There is no way to get the diagnostics to see if any are actually errors? - line 809: ' TODO: currently we only support method for now src/Tools/Source/CompilerGeneratorTools/Source/VisualBasicSyntaxGenerator/RedNodes/RedNodeFactoryWriter.vb (2 lines): - line 79: ' TODO: consider renaming the nodes or the general factory method to be unambiguous for - line 105: ' TODO: (if there is only one child and it is an optional list, then allow it to be optional unless there is already a zero-parameter factory) src/Compilers/VisualBasic/Portable/SymbolDisplay/SymbolDisplayVisitor.Types.vb (2 lines): - line 133: 'TODO: aliases - line 309: ' TODO: revise to generate user-friendly name src/Features/CSharp/Portable/CodeRefactorings/InlineTemporary/InlineTemporaryCodeRefactoringProvider.cs (2 lines): - line 74: // TODO: inlining ref returns: - line 119: // TODO: Check for more conflict types. src/Compilers/VisualBasic/Portable/Binding/Binder_Delegates.vb (2 lines): - line 283: ' TODO: Check boxing of restricted types, report ERRID_RestrictedConversion1 and continue. - line 460: ' TODO: Switch to using BoundValuePlaceholder, but we need it to be able to appear src/Workspaces/CSharpTest/Formatting/FormattingTests.cs (2 lines): - line 723: // // TODO: Add the implementation for Class1.goo() here. - line 730: // // TODO: Add the implementation for Class1.goo() here. src/VisualStudio/InteractiveServices/Interactive/VsInteractiveWindowProvider.cs (2 lines): - line 33: // TODO: support multi-instance windows - line 115: // TODO: we don't support multi-instance yet src/Tools/Source/CompilerGeneratorTools/Source/VisualBasicSyntaxGenerator/GreenNodes/GreenNodeWriter.vb (2 lines): - line 401: 'TODO: BLUE - line 589: 'TODO: BLUE src/Scripting/Core/Script.cs (2 lines): - line 300: // TODO: remove - line 328: // TODO: avoid throwing exception, report all diagnostics https://github.com/dotnet/roslyn/issues/5949 src/Workspaces/CSharp/Portable/Formatting/Engine/Trivia/CSharpTriviaFormatter.cs (2 lines): - line 305: // TODO : make document comment to be formatted by structured trivia formatter as well. - line 332: // TODO : make document comment to be formatted by structured trivia formatter as well. src/Compilers/Core/Portable/PEWriter/MetadataWriter.DynamicAnalysis.cs (2 lines): - line 79: // TODO: avoid making a copy if the blob exists in the index - line 242: // TODO: avoid allocations src/Compilers/VisualBasic/Portable/Parser/BlockContexts/FinallyPartBlock.vb (2 lines): - line 26: 'TODO - Davidsch - line 32: 'TODO - Davidsch src/Compilers/VisualBasic/Portable/Symbols/EventSymbol.vb (2 lines): - line 203: ' TODO: Perhaps the error wording could be changed a bit to say "type of event..." ? - line 211: ' TODO: Perhaps the error wording could be changed a bit to say "type of event..." ? src/Compilers/VisualBasic/Portable/Preprocessor/OperatorResolution.vb (2 lines): - line 7: ' TODO: this is not in Preprocessor namespace as it may be generally useful. - line 24: ' TODO: need to fix the tables and remove the mapping. src/VisualStudio/VisualBasic/Impl/ObjectBrowser/DescriptionBuilder.vb (2 lines): - line 336: ' TODO 'Shadows' modifier isn't exposed on symbols. Do we need it? - line 388: ' TODO: Declare methods may implicitly make string parameters ByRef. To fix this, src/Compilers/VisualBasic/Portable/Scanner/XmlCharacterGlobalHelpers.vb (2 lines): - line 16: ' TODO: which of the following is correct? - line 22: ' TODO: which of the following is correct? src/Compilers/CSharp/Portable/Compilation/CSharpCompilation.cs (2 lines): - line 2343: // TODO: Enable the below commented assert and remove the filtering code in the next line. - line 2485: // TODO (tomat): NoPIA: src/Compilers/Core/Portable/Compilation/Compilation.cs (2 lines): - line 348: // TODO (tomat): remove recursion - line 2295: // TODO: check if baseline is an assembly manifest module/netmodule src/Compilers/VisualBasic/Portable/CodeGen/Optimizer/StackScheduler.Analyzer.vb (2 lines): - line 238: ''' TODO: an interesting case arises in unused x[i]++ and ++x[i] : - line 754: ' TODO: do we need to save/restore old context here? src/Workspaces/Core/Portable/Execution/SolutionAsset.cs (2 lines): - line 59: // TODO: the way recoverable text works is a bit different than how recoverable tree works. - line 77: // TODO: make TextDocumentState to implement ISupportTemporaryStorage? src/VisualStudio/Core/Impl/CodeModel/InternalElements/CodeImport.cs (2 lines): - line 120: // TODO: Implement - line 134: // TODO: Implement src/Workspaces/CoreTest/Execution/SnapshotSerializationTests.cs (2 lines): - line 655: // TODO: do we need version? - line 694: // TODO: do we need version? src/Compilers/CSharp/Portable/SymbolDisplay/SymbolDisplayVisitor.Types.cs (2 lines): - line 327: // TODO: do we want to skip these if we're being visited as a containing type? - line 364: // TODO: revise to generate user-friendly name src/Compilers/VisualBasic/Portable/Symbols/ReferenceManager.vb (2 lines): - line 244: ' TODO: https://github.com/dotnet/roslyn/issues/9004 - line 887: ' TODO (tomat): src/Features/CSharp/Portable/ExtractMethod/CSharpSelectionValidator.cs (2 lines): - line 309: // TODO : check whether this can be handled by control flow analysis engine - line 315: // TODO : check behavior of control flow analysis engine around exception and exception handling. src/Features/Core/Portable/TodoComments/ITodoCommentService.cs (2 lines): - line 12: /// Description of a TODO comment type to find in a user's comments. - line 27: /// A TODO comment that has been found within the user's code. src/VisualStudio/Core/Def/Implementation/EditAndContinue/VsReadOnlyDocumentTracker.cs (2 lines): - line 150: // TODO (https://github.com/dotnet/roslyn/issues/1204): this check should be unnecessary. - line 165: // TODO: should not happen since the document is open (see bug 896058) src/Compilers/CSharp/Portable/FlowAnalysis/DataFlowsInWalker.cs (2 lines): - line 19: // TODO: normalize the result by removing variables that are unassigned in an unmodified - line 96: // TODO: how to handle fields of structs? src/Workspaces/Core/Desktop/Workspace/MSBuild/VisualBasic/VisualBasicProjectFileLoader.cs (2 lines): - line 41: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 - line 50: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 src/Interactive/EditorFeatures/CSharp/Completion/CompletionProviders/CSharpReplCommandCompletionProvider.cs (2 lines): - line 30: // TODO (tomat): REPL commands should have their own providers: - line 41: // TODO (tomat): REPL commands should have their own handlers: src/Compilers/VisualBasic/Portable/Binding/Binder_Query.vb (2 lines): - line 37: ' TODO: Do we need to report any diagnostic? - line 44: ' TODO: Do we need to report any diagnostic? src/Compilers/VisualBasic/Portable/Scanner/CharacterInfo.vb (2 lines): - line 231: 'TODO: make easy cases fast (or check if they already are) - line 239: ' TODO: replace CByte with something faster. src/Compilers/Core/Portable/InternalUtilities/StringTable.cs (2 lines): - line 28: // TODO: Need to tweak the size with more scenarios. - line 602: // TODO: This should be Encoding.UTF8.GetString (for better layering) but the unsafe variant isn't portable. src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_InterpolatedString.vb (2 lines): - line 29: ' TODO: A number of optimizations would be beneficial in the generated code. - line 72: ' TODO: A number of optimizations would be beneficial in the generated code. src/Workspaces/Core/Portable/Shared/Extensions/ICodeDefinitionFactoryExtensions.cs (2 lines): - line 137: // TODO: I'm not sure that's what we really want for ref parameters. - line 230: // TODO: I'm not sure that's what we really want for ref parameters. src/Compilers/CSharp/Portable/Symbols/MergedNamespaceSymbol.cs (2 lines): - line 234: // TODO - This is really inefficient. Creating a new array on each lookup needs to fixed! - line 270: //TODO: cache src/Compilers/VisualBasic/Portable/Emit/EditAndContinue/EmitHelpers.vb (2 lines): - line 47: ' TODO: better error code (https://github.com/dotnet/roslyn/issues/8910) - line 126: ' TODO can we reuse some data from the previous matcher? src/Compilers/VisualBasic/Portable/Lowering/Diagnostics/DiagnosticsPass_ExpressionLambdas.vb (2 lines): - line 43: ' TODO: There is a bug in Dev11 when the resulting expression tree fails to build in - line 97: ' TODO: There is a bug in Dev11 when the resulting expression tree fails to build in src/Features/CSharp/Portable/EditAndContinue/StatementSyntaxComparer.cs (2 lines): - line 126: // TODO: avoid allocation of closure - line 188: // TODO: src/Workspaces/Core/Desktop/Workspace/MSBuild/CSharp/CSharpProjectFileLoader.CSharpProjectFile.cs (2 lines): - line 27: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 - line 36: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 src/Compilers/Core/Portable/Emit/EditAndContinue/DefinitionMap.cs (2 lines): - line 60: // TODO (bug https://github.com/dotnet/roslyn/issues/2504) - line 166: // TODO (bug https://github.com/dotnet/roslyn/issues/2504): src/Compilers/Core/Portable/Emit/EditAndContinue/EncVariableSlotAllocator.cs (2 lines): - line 160: // TODO (bug #781309): Should report a warning if the type of the local has changed - line 214: // TODO (bug #781309): Should report a warning if the type of the local has changed src/Compilers/CSharp/Portable/Emitter/EditAndContinue/EmitHelpers.cs (2 lines): - line 51: // TODO: better error code (https://github.com/dotnet/roslyn/issues/8910) - line 140: // TODO: can we reuse some data from the previous matcher? src/Workspaces/Core/Portable/Workspace/Solution/Solution.cs (2 lines): - line 355: // TODO: make it public - line 1033: // TODO: actually make this a snapshot src/Workspaces/Remote/Core/Services/AssetService.cs (2 lines): - line 16: /// TODO: change this service to workspace service - line 50: // TODO: what happen if service doesn't come back. timeout? src/Features/Core/Portable/Diagnostics/AnalyzerHelper.cs (2 lines): - line 113: // TODO: find better way. - line 192: // TODO: It is not ideal to create a new descriptor per analyzer exception diagnostic instance. src/Compilers/VisualBasic/Portable/Compilation/DocumentationComments/DocumentationCommentCompiler.Common.vb (2 lines): - line 318: value = crefAttr.Reference.ToFullString().Trim() ' TODO: revise - line 323: value = nameAttr.Reference.Identifier.ToString() ' TODO: revise src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_Call.cs (2 lines): - line 582: // TODO: In case of __arglist, we will have more arguments than parameters, - line 1236: // TODO: Ideally, the enableCallerInfo parameter would be of just bool type with only 'true' and 'false' values, and all callers src/Compilers/VisualBasic/Portable/Scanner/XmlTokenFactories.vb (2 lines): - line 375: 'TODO - Normalize new lines. - line 400: 'TODO - Normalize new lines. src/Scripting/Core/Hosting/ObjectFormatter/CommonTypeNameFormatter.cs (2 lines): - line 28: // TODO (tomat): Use DebuggerDisplay.Type if specified? - line 234: // TODO (DevDiv #173210): shouldn't need parameters, but StackTrace gives us unconstructed symbols. src/Compilers/CSharp/Portable/Binder/Binder_Attributes.cs (2 lines): - line 330: // TODO: use fully qualified identifier name for boundNamedArgumentsSet - line 384: // TODO: should we create an entry even if there are binding errors? src/VisualStudio/Core/Impl/CodeModel/InternalElements/CodeAttributeArgument.cs (2 lines): - line 74: // TODO: VB returns (EnvDTE.vsCMElement)EnvDTE80.vsCMElement2.vsCMElementAttributeArgument - line 91: // TODO: VB throws E_NOTIMPL src/VisualStudio/Core/Def/Implementation/ProjectSystem/AbstractProject_Options.cs (2 lines): - line 149: // TODO: #r support, should it include bin path? - line 152: // TODO: #load support src/ExpressionEvaluator/Core/Source/ExpressionCompiler/PDB/MethodDebugInfo.Native.cs (2 lines): - line 60: EESymbolProvider symbolProviderOpt, // TODO: only null in DTEE case where we looking for default namesapace - line 673: // TODO (https://github.com/dotnet/roslyn/issues/1815): report error properly when the symbol is used src/Compilers/CSharp/Portable/Symbols/Symbol_Attributes.cs (2 lines): - line 307: // TODO: what if this succeeds on another thread, not ours? - line 327: // TODO: what if this succeeds on another thread but not this thread? src/EditorFeatures/Core.Wpf/Adornments/AdornmentManager.cs (2 lines): - line 327: // map start and end TODO: verify that affinity is correct. If it does not matter we - line 337: // TODO: actually adornments do not make much sense on "cropped" spans either - Consider line separator on "nd Su" src/Compilers/CSharp/Portable/Symbols/ModuleSymbol.cs (2 lines): - line 221: internal abstract ImmutableArray GetReferencedAssemblies(); // TODO: Remove this method and make ReferencedAssemblies property abstract instead. - line 246: internal abstract ImmutableArray GetReferencedAssemblySymbols(); // TODO: Remove this method and make ReferencedAssemblySymbols property abstract instead. src/Workspaces/Remote/Core/Diagnostics/DiagnosticComputer.cs (2 lines): - line 75: // TODO: can we support analyzerExceptionFilter in remote host? - line 117: // TODO: this probably need to be cached as well in analyzer service? src/Compilers/CSharp/Portable/Binder/Binder_Statements.cs (2 lines): - line 413: // TODO: verify that goto label lookup was valid (e.g. error checking of symbol resolution for labels) - line 2579: // TODO: Loop is O(n), caller is O(n^2). Perhaps we could iterate in reverse order (since it's easier to find src/Compilers/VisualBasic/Portable/Parser/BlockContexts/MethodBlockContext.vb (2 lines): - line 37: ' TODO - This check does not catch error for exit in a block in an method. Is this a syntactic or a - line 38: ' semantic error? TODO - Exit checking for Property done in parser but other Exit error checking is done src/Compilers/VisualBasic/Portable/Binding/NamedTypeBinder.vb (2 lines): - line 40: ' TODO (tomat): this is a temporary workaround, we need a special script class binder - line 46: ' TODO (tomat): this is a temporary workaround, we need a special script class binder src/Compilers/CSharp/Portable/Syntax/LambdaUtilities.cs (2 lines): - line 369: // TODO (bug https://github.com/dotnet/roslyn/issues/2663): - line 435: // TODO: EE expression src/EditorFeatures/VisualBasicTest/CodeActions/IntroduceVariable/IntroduceVariableTests.vb (2 lines): - line 2344: [|1|], ' TODO: Comment - line 2358: V, ' TODO: Comment src/Workspaces/Core/Portable/Shared/NormalizedTextSpanCollection.cs (2 lines): - line 431: // TODO: binary search - line 485: // TODO: binary search src/Features/Core/Portable/Diagnostics/EngineV2/DiagnosticAnalyzerExecutor.cs (2 lines): - line 45: // TODO: this should be removed once we move options down to compiler layer - line 177: // TODO: we need better way to deal with options. optionSet itself is green node but src/Features/Core/Portable/GenerateMember/GenerateParameterizedMember/AbstractGenerateParameterizedMemberService.SignatureInfo.cs (2 lines): - line 245: // TODO: Code coverage - line 254: // TODO: Code coverage src/EditorFeatures/Core/Implementation/ChangeSignature/AbstractChangeSignatureCommandHandler.cs (2 lines): - line 68: // TODO: reuse GetCommandState instead - line 139: // TODO: handle failure src/Workspaces/Remote/Core/Services/AssetStorage.cs (2 lines): - line 16: /// TODO: change this service to workspace service - line 20: // TODO: think of a way to use roslyn option service in OOP src/Compilers/VisualBasic/Portable/Preprocessor/ExpressionEvaluator.vb (2 lines): - line 218: ' TODO: should we use the node? - line 1347: ' TODO: verify with native code. src/Compilers/CSharp/Portable/Symbols/AnonymousTypes/AnonymousTypeManager.Templates.cs (2 lines): - line 125: // TODO (tomat): avoid recursion - line 147: // TODO (tomat): avoid recursion src/Compilers/CSharp/Portable/Binder/LookupResult.cs (2 lines): - line 199: // TODO: determine correct diagnosis - line 293: //TODO: consider if global pool is ok. src/Compilers/VisualBasic/Portable/Symbols/ModuleSymbol.vb (2 lines): - line 140: Friend MustOverride Function GetReferencedAssemblies() As ImmutableArray(Of AssemblyIdentity) ' TODO: Remove this method and make ReferencedAssemblies property abstract instead. - line 162: Friend MustOverride Function GetReferencedAssemblySymbols() As ImmutableArray(Of AssemblySymbol) ' TODO: Remove this method and make ReferencedAssemblySymbols property abstract instead. src/Compilers/CSharp/Portable/Symbols/Source/SourceDelegateMethodSymbol.cs (2 lines): - line 112: // TODO: move more functionality into here, making these symbols more lazy - line 183: // TODO: This implementation looks strange. It might make sense for the Invoke method, but src/VisualStudio/VisualBasic/Impl/CodeModel/VisualBasicCodeModelService.vb (2 lines): - line 154: 'TODO: This has been copied from CSharpCodeModelService. Tweak to implement VB semantics. - line 2019: ' TODO (tomescht): The Dev11 code allowed different sets of modifiers to be src/Compilers/VisualBasic/Portable/Symbols/Source/SourceNamedTypeSymbol.vb (2 lines): - line 1367: ' TODO: if there is a cycle dependency in base type we might want to ignore all - line 2461: 'TODO check that there's not a user supplied instance already. This attribute is AllowMultiple:=False. src/Compilers/VisualBasic/Portable/Symbols/TypedConstant.vb (2 lines): - line 40: ' TODO (tomat): use SymbolDisplay instead - line 325: ' TODO (tomat): use SymbolDisplay instead src/VisualStudio/VisualBasic/Impl/Debugging/ProximityExpressionsGetter.Worker.vb (2 lines): - line 139: Return True ' // TODO: need to hit this with unit-tests - line 144: Return True ' // // TODO: need to hit this with unit-tests src/Compilers/CSharp/Portable/Compilation/SyntaxTreeSemanticModel.cs (2 lines): - line 1065: // TODO (tomat): handle misplaced global statements - line 1816: // TODO: If this becomes a bottleneck, put the extern aliases in a dictionary, as for using aliases. src/Compilers/Core/Portable/Compilation/CompilationOptions.cs (2 lines): - line 169: // TODO: change visibility of the DebugPlusMode setter to internal & protected - line 186: // TODO: change visibility of the ReferencesSupersedeLowerVersions setter to internal & protected src/EditorFeatures/CSharpTest/GenerateVariable/GenerateVariableTests.cs (2 lines): - line 1445: // TODO: Move to TypeInferrer.InferTypes, or something - line 1469: // TODO: Move to TypeInferrer.InferTypes, or something src/Compilers/VisualBasic/Portable/Symbols/AnonymousTypes/PublicSymbols/AnonymousTypeOrDelegatePublicSymbol.vb (2 lines): - line 125: ' TODO - Perf - line 131: ' TODO - Perf src/Compilers/CSharp/Portable/Binder/Binder_Conversions.cs (2 lines): - line 911: // TODO: Some conversions can produce errors or warnings depending on checked/unchecked. - line 912: // TODO: Fold conversions on enums and strings too. src/Compilers/Core/Portable/CommandLine/CommonCommandLineParser.cs (2 lines): - line 93: // TODO: consider introducing "/s:path" to disambiguate paths starting with / - line 802: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 src/Compilers/VisualBasic/Portable/CodeGen/CodeGenerator.vb (2 lines): - line 187: ' TODO: combination load/store for +=; addresses for ref - line 206: ' TODO: combination load/store for +=; addresses for ref src/Compilers/CSharp/Portable/Binder/Semantics/OverloadResolution/OverloadResolutionResult.cs (2 lines): - line 165: // TODO: There might be a situation in which there were no valid results but we still want to identify a "best of a bad lot" result for - line 166: // TODO: error reporting. src/Compilers/CSharp/Portable/Symbols/Source/SourceLocalSymbol.cs (2 lines): - line 364: // TODO: this method must be overridden for pattern variables to bind the - line 390: /// TODO: check if there are multiple same name local variables - error symbol or local symbol? src/Scripting/Core/ScriptState.cs (2 lines): - line 104: // TODO: synthesized fields of submissions shouldn't be public - line 255: // TODO: better matching.. src/Compilers/CSharp/Portable/Compiler/MethodCompiler.cs (2 lines): - line 124: // TODO: revise to use a loop instead of a recursion - line 1416: // TODO (https://github.com/dotnet/roslyn/issues/2846): This will need to change for member initializers in partial class. src/VisualStudio/CSharp/Impl/ProjectSystemShim/TempPECompilerService.cs (2 lines): - line 46: // TODO (tomat): Revisit compilation options: app.config, strong name, search paths, etc? (bug #869604) - line 47: // TODO (tomat): move resolver initialization (With* methods below) to CommandLineParser.Parse src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter.vb (2 lines): - line 610: ' TODO: If the type is generic, we might want to use VB style name rather than emitted name. - line 646: ' TODO: If the type is generic, we might want to use VB style name rather than emitted name. src/VisualStudio/Core/Def/Implementation/AbstractOleCommandTarget.Query.cs (2 lines): - line 26: // TODO: We'll need to extend the command handler interfaces at some point when we have commands that - line 212: /// TODO: Revert the change to use standard VS11 command pending https://github.com/dotnet/roslyn/issues/8927 . src/Setup/DevDivInsertionFiles/BuildDevDivInsertionFiles.vb (2 lines): - line 476: ' TODO: remove (https://github.com/dotnet/roslyn/issues/13204) - line 622: ' TODO: remove (https://github.com/dotnet/roslyn/issues/13204) src/Compilers/VisualBasic/Portable/Symbols/Source/SourceEventSymbol.vb (2 lines): - line 157: ' TODO: why AsClause is not a SimpleAsClause in events? There can't be "As New" - line 370: ' TODO: Handle custom modifiers, including modifiers. src/EditorFeatures/VisualBasicTest/Diagnostics/GenerateVariable/GenerateVariableTests.vb (2 lines): - line 1012: parseOptions:=Nothing) ' TODO (tomat): Modules nested in Script class not supported yet - line 1045: parseOptions:=Nothing) ' TODO (tomat): Modules nested in Script class not supported yet) src/Compilers/VisualBasic/Portable/Binding/Binder_Latebound.vb (2 lines): - line 132: 'TODO: may need to distinguish indexing/calling/dictionary - line 133: 'TODO: for example "Dim a = ("a".Clone)()" is an IndexGet src/Compilers/CSharp/Portable/Lowering/LambdaRewriter/LambdaRewriter.cs (2 lines): - line 789: // TODO: Why don't we drop "this" while lowering if method is static? - line 1120: // TODO: we may not need to update if there was nothing to rewrite. src/Workspaces/Core/Portable/SemanticModelWorkspaceService/SemanticModelWorkspaceServiceFactory.cs (2 lines): - line 401: // TODO: all this could go away if this is maintained by project itself and one can just get the map from it. - line 472: // TODO: Remove this check and add logic to fetch the #load'ed tree's src/Compilers/CSharp/Portable/Syntax/SyntaxFactory.cs (2 lines): - line 1172: // TODO: [RobinSedlaczek] It is no compiler hot path here I think. But the contribution guide - line 1603: // TODO: If this becomes a real API, we'll need to add an offset parameter to src/Compilers/Core/CodeAnalysisTest/FileUtilitiesTests.cs (2 lines): - line 97: // TODO (tomat): long UNC paths - line 151: // TODO (tomat): src/Features/CSharp/Portable/ExtractMethod/CSharpMethodExtractor.CSharpCodeGenerator.CallSiteContainerRewriter.cs (2 lines): - line 100: // TODO : think about a way to move the trivia to next token. - line 110: // TODO : fix how it manipulate trivia later src/Compilers/VisualBasic/Portable/Lowering/IteratorRewriter/IteratorRewriter.vb (1 line): - line 96: ' TODO: do we need these here? They are used on the actual iterator method. src/EditorFeatures/CSharpTest/Diagnostics/Suppression/SuppressionTest_FixAllTests.cs (1 line): - line 535: [Fact(Skip = "TODO: File a GitHubIssue for test framework unable to handle multiple projects in solution with same file name.")] src/Compilers/VisualBasic/Portable/Symbols/Source/SourceFile.vb (1 line): - line 335: ' TODO: Dev10 reports error on specific type parts rather than the import src/Workspaces/Core/Portable/Workspace/Solution/MetadataOnlyImage.cs (1 line): - line 43: // TODO: make it to use SerializableBytes.WritableStream rather than MemoryStream so that src/EditorFeatures/CSharpTest/EditAndContinue/CSharpEditAndContinueAnalyzerTests.cs (1 line): - line 217: // TODO: test src/EditorFeatures/Core/Implementation/NavigationBar/NavigationBarController_ModelComputation.cs (1 line): - line 66: // TODO: remove .FirstOrDefault() src/Workspaces/VisualBasic/Portable/CodeGeneration/EventGenerator.vb (1 line): - line 181: ' TODO: Someday support events without as clauses (with parameter lists instead) src/Workspaces/VisualBasic/Portable/Rename/LocalConflictVisitor.vb (1 line): - line 60: ' TODO: fully implement handling of all range vars incl. hiding rules. src/Compilers/VisualBasic/Portable/Symbols/Source/SynthesizedSubmissionConstructorSymbol.vb (1 line): - line 69: ' TODO: report erroneous Int32 src/Scripting/CSharp/CSharpScriptCompiler.cs (1 line): - line 40: // TODO: report diagnostics src/Compilers/VisualBasic/Portable/Syntax/SyntaxNodeExtensions.vb (1 line): - line 101: ' TODO: revise and remove code duplication src/Compilers/VisualBasic/Portable/CodeGen/Optimizer/Optimizer.vb (1 line): - line 24: ' TODO: run other optimizing passes here. src/Compilers/Core/Portable/DiagnosticAnalyzer/AnalyzerExecutor.cs (1 line): - line 1295: // TODO: It is not ideal to create a new descriptor per analyzer exception diagnostic instance. src/Interactive/EditorFeatures/VisualBasic/Interactive/FileSystem/LoadDirectiveCompletionProvider.vb (1 line): - line 10: #If TODO Then ' VB doesn't have LoadDirectiveTrivia defined yet src/Compilers/VisualBasic/Portable/Emit/NoPia/EmbeddedTypesManager.vb (1 line): - line 293: ' TODO: When building debug versions in the IDE, the compiler will insert some extra members src/VisualStudio/Core/Def/Implementation/Library/ObjectBrowser/ObjectList.cs (1 line): - line 732: // TODO: Get itemid src/Compilers/Core/Portable/PEWriter/Core.cs (1 line): - line 23: IEnumerable GetAttributes(EmitContext context); // TODO: consider moving this to IDefinition, we shouldn't need to examine attributes on references. src/Workspaces/Core/Portable/CodeFixes/FixAllOccurrences/FixAllContext.cs (1 line): - line 224: // TODO: We should change this API to be a virtual method, as the class is not sealed. src/Compilers/Core/Portable/MetadataReader/MetadataDecoder.cs (1 line): - line 494: // TODO: I believe refs can be parented by a def tokens too, not common, but. src/Compilers/Core/Portable/EncodedStringText.cs (1 line): - line 212: // TODO: Can this allocation be pooled? src/Features/CSharp/Portable/ExtractMethod/CSharpMethodExtractor.CSharpCodeGenerator.cs (1 line): - line 379: // TODO : think about a way to trivia attached to next token src/Features/VisualBasic/Portable/GenerateType/VisualBasicGenerateTypeService.vb (1 line): - line 59: ' TODO: Code Coverage src/Compilers/VisualBasic/Portable/Symbols/SubstitutedErrorType.vb (1 line): - line 110: ' TODO: Is creating a new list here every time the right approach? Should this be cached? src/Compilers/CSharp/Portable/Compilation/MemberSemanticModel.cs (1 line): - line 1607: // TODO: should we look for the "nearest" argument as a fallback? src/Workspaces/CSharpTest/CodeGeneration/SyntaxGeneratorTests.cs (1 line): - line 2605: // TODO: find way to avoid this src/Compilers/VisualBasic/Portable/Symbols/SymbolExtensions.vb (1 line): - line 47: 'TODO: do we need string s for ByRef, Array, TypeParameter etc? src/Compilers/VisualBasic/Portable/CodeGen/Optimizer/StackScheduler.Rewriter.vb (1 line): - line 128: ' TODO: Why?? src/Tools/Source/CompilerGeneratorTools/Source/VisualBasicSyntaxGenerator/Util/WriteUtils.vb (1 line): - line 485: ' TODO: need way to get the ordering right. src/VisualStudio/Core/Def/Implementation/Venus/ContainedLanguage.cs (1 line): - line 74: // TODO: Can contained documents be linked or shared? src/Features/Core/Portable/DocumentHighlighting/AbstractDocumentHighlightsService.cs (1 line): - line 228: // TODO: Remove this check and add logic to fetch the #load'ed tree's src/EditorFeatures/VisualBasicTest/Formatting/Indentation/SmartIndenterTests.vb (1 line): - line 2227: ' TODO: Fix this to indent relative to the previous statement, src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_StringConcat.cs (1 line): - line 246: // TODO: if transient string allocations are an issue, consider introducing constants that contain builders. src/Features/VisualBasic/Portable/Completion/KeywordRecommenders/Declarations/OverloadableOperatorRecommender.vb (1 line): - line 34: ' TODO: fix parser to actually deal with such stupidities like "Operator Narrowing" src/Compilers/VisualBasic/Portable/BoundTree/BoundNodes.xml (1 line): - line 978: TODO: Note - in VB variable's extent is the whole method and can be larger than its scope. src/VisualStudio/CSharp/Impl/ObjectBrowser/DescriptionBuilder.cs (1 line): - line 260: // TODO: 'new' modifier isn't exposed on symbols. Do we need it? src/Features/Core/Portable/ReplacePropertyWithMethods/ReplacePropertyWithMethodsCodeRefactoringProvider.cs (1 line): - line 195: // TODO: add things to be concerned about here. For example: src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_Event.cs (1 line): - line 71: /// TODO: use or delete isDynamic. src/Workspaces/Core/Portable/Execution/Serializer.cs (1 line): - line 204: // TODO: convert this to sub class rather than using enum with if statement. src/Compilers/Core/Portable/PEWriter/PooledBlobBuilder.cs (1 line): - line 24: // TODO: use size src/VisualStudio/Core/Def/Implementation/TableDataSource/Suppression/VisualStudioDiagnosticListTableCommandHandler.cs (1 line): - line 34: // TODO: Figure out how to access menu commands registered by CodeAnalysisPackage and src/Features/Core/Portable/SolutionCrawler/IdleProcessor.cs (1 line): - line 68: // TODO: will safestart/unwarp capture cancellation exception? src/Compilers/CSharp/Portable/Symbols/TypeSymbolExtensions.cs (1 line): - line 305: // TODO: there must be a better way! src/EditorFeatures/CSharpTest/SignatureHelp/InvocationExpressionSignatureHelpProviderTests.cs (1 line): - line 330: // TODO: Once we do the work to allow extension methods in nested types, we should change this. src/Compilers/Core/Portable/DocumentationComments/DocumentationCommentIncludeCache.cs (1 line): - line 14: // TODO: tune src/EditorFeatures/VisualBasicTest/Completion/CompletionProviders/NamespaceContextTests.vb (1 line): - line 7: ' TODO: consider merging these tests with the keyword recommending tests in some way src/Compilers/VisualBasic/Portable/Symbols/Metadata/PE/PEAssemblySymbol.vb (1 line): - line 96: ' TODO: https://github.com/dotnet/roslyn/issues/9000 src/Compilers/VisualBasic/Portable/Symbols/Source/UnboundLambdaParameterSymbol.vb (1 line): - line 81: 'TODO: Should report the following error for Option Strict, but not here, after inference. src/VisualStudio/Core/Impl/Options/OptionPreviewControl.xaml (1 line): - line 42: src/Features/Core/Portable/SolutionCrawler/WorkCoordinator.SemanticChangeProcessor.cs (1 line): - line 93: // TODO: if there is a reliable way to track changed member, we could use GetSemanticModel here which could src/Compilers/VisualBasic/Portable/SymbolDisplay/ObjectDisplay.vb (1 line): - line 360: ' TODO: consider making "token" returned by this function a structure to abstract bit masking operations src/EditorFeatures/VisualBasicTest/EditAndContinue/ActiveStatementTests.vb (1 line): - line 258: ' TODO (bug 755959): better deleted active statement span src/Compilers/CSharp/Portable/Emitter/NoPia/EmbeddedTypesManager.cs (1 line): - line 346: // TODO: When building debug versions in the IDE, the compiler will insert some extra members src/Workspaces/Core/Portable/LanguageServices/SyntaxFactsService/AbstractDocumentationCommentService.cs (1 line): - line 64: // TODO: Consider unifying code to extract text from an Xml Documentation Comment (https://github.com/dotnet/roslyn/issues/2290) src/Workspaces/Core/Portable/Differencing/Match.cs (1 line): - line 205: // TODO (tomat): For nodes tied to their parents, src/Compilers/CSharp/Portable/Lowering/SynthesizedSubmissionFields.cs (1 line): - line 79: // TODO: generate better name? src/VisualStudio/Core/Def/Implementation/ProjectSystem/DocumentProvider.cs (1 line): - line 676: // TODO: This was previously faster, need a bidirectional 1-to-many map src/ExpressionEvaluator/Core/Source/ExpressionCompiler/PDB/MethodDebugInfo.Portable.cs (1 line): - line 371: // TODO: consider looking this up once per module (not for every method) src/Interactive/EditorFeatures/Core/Extensibility/Interactive/ResetInteractive.cs (1 line): - line 103: // TODO: load context from an rsp file. src/EditorFeatures/Core.Wpf/InlineRename/AbstractEditorInlineRenameService.cs (1 line): - line 120: // Cannot rename constructors in VB. TODO: this logic should be in the VB subclass of this type. src/Compilers/CSharp/Portable/Symbols/Metadata/PE/MetadataDecoder.cs (1 line): - line 49: // TODO (tomat): if the containing assembly is a source assembly and we are about to decode assembly level attributes, we run into a cycle, src/Compilers/VisualBasic/Portable/Compilation/SyntaxTreeSemanticModel.vb (1 line): - line 1036: ' TODO: consider always creating a (missing) block around the statements in the parser src/EditorFeatures/VisualBasic/RenameTracking/RenameTrackingCodeFixProvider.vb (1 line): - line 10: ' TODO: Remove the ExtensionOrder attributes once a better ordering mechanism is available src/Tools/Source/MetadataVisualizer/Program.cs (1 line): - line 383: // TODO: handle nested types src/Features/Core/Portable/SignatureHelp/SignatureHelpItem.cs (1 line): - line 26: // TODO: This probably won't be sufficient for VB query signature help. It has src/Compilers/VisualBasic/Portable/Symbols/InstanceTypeSymbol.vb (1 line): - line 176: ' TODO (tomat): use-site errors should be reported on each part of a qualified name, src/Compilers/VisualBasic/Portable/OptionsValidator.vb (1 line): - line 74: ' TODO: consider calling Syntax.ParseName and analyze the result (reject () or src/Compilers/Core/MSBuildTask/AssemblyResolution.cs (1 line): - line 12: // TODO: Remove once https://github.com/Microsoft/msbuild/issues/1309 is fixed. src/Compilers/CSharp/Portable/Binder/Semantics/Operators/BinaryOperatorOverloadResolution.cs (1 line): - line 564: // TODO (tomat): The spec needs to be updated to use identity conversion instead of type equality. src/Workspaces/Core/Portable/Workspace/Workspace.cs (1 line): - line 774: // TODO: make it public src/VisualStudio/Core/Def/xlf/ServicesVSResources.es.xlf (1 line): - line 734: TODO EN MAYÚSCULAS src/Features/CSharp/Portable/EditAndContinue/TopSyntaxComparer.cs (1 line): - line 173: // TODO: src/Compilers/VisualBasic/Portable/Semantics/TypeInference/TypeInferenceCollection.vb (1 line): - line 589: ' TODO: should we simply get out of the loop here? For some reason Dev10 continues, I guess it verifies uniqueness this way. src/ExpressionEvaluator/VisualBasic/Source/ExpressionCompiler/Symbols/ObjectIdLocalSymbol.vb (1 line): - line 87: ' TODO: refs are available src/Compilers/Core/Portable/MetadataReader/ModuleExtensions.cs (1 line): - line 118: // TODO: Keep some record of vtable gaps (DevDiv #17472). src/Compilers/Core/Portable/PEWriter/DebugDirectoryExtensions.cs (1 line): - line 10: // TODO: move to SRM: https://github.com/dotnet/roslyn/issues/24712 src/Compilers/VisualBasic/Portable/Scanner/Blender.vb (1 line): - line 409: ' TODO: This is a temporary measure to get around contextual errors. src/VisualStudio/Core/Def/Implementation/ProjectSystem/IVisualStudioWorkspaceHost.cs (1 line): - line 9: // TODO: Find a better name for this src/Features/Core/Portable/SolutionCrawler/AggregateIncrementalAnalyzer.cs (1 line): - line 60: // TODO: Is this correct? src/Features/Core/Portable/SolutionCrawler/WorkCoordinator.IncrementalAnalyzerProcessor.cs (1 line): - line 241: // TODO: manage bad workers like what code actions does now src/Workspaces/VisualBasic/Portable/Simplification/VisualBasicSimplificationService.Expander.vb (1 line): - line 709: ' TODO: use and expand Generate*Syntax(isymbol) to not depend on symbol display any more. src/EditorFeatures/Core/Shared/Extensions/IBufferGraphExtensions.cs (1 line): - line 66: // TODO (https://github.com/dotnet/roslyn/issues/5281): Remove try-catch. src/Compilers/VisualBasic/Portable/Lowering/AsyncRewriter/AsyncRewriter.vb (1 line): - line 406: ' TODO: Do we want to extend support of this constant src/Compilers/VisualBasic/Portable/Syntax/InternalSyntax/IdentifierTokenSyntax.vb (1 line): - line 42: ' TODO: do we need IdentifierText? src/Interactive/Features/Interactive/Core/InteractiveHost.RemoteService.cs (1 line): - line 40: // TODO (tomat): consider using single-thread async readers src/Compilers/VisualBasic/Portable/Lowering/SynthesizedSubmissionFields.vb (1 line): - line 53: ' TODO (tomat): Dim hostObjectTypeSymbol = compilation.GetHostObjectTypeSymbol() src/VisualStudio/Core/Impl/Options/GridOptionPreviewControl.xaml.cs (1 line): - line 62: // TODO: make the combo to drop down on space or some key. src/EditorFeatures/Core.Wpf/EditAndContinue/ActiveStatementTagFormatDefinition.cs (1 line): - line 19: // TODO (tomat): bug 777271 src/Scripting/CoreTest.Desktop/MetadataShadowCopyProviderTests.cs (1 line): - line 17: // TODO: clean up and move to portable tests src/Compilers/VisualBasic/Portable/Errors/Errors.vb (1 line): - line 1186: ERR_OptionalsCantBeStructGenericParams = 32124 'TODO: remove src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_UsingStatement.cs (1 line): - line 88: Debug.Assert(node.Locals.IsEmpty); // TODO: This might not be a valid assumption in presence of semicolon operator. src/Compilers/Core/Portable/PEWriter/TypeNameSerializer.cs (1 line): - line 116: // TODO: error src/Compilers/VisualBasic/Portable/Symbols/Metadata/PE/MetadataDecoder.vb (1 line): - line 53: ' TODO (tomat): if the containing assembly is a source assembly and we are about to decode assembly level attributes, we run into a cycle, src/Compilers/CSharp/Portable/Binder/Imports.cs (1 line): - line 327: // TODO (https://github.com/dotnet/roslyn/issues/5517): skip namespace expansion if references haven't changed. src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LoweredDynamicOperation.cs (1 line): - line 65: // TODO (tomat): we might be able to use SiteInvocation.Type instead of resultType once we stop using GetLoweredType src/Compilers/VisualBasic/Portable/Symbols/NamespaceOrTypeSymbol.vb (1 line): - line 207: ' TODO: Should we check whether nameSet already contains pair.Key and src/Compilers/CSharp/Portable/Binder/Binder_Symbols.cs (1 line): - line 1549: // TODO: this is not an appropriate error message here, but used as a fallback until the build/Targets/SourcePackage.targets (1 line): - line 42: src/Compilers/Core/Portable/CodeGen/LocalDefinition.cs (1 line): - line 13: //TODO: locals are really just typed slots. They do not have names. src/ExpressionEvaluator/Core/Source/ExpressionCompiler/ILSpan.cs (1 line): - line 9: // TODO: use TextSpan? src/Compilers/Core/Portable/CommandLine/CommonCompiler.cs (1 line): - line 395: // TODO: This invariant fails (at least) in the case where we see a member declaration "x = 1;". src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_ConditionalExpressions.vb (1 line): - line 189: ' TODO: Checking type equality of test and else is not strictly needed. src/EditorFeatures/Core/Shared/Utilities/LinkedEditsTracker.cs (1 line): - line 50: // TODO: custom tracking spans! src/Compilers/CSharp/Portable/CommandLine/CSharpCommandLineParser.cs (1 line): - line 1314: // TODO: set from /checksumalgorithm (see https://github.com/dotnet/roslyn/issues/24735) src/Compilers/VisualBasic/Portable/Compilation/LookupSymbolsInfo.vb (1 line): - line 10: ' TODO: tune pool size src/Workspaces/Remote/ServiceHub/Shared/ServiceHubServiceBase.cs (1 line): - line 16: // TODO: all service hub service should be extract to interface so that it can support multiple hosts. src/VisualStudio/Core/Def/Implementation/AbstractOleCommandTarget.Execute.cs (1 line): - line 558: /// TODO: Revert the change to use standard VS11 command pending https://github.com/dotnet/roslyn/issues/8927 . src/Compilers/VisualBasic/Portable/Symbols/PropertySymbol.vb (1 line): - line 454: ' TODO: Should CustomModifiers be used from this property src/Workspaces/CSharp/Portable/Formatting/FormattingHelpers.cs (1 line): - line 15: // TODO: Need to determine correct way to handle newlines src/VisualStudio/VisualBasic/Impl/Debugging/LocationInfoGetter.vb (1 line): - line 14: ' TODO: Make this class static when we add that functionality to VB. src/Features/Core/Portable/GenerateMember/GenerateVariable/AbstractGenerateVariableService.CodeAction.cs (1 line): - line 191: // TODO: Code coverage - we need a unit-test that generates across projects src/Compilers/CSharp/Portable/Symbols/NoPiaMissingCanonicalTypeSymbol.cs (1 line): - line 17: // TODO: Should probably inherit from MissingMetadataType.TopLevel, but review TypeOf checks for MissingMetadataType. src/Workspaces/Core/Portable/Utilities/AbstractSpeculationAnalyzer.cs (1 line): - line 311: // TODO: Lambda function comparison performs syntax equality, hence is non-trivial to compare lambda methods across different compilations. src/Compilers/VisualBasic/Portable/Binding/SyntheticBoundTrees/AnonymousTypeSyntheticMethods.vb (1 line): - line 267: ' TODO: Make sure we don't call GetObjectValue here src/Compilers/VisualBasic/Portable/Symbols/NonMissingModuleSymbol.vb (1 line): - line 89: ' TODO (tomat): we should report an error for all unified references, not just the first one. src/Workspaces/Remote/Core/Services/SolutionService.cs (1 line): - line 13: /// TODO: change this to workspace service src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_IsOperator.cs (1 line): - line 51: // TODO: Handle dynamic operand type and target type src/VisualStudio/Core/Impl/CodeModel/ExternalElements/AbstractExternalCodeElement.cs (1 line): - line 137: // TODO: We should be returning the RootCodeModel object here. src/Workspaces/Core/Portable/FindSymbols/FindReferences/FindReferenceCache.cs (1 line): - line 16: // TODO : this can be all moved down to compiler side. src/VisualStudio/Core/Def/Implementation/ChangeSignature/ChangeSignatureDialogViewModel.cs (1 line): - line 187: // TODO: Should probably use original syntax & formatting exactly instead of regenerating here src/Compilers/VisualBasic/Portable/Symbols/Source/SynthesizedEventAccessorSymbol.vb (1 line): - line 24: ' TODO: custom modifiers src/Compilers/CSharp/Portable/Binder/Semantics/Conversions/Conversion.cs (1 line): - line 517: // TODO: update the language reference section number below. src/Compilers/VisualBasic/Portable/Syntax/VisualBasicSyntaxNode.vb (1 line): - line 35: 'TODO: may be eventually not needed src/Compilers/VisualBasic/Portable/Binding/BlockBaseBinder.vb (1 line): - line 67: 'TODO: 6 is sufficiently small, but may need tuning. src/VisualStudio/Core/Def/Implementation/TableDataSource/AbstractTableEntriesSnapshot.cs (1 line): - line 22: // TODO : remove these once we move to new drop which contains API change from editor team src/Compilers/CSharp/Portable/Symbols/Source/SourcePropertySymbol.cs (1 line): - line 22: // TODO (tomat): consider splitting into multiple subclasses/rare data. src/Features/VisualBasic/Portable/ExtractMethod/VisualBasicSelectionValidator.vb (1 line): - line 107: ' TODO : check behavior of control flow analysis engine around exception and exception handling. src/Features/Core/Portable/EditAndContinue/RudeEditKind.cs (1 line): - line 100: // TODO: remove values below src/Compilers/VisualBasic/Portable/Lowering/WithExpressionRewriter.vb (1 line): - line 161: ' TODO: Might need to do some optimization for compiler generated locals. src/Compilers/VisualBasic/Portable/Symbols/Attributes/SourceAttributeData.vb (1 line): - line 20: Private ReadOnly _attributeClass As NamedTypeSymbol ' TODO - Remove attribute class. It is available from the constructor. src/EditorFeatures/VisualBasic/LineCommit/CommitFormatter.vb (1 line): - line 187: ' TODO: remove this in dev14 src/Compilers/CSharp/Portable/Lowering/LambdaRewriter/LambdaRewriter.Analysis.Tree.cs (1 line): - line 446: // TODO: Why don't we drop "this" while lowering if method is static? src/Compilers/Core/Portable/Symbols/TypedConstant.cs (1 line): - line 205: // TODO (tomat): replace with SymbolDisplay src/Scripting/CSharpTest/ScriptTests.cs (1 line): - line 373: #if TODO // https://github.com/dotnet/roslyn/issues/3720 src/Compilers/Core/Portable/Syntax/SyntaxTokenList.cs (1 line): - line 67: // TODO: we could remove the unnecessary builder allocations here and go directly src/Compilers/Core/Portable/Emit/CommonPEModuleBuilder.cs (1 line): - line 127: /// Project level imports (VB only, TODO: C# scripts). src/VisualStudio/VisualBasic/Impl/Debugging/BreakpointResolver.vb (1 line): - line 89: ' TODO: This method can go away once https://roslyn.codeplex.com/workitem/231 is fixed. src/Workspaces/Core/Portable/Diagnostics/DiagnosticAnalysisResultBuilder.cs (1 line): - line 155: // TODO: currently additional file location is not supported. src/Workspaces/Remote/Core/Services/CompilationService.cs (1 line): - line 12: /// TODO: change this to workspace service src/Compilers/CSharp/Portable/Symbols/TypeUnification.cs (1 line): - line 190: // TODO: Calling CanUnifyHelper for the containing type is an overkill, we simply need to go through type arguments for all containers. src/Features/Core/Portable/CodeFixes/NamingStyle/AbstractNamingStyleCodeFixProvider.cs (1 line): - line 43: // TODO: We should always be able to find the symbol that generated this diagnostic, src/Compilers/VisualBasic/Portable/Lowering/AsyncRewriter/AsyncRewriter.AsyncMethodToClassRewriter.Spilling.vb (1 line): - line 345: ' TODO: do we want to do that for all/some other nodes with const values? src/Compilers/CSharp/Portable/Symbols/Source/SourceFieldSymbol.cs (1 line): - line 132: // TODO: Consider checking presence of core type System.Runtime.CompilerServices.IsVolatile src/VisualStudio/Core/Def/Implementation/ProjectSystem/AbstractProject.cs (1 line): - line 969: // TODO: Venus does this for .aspx/.cshtml files which is completely unnecessary for Roslyn. We should remove that code. src/Compilers/Core/Portable/Symbols/Attributes/AttributeDescription.cs (1 line): - line 194: // TODO: We should reuse the byte arrays for well-known attributes with same signatures. src/EditorFeatures/VisualBasicTest/Completion/CompletionProviders/ObjectInitializerCompletionProviderTests.vb (1 line): - line 22: ' TODO: Re-enable these when interactive is back in the product. src/Compilers/Core/Portable/Diagnostic/DiagnosticInfo.cs (1 line): - line 384: // TODO (tomat): remove src/Compilers/CSharp/Portable/Binder/LocalScopeBinder.cs (1 line): - line 499: // TODO: Handle case where 'name' exists in both localsMap and localFunctionsMap. Right now locals are preferred over local functions. src/Features/Core/Portable/SolutionCrawler/WorkCoordinator.AsyncDocumentWorkItemQueue.cs (1 line): - line 103: // TODO: should I care about language when replace it? src/Features/Core/Portable/CodeRefactorings/MoveType/AbstractMoveTypeService.RenameTypeEditor.cs (1 line): - line 32: // TODO: detect conflicts ahead of time and open an inline rename session if any exists. src/Workspaces/Core/Portable/FindSymbols/FindReferences/Finders/LinkedFileReferenceFinder.cs (1 line): - line 25: // TODO: Remove this check and add logic to fetch the #load'ed tree's src/Compilers/CSharp/Portable/FlowAnalysis/DataFlowPass.cs (1 line): - line 1854: // TODO: in a struct constructor, "this" is not initially assigned. src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_AsOperator.cs (1 line): - line 30: // TODO: Handle dynamic operand type and target type src/Compilers/VisualBasic/Portable/Semantics/Operators.vb (1 line): - line 3234: ' TODO: Need to get rid of dependency on method group. For now create fake one. src/Workspaces/Core/Portable/Shared/Utilities/BloomFilter_Serialization.cs (1 line): - line 67: // TODO: find a way to use pool src/Workspaces/VisualBasic/Portable/Classification/ClassificationHelpers.vb (1 line): - line 123: ' TODO: Do we need to do the same work here that we do in C#? src/Compilers/CSharp/Portable/Binder/LookupSymbolsInfo.cs (1 line): - line 12: // TODO: tune pool size. src/Compilers/CSharp/Portable/Parser/LexerCache.cs (1 line): - line 80: // TODO: remove this when done tweaking this cache. src/Compilers/VisualBasic/Portable/Preprocessor/TypeHelpers.vb (1 line): - line 16: ' TODO: figure how to do this in VB. src/Workspaces/Core/Portable/Workspace/Solution/SolutionState.CompilationTracker.cs (1 line): - line 667: // TODO: Record source assembly to project mapping src/Features/CSharp/Portable/Completion/CompletionProviders/SnippetCompletionProvider.cs (1 line): - line 52: // TODO (https://github.com/dotnet/roslyn/issues/5107): Enable in Interactive. src/VisualStudio/Core/Def/Implementation/Snippets/AbstractSnippetCommandHandler.cs (1 line): - line 303: // TODO (https://github.com/dotnet/roslyn/issues/5107): enable in interactive src/Compilers/VisualBasic/Portable/Preprocessor/CConst.vb (1 line): - line 240: ' TODO: we support only one error for now. src/ExpressionEvaluator/VisualBasic/Source/ExpressionCompiler/EvaluationContext.vb (1 line): - line 316: ' TODO: this can be done better src/EditorFeatures/Core/Implementation/NavigationBar/NavigationBarController.cs (1 line): - line 372: // TODO: navigate to document / focus text view src/Compilers/CSharp/Portable/Symbols/Metadata/PE/PENamespaceSymbol.cs (1 line): - line 97: // TODO - Eliminate the copy by storing all members and type members instead of non-type and type members? src/Compilers/VisualBasic/Portable/Analysis/InitializerRewriter.vb (1 line): - line 63: ' TODO: this can be skipped if we have equal number of initializers and fields src/Workspaces/CSharp/Portable/Simplification/CSharpSimplificationService.Expander.cs (1 line): - line 940: //// TODO: use and expand Generate*Syntax(isymbol) to not depend on symbol display any more. src/Features/VisualBasic/Portable/ExtractMethod/VisualBasicMethodExtractor.VisualBasicCodeGenerator.vb (1 line): - line 250: ' TODO : think about a way to trivia attached to next token src/Compilers/VisualBasic/Portable/Lowering/MethodToClassRewriter/MethodToClassRewriter.vb (1 line): - line 415: 'TODO: we may not need to update if there was nothing to rewrite. src/EditorFeatures/CSharpTest/SignatureHelp/GenericNameSignatureHelpProviderTests.cs (1 line): - line 555: // TODO: Enable the script case when we have support for extension methods in scripts src/EditorFeatures/Core/Implementation/EditAndContinue/ActiveStatementTagger.cs (1 line): - line 38: // TODO: call the handler only if the spans affect this buffer src/Workspaces/Core/Portable/Workspace/Workspace_Editor.cs (1 line): - line 327: // TODO: remove linear search src/VisualStudio/Core/Def/Implementation/ProjectSystem/MetadataReferences/VisualStudioMetadataReferenceManager.Factory.cs (1 line): - line 13: // TODO: Remove this type. This factory is needed just to instantiate a singleton of VisualStudioMetadataReferenceProvider. src/Compilers/VisualBasic/Portable/Symbols/Source/SynthesizedMyGroupCollectionPropertyAccessorSymbol.vb (1 line): - line 69: ' TODO: It looks like Dev11 respects project level conditional compilation here. src/Compilers/VisualBasic/Portable/Symbols/ExtendedErrorTypeSymbol.vb (1 line): - line 86: _resultKind = LookupResultKind.NotATypeOrNamespace ' TODO: Replace. src/VisualStudio/CSharp/Impl/ProjectSystemShim/CSharpProjectShim.cs (1 line): - line 172: // TODO: appConfigPath: GetFilePathOption(CompilerOptions.OPTID_FUSIONCONFIG), bug #869604 src/VisualStudio/Core/Test.Next/Services/ServiceHubServicesTests.cs (1 line): - line 72: var code = @"// TODO: Test"; src/Compilers/CSharp/Portable/Binder/Binder_QueryErrors.cs (1 line): - line 146: // TODO: refine this test to check if the identifier is the name of a range src/Compilers/CSharp/Portable/Parser/LanguageParser_InterpolatedString.cs (1 line): - line 152: // TODO: some of the trivia in the interpolation maybe should be trailing trivia of the openBraceToken src/Compilers/CSharp/Portable/CSharpCompilationOptions.cs (1 line): - line 591: // TODO: add check for src/ExpressionEvaluator/Core/Source/ResultProvider/Helpers/TypeAndCustomInfo.cs (1 line): - line 28: //TODO: Sometimes we get byref types here when dealing with ref-returning members. src/VisualStudio/VisualBasic/Impl/Debugging/DataTipInfoGetter.vb (1 line): - line 13: ' TODO: Make this class static when we add that functionality to VB. src/EditorFeatures/Core/Implementation/TodoComment/TodoCommentIncrementalAnalyzer.cs (1 line): - line 149: // TODO let's think about what to do here. for now, let call it synchronously. also, there is no actual async-ness for the src/Compilers/CSharp/Portable/Binder/WithTypeParametersBinder.cs (1 line): - line 17: // TODO: Change this to a data structure that won't allocate enumerators src/Compilers/CSharp/Portable/Lowering/AsyncRewriter/AsyncStateMachine.cs (1 line): - line 23: // TODO: report use-site errors on these types src/Compilers/Core/CodeAnalysisTest/MetadataReferences/AssemblyIdentityDisplayNameTests.cs (1 line): - line 546: // TODO: need to calculate the correct token for the ECMA key. src/Features/Core/Portable/Diagnostics/EngineV2/DiagnosticIncrementalAnalyzer.StateSet.cs (1 line): - line 263: // TODO: we do this since InMemoryCache is static type. we might consider making it instance object src/Workspaces/CSharp/Portable/LanguageServices/CSharpSyntaxFactsService.cs (1 line): - line 1015: // TODO: currently we only support method for now src/Scripting/CoreTestUtilities/TestCompilationFactory.cs (1 line): - line 11: // TODO: we need to clean up and refactor CreateCompilationWithMscorlib in compiler tests src/Features/VisualBasic/Portable/Completion/KeywordRecommenders/Declarations/ImportsKeywordRecommender.vb (1 line): - line 24: ' TODO: does this break our rule of not looking forward? src/Features/Core/Portable/RQName/Nodes/RQTypeOrNamespace.cs (1 line): - line 21: // TODO: C# Specific? src/Features/VisualBasic/Portable/Structure/Providers/DocumentationCommentStructureProvider.vb (1 line): - line 25: ' TODO: Need to redo this when DocumentationCommentTrivia.SpanStart points to the start of the exterior trivia. src/Compilers/VisualBasic/Portable/Syntax/SyntaxFactory.vb (1 line): - line 898: ' TODO: [RobinSedlaczek] It is no compiler hot path here I think. But the contribution guide src/EditorFeatures/Core/Implementation/TodoComment/TodoCommentOptions.cs (1 line): - line 12: public static readonly Option TokenList = new Option(nameof(TodoCommentOptions), nameof(TokenList), defaultValue: "HACK:1|TODO:1|UNDONE:1|UnresolvedMergeConflict:0"); src/Features/Core/Portable/ReplaceMethodWithProperty/ReplaceMethodWithPropertyCodeRefactoringProvider.cs (1 line): - line 456: // TODO: add things to be concerned about here. For example: src/Compilers/Core/Portable/PEWriter/PeWriter.cs (1 line): - line 142: // TODO: replace SAH1 with non-crypto alg: https://github.com/dotnet/roslyn/issues/24737 src/Compilers/Core/Portable/DiagnosticAnalyzer/AnalyzerFileReference.cs (1 line): - line 421: // TODO: Once we move to CoreCLR we should just call GetType(typeName, throwOnError: true, ignoreCase: false) directly. src/Workspaces/VisualBasic/Portable/Extensions/SyntaxTreeExtensions.vb (1 line): - line 167: ' TODO : insert point at the same line as preprocessor? src/Compilers/CSharp/Portable/Lowering/LocalRewriter/LocalRewriter_MultipleLocalDeclarations.cs (1 line): - line 38: return null; // TODO: but what if hasErrors? Have we lost that? src/Compilers/Core/Portable/Emit/NoPia/CommonEmbeddedMethod.cs (1 line): - line 209: // TODO: src/Compilers/Core/Portable/Diagnostic/DiagnosticDescriptor.cs (1 line): - line 213: // TODO: Once https://github.com/dotnet/roslyn/issues/3650 is fixed, we can avoid creating a no-location diagnostic here. src/Compilers/VisualBasic/Portable/Lowering/StateMachineRewriter/StateMachineRewriter.vb (1 line): - line 272: ' TODO: so, we have to check if it is already in or not. See the early impl. src/EditorFeatures/Core/CommandHandlers/GoToAdjacentMemberCommandHandler.cs (1 line): - line 129: // TODO: Better position within the node (e.g. attributes?) src/Features/CSharp/Portable/ChangeSignature/CSharpChangeSignatureService.cs (1 line): - line 185: // TODO: file bug about this: var invocation = csnode.Ancestors().FirstOrDefault(a => a.Kind == SyntaxKind.InvocationExpression); src/VisualStudio/VisualBasic/Impl/LanguageService/VisualBasicPackage.IVbCompiler.vb (1 line): - line 62: ' TODO: It's not clear what the compiler should do with Watson information at this point. src/Features/Core/Portable/LanguageServices/SymbolDisplayService/AbstractSymbolDisplayService.AbstractSymbolDescriptionBuilder.cs (1 line): - line 486: // TODO : show duplicated member case src/Features/Core/Portable/GenerateMember/GenerateVariable/AbstractGenerateVariableService.State.cs (1 line): - line 111: // TODO: Code coverage src/Compilers/CSharp/Portable/Lowering/SynthesizedMethodBaseSymbol.cs (1 line): - line 59: // TODO: move more functionality into here, making these symbols more lazy src/Features/Core/Portable/CodeFixes/GenerateMember/AbstractGenerateMemberCodeFixProvider.cs (1 line): - line 21: // TODO: https://github.com/dotnet/roslyn/issues/5777 src/Compilers/CSharp/Portable/Syntax/SyntaxKindFacts.cs (1 line): - line 152: // TODO: What about "<>f_AnonymousMethod"? Or "123#"? What's this used for? src/VisualStudio/Core/Def/Implementation/ProjectSystem/LinkedFileUtilities.cs (1 line): - line 196: // TODO: we need better way to see whether a project is project k project or not. src/EditorFeatures/Core.Wpf/InlineRename/AbstractEditorInlineRenameService.SymbolRenameInfo.cs (1 line): - line 88: return triggerText.StartsWith(triggerText); // TODO: Always true? What was it supposed to do? src/VisualStudio/Core/Def/Implementation/Library/ObjectBrowser/AbstractObjectBrowserLibraryManager_Search.cs (1 line): - line 43: // TODO: Support wildcards (e.g. *xyz, *xyz* and xyz*) like the old language service did. src/Compilers/VisualBasic/Portable/Binding/BinderFactory.vb (1 line): - line 166: ' TODO (tomat): this is just a simple temporary solution, we'll need to plug-in submissions, interactive imports and host object members: src/Scripting/VisualBasicTest/ScriptTests.vb (1 line): - line 69: ' TODO: port C# tests src/Compilers/VisualBasic/Portable/Binding/SpeculativeBinder.vb (1 line): - line 38: ' TODO override SyntaxTree property to return correct tree. (after e.g. bugs 2174, 5848) src/Compilers/Core/Portable/Collections/SmallDictionary.cs (1 line): - line 48: // TODO: if comparers are same (often they are), then just need to clone the tree. src/Compilers/VisualBasic/Portable/Symbols/Metadata/PE/PEModuleSymbol.vb (1 line): - line 137: 'TODO - Create a Module.Token to return token similar to Assembly.Token src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_Conversion.vb (1 line): - line 914: ' TODO: A chain of widening reference conversions that starts from NOTHING literal can be collapsed to a single node. src/VisualStudio/Core/Def/Implementation/CallHierarchy/CallHierarchyItem.cs (1 line): - line 107: // TODO: Use Dustin's find-references-from-symbol service. src/VisualStudio/Core/Def/Implementation/Library/ObjectBrowser/AbstractObjectBrowserLibraryManager.cs (1 line): - line 422: // TODO: Make sure we pass the right value for Visual Basic. src/Workspaces/Core/Desktop/Workspace/MSBuild/ProjectFile/ProjectFile.cs (1 line): - line 441: // TODO (tomat): should we report an error when drive-relative path (e.g. "C:goo.cs") is encountered? src/Features/CSharp/Portable/Structure/Providers/MetadataAsSource/AbstractMetadataAsSourceStructureProvider.cs (1 line): - line 36: // TODO (tomescht): Mark the regions to be collapsed by default. src/Compilers/CSharp/Portable/Compilation/MemberSemanticModel.NodeMapBuilder.cs (1 line): - line 73: // TODO: it would be great if we could check !ReferenceEquals(existing[i], added[i]) (DevDiv #11584). src/Compilers/Core/Portable/MetadataReader/MetadataTypeName.cs (1 line): - line 57: /// TODO: PERF: Encode this field elsewhere to save 4 bytes src/Compilers/VisualBasic/Portable/Analysis/FlowAnalysis/DataFlowsInWalker.vb (1 line): - line 20: ' TODO: normalize the result by removing variables that are unassigned in an unmodified flow analysis. src/EditorFeatures/Test2/Rename/RenameCommandHandlerTests.vb (1 line): - line 250: ' TODO: should we make tab wait instead? src/Compilers/VisualBasic/Portable/Parser/BlockContexts/SelectBlockContext.vb (1 line): - line 41: ' TODO - davidsch src/VisualStudio/Core/Def/Implementation/Options/LanguageSettingsPersister.cs (1 line): - line 57: // TODO: make this configurable src/VisualStudio/Core/Impl/CodeModel/ExternalElements/ExternalCodeVariable.cs (1 line): - line 134: // TODO: C# checks whether the field Type is generic. What does VB do? src/Features/Core/Portable/ChangeSignature/AbstractChangeSignatureService.cs (1 line): - line 508: // TODO: file bug for var match = originalParameters.FirstOrDefault(p => p.Name == ); src/VisualStudio/Core/Def/Implementation/AbstractVsTextViewFilter`2.cs (1 line): - line 64: // TODO: broken in REPL src/VisualStudio/Core/Def/Implementation/Workspace/VisualStudioTextUndoHistoryWorkspaceServiceFactory.cs (1 line): - line 48: // TODO: Handle undo if context changes src/Compilers/CSharp/Portable/FlowAnalysis/UnassignedVariablesWalker.cs (1 line): - line 54: // TODO: how to handle fields of structs? src/Compilers/Core/Portable/Symbols/ILocalSymbol.cs (1 line): - line 47: // TODO: Add XML doc comment. src/Compilers/VisualBasic/Portable/Binding/MemberSemanticModel.vb (1 line): - line 204: ' TODO: Should these two call be merged into a single one for efficiency? src/Compilers/VisualBasic/Portable/Binding/InitializerSemanticModel.vb (1 line): - line 200: 'TODO - Do parameters need to do anything here? src/Workspaces/Core/Portable/CodeFixes/CodeFixContext.cs (1 line): - line 209: // TODO: src/Workspaces/VisualBasic/Portable/Recommendations/VisualBasicRecommendationService.vb (1 line): - line 270: ' TODO remove this when 531549 which causes leftHandTypeInfo to be an error type is fixed src/EditorFeatures/VisualBasicTest/SymbolId/SymbolKeyTestBase.vb (1 line): - line 149: ' TODO: anonymous type & func src/Features/Core/Portable/GenerateType/AbstractGenerateTypeService.Editor.cs (1 line): - line 642: // TODO: Code coverage src/Workspaces/Core/Portable/Formatting/Context/FormattingContext.cs (1 line): - line 271: // TODO: think about a way to figure out whether it is already suppressed and skip the expensive check below. src/Compilers/Core/Portable/Emit/NoPia/CommonEmbeddedTypeParameter.cs (1 line): - line 189: // TODO: src/Workspaces/Core/Portable/Execution/PinnedRemotableDataScope.cs (1 line): - line 78: /// TODO: currently, this asset must be something can understand src/Workspaces/Core/Portable/Utilities/ExceptionHelpers.cs (1 line): - line 11: // TODO: re-implement this mechanism in a portable way src/Scripting/Core/Hosting/Resolvers/RuntimeMetadataReferenceResolver.cs (1 line): - line 44: // TODO: Look for .winmd, but only if the identity has content WindowsRuntime (https://github.com/dotnet/roslyn/issues/6483) src/Features/VisualBasic/Portable/CodeFixes/SimplifyTypeNames/SimplifyTypeNamesCodeFixProvider.vb (1 line): - line 84: Case IDEDiagnosticIds.PreferIntrinsicPredefinedTypeInDeclarationsDiagnosticId 'TODO use dedicated resource strings? src/Features/Core/Portable/CodeRefactorings/CodeRefactoringService.cs (1 line): - line 131: // TODO: Can we share code between similar lambdas that we pass to this API in BatchFixAllProvider.cs, CodeFixService.cs and CodeRefactoringService.cs? src/Compilers/CSharp/Portable/Symbols/Source/SourceMemberContainerSymbol.cs (1 line): - line 3059: // TODO: can we leave this out of the member list? src/Workspaces/CSharp/Portable/Simplification/Reducers/CSharpEscapingReducer.cs (1 line): - line 117: // TODO: handle crefs and param names of xml doc comments. src/Compilers/VisualBasic/Portable/Emit/SymbolTranslator.vb (1 line): - line 14: ' TODO: Need to estimate amount of elements for this map and pass that value to the constructor. src/Workspaces/Core/Portable/SolutionCrawler/ExportIncrementalAnalyzerProviderAttribute.cs (1 line): - line 20: // TODO: this will be removed once closed side changes are in. src/Workspaces/Core/Desktop/Execution/DesktopReferenceSerializationServiceFactory.cs (1 line): - line 115: // TODO: find out a way to get analyzer assembly location src/VisualStudio/Core/Impl/CodeModel/InternalElements/CodeEvent.cs (1 line): - line 138: // TODO: C# throws E_FAIL but VB throws E_NOTIMPL. src/Compilers/CSharp/Portable/Syntax/CSharpSyntaxNode.cs (1 line): - line 138: // TODO: may be eventually not needed src/EditorFeatures/CSharpTest/EditAndContinue/ActiveStatementTests.Methods.cs (1 line): - line 42: // TODO (bug 755959): better deleted active statement span src/Compilers/CSharp/Portable/Symbols/TypeSymbol.cs (1 line): - line 30: // TODO (tomat): Consider changing this to an empty name. This name shouldn't ever leak to the user in error messages. src/Compilers/VisualBasic/Portable/Compilation/DocumentationComments/DocumentationCommentCompiler.vb (1 line): - line 74: ' TODO: get preferred culture from compilation(?) src/Compilers/Core/Portable/Binding/AbstractLookupSymbolsInfo.cs (1 line): - line 88: // TODO: Is the cost of boxing every instance of UniqueSymbolOrArities that we src/Features/Core/Portable/Diagnostics/EngineV2/DiagnosticIncrementalAnalyzer.StateManager.cs (1 line): - line 250: // TODO: src/Compilers/Core/Portable/Syntax/SyntaxNode.cs (1 line): - line 805: /// TODO: This should probably be reimplemented with src/Compilers/CSharp/Portable/Symbols/Source/SourceEventSymbol.cs (1 line): - line 36: // TODO: CLSCompliantAttribute src/Compilers/VisualBasic/Portable/Lowering/UseTwiceRewriter.vb (1 line): - line 189: ' TODO: Might need to do some optimization for compiler generated locals. src/Compilers/CSharp/Portable/Symbols/AbstractTypeMap.cs (1 line): - line 60: // TODO: we could construct the result's ConstructedFrom lazily by using a "deep" src/Compilers/Core/Portable/Desktop/DesktopAssemblyIdentityComparer.Fx.cs (1 line): - line 14: // TODO: Consider reducing the table memory footprint. src/Features/VisualBasic/Portable/TodoComments/BasicTodoCommentIncrementalAnalyzerProvider.vb (1 line): - line 111: ' TODO: remove this if SyntaxFacts.IsSingleQuote become public src/Compilers/VisualBasic/Portable/Symbols/NamedTypeSymbolExtensions.vb (1 line): - line 18: ' TODO: Should we do a quick check for [Not IsDefinition()]? src/Compilers/VisualBasic/Portable/Parser/BlockContexts/IfPartContext.vb (1 line): - line 11: ' TODO: Rename this to ElseOrElseIfBlockContext. src/Workspaces/Core/Portable/Shared/Utilities/AliasSymbolCache.cs (1 line): - line 26: // TODO: given semantic model must be not speculative semantic model for now. src/VisualStudio/VisualBasic/Impl/Help/VisualBasicHelpContextService.vb (1 line): - line 49: If text.StartsWith("TODO:", StringComparison.CurrentCultureIgnoreCase) Then src/EditorFeatures/CSharpTest/SignatureHelp/GenericNamePartiallyWrittenSignatureHelpProviderTests.cs (1 line): - line 91: // TODO: Enable the script case when we have support for extension methods in scripts src/Compilers/VisualBasic/Portable/Symbols/TypeSymbol.vb (1 line): - line 28: ' TODO (tomat): Consider changing this to an empty name. This name shouldn't ever leak to the user in error messages. src/Compilers/Core/Portable/CodeGen/EmitState.cs (1 line): - line 66: //TODO: for debugging we could also record what we have in the stack (I, F, O, &, ...) src/VisualStudio/VisualBasic/Impl/Progression/VisualBasicProgressionLanguageService.vb (1 line): - line 18: ' TODO: Implement this lazily like in C#? src/VisualStudio/CSharp/Impl/ProjectSystemShim/CSharpProjectShim.ICSharpProjectSite.cs (1 line): - line 62: // TODO: uncomment when fixing https://github.com/dotnet/roslyn/issues/5325 src/EditorFeatures/VisualBasicTest/Completion/CompletionProviders/TypeContextTests.vb (1 line): - line 7: ' TODO: consider merging these tests with the keyword recommending tests in some way src/Compilers/VisualBasic/Portable/Utilities/FunctionExtensions.vb (1 line): - line 12: 'TODO - This is copied from C# and should be moved to common assemble. src/Compilers/Core/Portable/CodeGen/PrivateImplementationDetails.cs (1 line): - line 268: // TODO: replace SAH1 with non-crypto alg: https://github.com/dotnet/roslyn/issues/24737 src/VisualStudio/Core/Def/Implementation/TaskList/ProjectExternalErrorReporter.cs (1 line): - line 155: // TODO: Use PreserveSig instead of throwing these exceptions for common cases. src/Compilers/VisualBasic/Portable/Utilities/TypeUnification.vb (1 line): - line 145: 'TODO: Calling CanUnifyHelper for the containing type is an overkill, we simply need to go through type arguments for all containers. src/Scripting/CSharpTest.Desktop/ObjectFormatterTests.cs (1 line): - line 85: // TODO: move to portable src/Compilers/CSharp/Portable/BoundTree/UnboundLambda.cs (1 line): - line 555: HashSet useSiteDiagnostics = null; // TODO: figure out if this should be somehow merged into BoundLambda.Diagnostics. src/Workspaces/Core/Portable/SymbolKey/SymbolKey.BodyLevelSymbolKey.cs (1 line): - line 85: // TODO (rchande): It might be better to call compilation.GetSemanticModel src/Compilers/CSharp/Portable/Symbols/Source/ExplicitInterfaceHelpers.cs (1 line): - line 62: // TODO: Revisit how explicit interface implementations are named. src/Workspaces/VisualBasic/Portable/LanguageServices/VisualBasicCompilationFactoryService.vb (1 line): - line 29: #If TODO Then ' https://github.com/dotnet/roslyn/issues/9063 src/Features/Core/Portable/GenerateDefaultConstructors/GenerateDefaultConstructorsCodeRefactoringProvider.cs (1 line): - line 33: // TODO: https://github.com/dotnet/roslyn/issues/5778 src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_StringConcat.vb (1 line): - line 214: ' TODO: if transient string allocations are an issue, consider introducing constants that contain builders. src/VisualStudio/Core/Def/Implementation/Progression/GraphProvider.cs (1 line): - line 349: // TODO: The check here is to see if the SymbolId property exists on the node src/VisualStudio/Core/Def/Implementation/Preview/FileChange.cs (1 line): - line 236: // TODO: it would be nice to have a syntax based differ for presentation here, src/Compilers/CSharp/Portable/FlowAnalysis/ControlFlowPass.cs (1 line): - line 179: // TODO: if in the body of a struct constructor, check that "this" is assigned at each return. src/Compilers/VisualBasic/Portable/Symbols/Attributes/PEAttributeData.vb (1 line): - line 21: Private _attributeClass As NamedTypeSymbol ' TODO - Remove class it is available from constructor. For now it is only used to know src/Features/Core/Portable/GenerateType/AbstractGenerateTypeService.State.cs (1 line): - line 387: // TODO: Code coverage src/VisualStudio/Core/Def/Implementation/ExtractInterface/ExtractInterfaceDialogViewModel.cs (1 line): - line 90: // TODO: Deal with filename already existing src/Scripting/Core/ScriptBuilder.cs (1 line): - line 146: // TODO: Should the #r resolver return contract metadata and runtime assembly path - src/EditorFeatures/Core/Shared/Tagging/EventSources/TaggerConstants.cs (1 line): - line 21: // TODO : should we remove TaggerBehavior enum all together and put NearImmediateDelay src/ExpressionEvaluator/Core/Source/ResultProvider/Helpers/TypeHelpers.cs (1 line): - line 728: // TODO: Re-enable commented out check. src/Features/Core/Portable/Shared/Extensions/ProjectExtensions.cs (1 line): - line 7: // TODO: Get the glyph from the hierarchy src/Scripting/VisualBasicTest/ObjectFormatterTests.vb (1 line): - line 96: ' TODO: port tests from C# src/Compilers/CSharp/CSharpAnalyzerDriver/CSharpDeclarationComputer.cs (1 line): - line 294: // TODO: use 'model.GetDeclaredSymbol(expressionBody)' when compiler is fixed to return the getter symbol for it. src/Compilers/Core/Portable/CodeGen/SwitchIntegralJumpTableEmitter.SwitchBucket.cs (1 line): - line 276: // TODO: consider changing threshold bucket density to 33% src/Compilers/VisualBasic/Portable/Lowering/LocalRewriter/LocalRewriter_ForTo.vb (1 line): - line 729: ' TODO: we could hoist (step >> 31) into a temp. Dev10 does not do this. src/Features/Core/Portable/Diagnostics/Log/DiagnosticAnalyzerLogger.cs (1 line): - line 54: // TODO: once we create description manager, pass that into here. src/Features/Core/Portable/MetadataAsSource/MetadataAsSourceHelpers.cs (1 line): - line 55: // TODO (https://github.com/dotnet/roslyn/issues/6859): compilation.GetMetadataReference(assemblySymbol)? src/Compilers/Core/Portable/MetadataReader/TypeNameDecoder.cs (1 line): - line 251: // TODO : lookup in mscorlib src/Compilers/VisualBasic/Portable/Binding/Binder_ObjectInitializer.vb (1 line): - line 338: ' TODO: handle cases of types that are not newable (see type libraries and the src/ExpressionEvaluator/VisualBasic/Source/ExpressionCompiler/Symbols/EELocalConstantSymbol.vb (1 line): - line 74: ' TODO (https://github.com/dotnet/roslyn/issues/1815): report diagnostics if the value is bad src/Compilers/VisualBasic/Portable/Symbols/NoPiaMissingCanonicalTypeSymbol.vb (1 line): - line 16: Inherits ErrorTypeSymbol ' TODO: Should probably inherit from MissingMetadataType.TopLevel, but review TypeOf checks for MissingMetadataType. src/Compilers/Core/Portable/InternalUtilities/StackGuard.cs (1 line): - line 21: // TODO (DevDiv workitem 966425): Replace exception name test with a type test once the type src/Compilers/Core/Portable/DiagnosticAnalyzer/SuppressMessageAttributeState.cs (1 line): - line 268: // TODO: How should we deal with multiple SuppressMessage attributes, with different suppression info/states? src/EditorFeatures/Core/Undo/NoOpGlobalUndoServiceFactory.cs (1 line): - line 28: // TODO: this is technically wrong -- Transaction shouldn't be a singleton. src/EditorFeatures/Core/Extensibility/NavigationBar/NavigationBarProjectItem.cs (1 line): - line 34: // TODO: Can we pass something better? src/Compilers/Core/MSBuildTask/Microsoft.CSharp.Core.targets (1 line): - line 72: