EE: Comment characters should be ignored in strings and XML literals · dotnet/roslyn#1453

2015-03-20T23:33:05.000Z

Ported from TFS WorkItem: 1120198 --- Repro Steps: 1. F5 the following class C { static object F(string s) { return new object(); } static void Main() { System.Diagnostics.Debugger.Break(); } } 2. Evaluate F("//") in the Watch window 3. Right-click on the expression and "Add Watch" Result: Invalid expression Same issue exists with VB with strings and XML literals: Imports System.Xml.Linq Class C Shared Function F(x As XElement) As Object Return New Object() End Function Shared Sub Main() System.Diagnostics.Debugger.Break() End Sub End Class --- Revisions: 1) Created By Chuck Stoner (2/5/2015 9:01:14 AM) ---

(0 留言) (0 反應) (0 負責人)C# (20,414 star) (4,257 fork)batch import

Area-InteractiveBugInteractive-Debugginghelp wanted

描述

Ported from TFS WorkItem: 1120198

Created By Chuck Stoner (2/5/2015 9:01:14 AM)

貢獻者指南

技術棧: csharp
領域: toolingdeveloper experience
議題類型: bug
難度: 4
預計時間: 1-2 days
活動狀態: stale
清晰度: clear
前置要求: C# or VB knowledgefamiliarity with debugger expression evaluationunderstanding of Roslyn architecture
新手友善度: 15
研究方向: The bug is in the expression evaluator (EE) for the Watch window, which incorrectly treats comment characters (e.g., '//') as comments even when inside string literals or XML literals. To fix, examine the EE's lexer/parser logic in the Roslyn debugger components, likely in the 'src/ExpressionEvaluator' directory. Determine where comment detection occurs and adjust to skip characters inside strings and XML literals. No prior discussion or linked PRs exist for this issue, so the fix can be implemented from scratch.