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

Invoking deltaDecorations recursively could lead to leaking decorations. #188348

Closed
vscodenpa opened this issue Jul 20, 2023 · 0 comments · Fixed by #188372
Closed

Invoking deltaDecorations recursively could lead to leaking decorations. #188348

vscodenpa opened this issue Jul 20, 2023 · 0 comments · Fixed by #188372
Assignees
Labels
error-telemetry VS Code - Issues generated by telemetry inline-completions insiders-released Patch has been released in VS Code Insiders
Milestone

Comments

@vscodenpa
Copy link

Error: Invoking deltaDecorations recursively could lead to leaking decorations.
at Qn.deltaDecorations in src/vs/editor/common/model/textModel.ts:1627:23
at o.dispose in src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsSource.ts:224:19
at i.e in src/vs/base/common/observableImpl/base.ts:322:16
at i.set in src/vs/base/common/observableImpl/base.ts:289:9
at l.clear in src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsSource.ts:108:26
at <anonymous> in src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsModel.ts:149:17
at fn in src/vs/base/common/observableImpl/base.ts:213:3
at r.stop in out/vs/workbench/workbench.desktop.main.js:756:85477
at <anonymous> in src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsController.ts:153:23
at fn in src/vs/base/common/observableImpl/base.ts:202:3
at s.value in out/vs/workbench/workbench.desktop.main.js:758:153844
at d.w in src/vs/base/common/event.ts:1088:13
at d.x in src/vs/base/common/event.ts:1099:9
at d.fire in src/vs/base/common/event.ts:1110:9
at s.value in src/vs/editor/browser/widget/codeEditorWidget.ts:1728:40
at d.w in src/vs/base/common/event.ts:1088:13
at d.fire in src/vs/base/common/event.ts:1119:9
at L.n in src/vs/editor/common/viewModelEventDispatcher.ts:64:18
at L.emitOutgoingEvent in src/vs/editor/common/viewModelEventDispatcher.ts:39:8
at s.value in src/vs/editor/common/viewModel/viewModelImpl.ts:463:26
at d.w in src/vs/base/common/event.ts:1088:13
at d.fire in src/vs/base/common/event.ts:1119:9
at ie.s in src/vs/editor/common/model/textModel.ts:2433:16
at ie.endDeferredEmit in src/vs/editor/common/model/textModel.ts:2378:10
at Qn.deltaDecorations in src/vs/editor/common/model/textModel.ts:1632:33
at o.dispose in src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsSource.ts:224:19
at i.e in src/vs/base/common/observableImpl/base.ts:322:16
at i.set in src/vs/base/common/observableImpl/base.ts:289:9
at l.clearSuggestWidgetInlineCompletions in src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsSource.ts:116:39
at <anonymous> in src/vs/editor/contrib/inlineCompletions/browser/inlineCompletionsModel.ts:121:18
at fn in src/vs/base/common/observableImpl/base.ts:202:3
at p.l in out/vs/workbench/workbench.desktop.main.js:756:82687
at p.s in src/vs/base/common/observableImpl/derived.ts:145:22
at p.get in src/vs/base/common/observableImpl/derived.ts:122:10
at p.reportChanges in src/vs/base/common/observableImpl/base.ts:147:8
at a.endUpdate in src/vs/base/common/observableImpl/utils.ts:314:15
at p.endUpdate in src/vs/base/common/observableImpl/derived.ts:201:7
at y.finish in src/vs/base/common/observableImpl/base.ts:239:13
at w in src/vs/base/common/observableImpl/base.ts:204:6
at l.m in out/vs/workbench/workbench.desktop.main.js:758:131695
at s.value in src/vs/editor/contrib/inlineCompletions/browser/suggestWidgetInlineCompletionProvider.ts:103:11
at d.w in src/vs/base/common/event.ts:1088:13
at d.x in src/vs/base/common/event.ts:1099:9
at d.fire in src/vs/base/common/event.ts:1123:9
at Br.hideWidget in src/vs/editor/contrib/suggest/browser/suggestWidget.ts:743:19
at s.value in src/vs/editor/contrib/suggest/browser/suggestController.ts:269:23
at d.w in src/vs/base/common/event.ts:1088:13
at d.x in src/vs/base/common/event.ts:1099:9
at d.fire in src/vs/base/common/event.ts:1123:9
at ul.cancel in src/vs/editor/contrib/suggest/browser/suggestModel.ts:317:22
at s.value in src/vs/editor/contrib/suggest/browser/suggestController.ts:274:16
at d.w in src/vs/base/common/event.ts:1088:13
at d.x in src/vs/base/common/event.ts:1099:9
at d.fire in src/vs/base/common/event.ts:1123:9
at G.setValue in src/vs/editor/browser/widget/codeEditorWidget.ts:1955:29
at s.value in src/vs/editor/browser/widget/codeEditorWidget.ts:322:28
at d.w in src/vs/base/common/event.ts:1088:13
at d.fire in src/vs/base/common/event.ts:1119:9
at s.value in src/vs/editor/browser/widget/codeEditorWidget.ts:2223:19
at d.w in src/vs/base/common/event.ts:1088:13
at d.fire in src/vs/base/common/event.ts:1119:9
at <anonymous> in src/vs/base/browser/dom.ts:905:23

Go to Errors Site

@vscodenpa vscodenpa added the error-telemetry VS Code - Issues generated by telemetry label Jul 20, 2023
@hediet hediet added this to the July 2023 milestone Jul 20, 2023
hediet added a commit that referenced this issue Jul 20, 2023
@hediet hediet added bug Issue identified by VS Code Team member as probable bug inline-completions labels Jul 20, 2023
hediet added a commit that referenced this issue Jul 20, 2023
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Jul 20, 2023
@hediet hediet removed the bug Issue identified by VS Code Team member as probable bug label Jul 21, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Sep 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
error-telemetry VS Code - Issues generated by telemetry inline-completions insiders-released Patch has been released in VS Code Insiders
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants