code-d icon indicating copy to clipboard operation
code-d copied to clipboard

Request - parse dmd exception output to make it compatible with vscode linkify

Open aliak00 opened this issue 6 years ago • 6 comments

The exception output format is e.g.

core.exception.AssertError@tests/optional.d(450): unittest failure

So vscode cannot recognize it as a file link since there's no file called core.exception.AssertError@tests in the cwd.

I wonder if there's anyway code-d can help out here?

aliak00 avatar Sep 20 '18 08:09 aliak00

good idea I can add a link provider but I don't know if it will work for the embedded console

WebFreak001 avatar Sep 20 '18 09:09 WebFreak001

Awesome! Fingers crossed ! And FYI opened this issue for DMD as well -> https://issues.dlang.org/show_bug.cgi?id=19254

aliak00 avatar Sep 20 '18 10:09 aliak00

I bet that won't go through though because it will break compatibility with programs (code-d included)

WebFreak001 avatar Sep 20 '18 14:09 WebFreak001

Aww :( well if it doesn’t then it doesn’t. Out of curiosity how would it break code-d? On Thu, 20 Sep 2018 at 16:56, Jan Jurzitza [email protected] wrote:

I bet that won't go through though because it will break compatibility with programs (code-d included)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Pure-D/code-d/issues/214#issuecomment-423214561, or mute the thread https://github.com/notifications/unsubscribe-auth/AChBzMBY5u_TOpXq_10jKNYK8LCPhc1zks5uc6yDgaJpZM4Wxlsi .

aliak00 avatar Sep 20 '18 17:09 aliak00

uh well after looking into it, it was actually compiler errors which I parsed so this wouldn't really be affected after all

WebFreak001 avatar Sep 20 '18 17:09 WebFreak001

there is now a proposed API in VSCode insiders so I am finally able to do this: https://code.visualstudio.com/updates/v1_47#_terminal-link-provider

It might still change and won't be usable for normal users at least before 1.48 though.

WebFreak001 avatar Jul 13 '20 06:07 WebFreak001