Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

debug: compound map keys cannot be expanded #1085

Closed
polinasok opened this issue Jan 7, 2021 · 3 comments
Closed

debug: compound map keys cannot be expanded #1085

polinasok opened this issue Jan 7, 2021 · 3 comments
Assignees
Labels
Debug Issues related to the debugging functionality of the extension. FrozenDueToAge

Comments

@polinasok
Copy link
Contributor

Code:

	mapWithCompoundValue := map[int]myStruct{123: {123}}
	mapWithCompoundKey := map[myStruct]int{{123}: 123}

UI:
image
Logs:

### Expand mapWithCompoundValue
VariablesResponse [
 {
  "name": "123",
  "value": "<main.myStruct>",
  "evaluateName": "mapWithCompoundValue[123]",
  "variablesReference": 1003 #### <============= this allows to expand again to view the value
 }
VariablesResponse [
 {
  "name": "i",
  "value": "123",
  "evaluateName": "mapWithCompoundValue.i",
  "variablesReference": 0
 }

### Expand mapWithConpoundKey
VariablesResponse [
 {
  "name": "<main.myStruct>",
  "value": "123",
  "evaluateName": "mapWithCompoundKey[<main.myStruct>]",
  "variablesReference": 0 #### <=============== no way to expand more to view the key
 }
@polinasok polinasok added Debug Issues related to the debugging functionality of the extension. DelveDAP labels Jan 7, 2021
@polinasok
Copy link
Contributor Author

This is how new adapter in delve repo deals with this: go-delve/delve#2111 (comment)

@polinasok
Copy link
Contributor Author

This is what the support looks like in dlv-dap:
image

@golang golang locked and limited conversation to collaborators Apr 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Debug Issues related to the debugging functionality of the extension. FrozenDueToAge
Projects
None yet
Development

No branches or pull requests

2 participants