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

Bad call hierarchy highlights #36512

Closed
jrieken opened this issue Jan 29, 2020 · 3 comments · Fixed by #36559
Closed

Bad call hierarchy highlights #36512

jrieken opened this issue Jan 29, 2020 · 3 comments · Fixed by #36559
Assignees

Comments

@jrieken
Copy link
Member

jrieken commented Jan 29, 2020

TS Template added by @mjbvz

Tested typeScript Version: 3.8.0-dev.20200128

Search Terms

  • Call hierarchy
  • provideCallHierarchyIncomingCalls
  • fromSpans

/cc @rbuckton


re microsoft/vscode#89386

  • invoke call hierarchy on isFalsyOrWhitespace
  • the resolve-child and likely others have broken highlights

image

@mjbvz mjbvz transferred this issue from microsoft/vscode Jan 29, 2020
@mjbvz
Copy link
Contributor

mjbvz commented Jan 29, 2020

Here's the result of one of the incoming calls to mergeSort in the VS Code code base:

  {
        "from": {
            "name": "applyEdits",
            "kind": "function",
            "file": "/Users/matb/projects/vscode/src/vs/base/common/jsonEdit.ts",
            "span": {
                "start": {
                    "line": 158,
                    "offset": 1
                },
                "end": {
                    "line": 177,
                    "offset": 2
                }
            },
            "selectionSpan": {
                "start": {
                    "line": 158,
                    "offset": 17
                },
                "end": {
                    "line": 158,
                    "offset": 27
                }
            }
        },
        "fromSpans": [
            {
                "start": {
                    "line": 238,
                    "offset": 98
                },
                "end": {
                    "line": 239,
                    "offset": 8
                }
            }
        ]
    },

The jsonEdit.ts file only has 182 lines so I'm not sure where the fromSpans are coming from.

@mjbvz mjbvz removed their assignment Jan 29, 2020
@rbuckton rbuckton self-assigned this Jan 30, 2020
@mjbvz
Copy link
Contributor

mjbvz commented Jan 31, 2020

@rbuckton @RyanCavanaugh Can we please investigate this for TS 3.8 (either for the final 3.8 release or for a 3.8.x recovery build that VS code would be able to pick up)

@rbuckton
Copy link
Member

rbuckton commented Feb 1, 2020

My bad, I should have included this as fixed by #36559. It will be in the 3.8 RC.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants