July 17, 2010 [Issue 4480] New: JSON: include location range, not just line numbers, for everything | ||||
---|---|---|---|---|
| ||||
http://d.puremagic.com/issues/show_bug.cgi?id=4480 Summary: JSON: include location range, not just line numbers, for everything Product: D Version: D1 & D2 Platform: All OS/Version: All Status: NEW Severity: normal Priority: P2 Component: DMD AssignedTo: nobody@puremagic.com ReportedBy: dhasenan@gmail.com --- Comment #0 from Chris Wright <dhasenan@gmail.com> 2010-07-17 10:09:50 PDT --- If you include the location range for everything in the json output, it's much easier to implement navigate-to-declaration in an IDE. For example, if there is code: logger.trace(...); json only outputs "line": 18. But if it instead put: "range": { "start": {"line": 18, "col": 4}, "end": {"line": 18, "col": 16}} Then it's easier for an IDE to find the identifier under the cursor. Similarly, IDEs often allow you to "fold" code -- collapse a declaration so you don't have to see it, or collapse a function body or a block. Including a range means the IDE doesn't need to parse D. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
February 24, 2012 [Issue 4480] JSON: include location range, not just line numbers, for everything | ||||
---|---|---|---|---|
| ||||
Posted in reply to Chris Wright | http://d.puremagic.com/issues/show_bug.cgi?id=4480 Matt Peterson <revcompgeek@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |revcompgeek@gmail.com --- Comment #1 from Matt Peterson <revcompgeek@gmail.com> 2012-02-23 20:38:09 PST --- DMD doesn't currently store line offset information, so this is a fairly invasive enhancement to do, although I agree it would be very nice to have this. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
Copyright © 1999-2021 by the D Language Foundation