| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
| @Deprecated | ||
| public boolean hex; |
There was a problem hiding this comment.
hex is part of the specification: https://microsoft.github.io/debug-adapter-protocol/specification#Types_ValueFormat, what's the rationale for deprecating this?
Sorry, something went wrong.
There was a problem hiding this comment.
Because of new functionality we can format values to different formats now, beside hex. And I added new property ValueFormat#type instead of ValueFormat#hex to support that possibility. I assumed we can delete ValueFormat#hex in future with no harm because of no need and marked hex as deprecated. Just tell me if I did something wrong and I'll fix it. :)
Sorry, something went wrong.
There was a problem hiding this comment.
Dmitrii Kniazev (@mylog00) The data types under the folder com/microsoft/java/debug/core/protocol are generated from the DAP spec, we can't just change it on debugger side.
It's OK to extend the debug settings to support more format types, but remember to support the old valueFormat.hex as well.
Sorry, something went wrong.
There was a problem hiding this comment.
I've removed @Deprecated annotation and added doc to new property ValueFormat#type. Support of the old valueFormat.hex already was in my code for backward compatibility.
Should I create PR to DAP or do something else?
Sorry, something went wrong.
There was a problem hiding this comment.
When contributing to the DAP, you need to open an issue to discuss the proposal first. Only if the proposal is accepted, then open a PR.
A quick search and found a duplicated issue for more formatting options. microsoft/debug-adapter-protocol#197
Sorry, something went wrong.
There was a problem hiding this comment.
Got it. Thanks)
Sorry, something went wrong.
There was a problem hiding this comment.
Hi! If there is no other issue could you approve this PR?)
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Hi there!
This is PR for allow represent variables in binary forma.
I think it may be helpful for vscode-java-debug issue #1078
I've added new protocol parameter 'formatType' should contains variable format such as [BIN, OCT, HEX, DEC].
I've remain "showHex" for backward compatibility and can be delete later when PR to vscode-java-debug will be merged.
If you are not interested in this changes fill free to close this PR :)