Skip to content

Wrap decoration of EventEmitter within a try...catch #2196

@DonJayamanne

Description

@DonJayamanne

If other extensions end up with code such as the following:

decorate(injectable(), EventEmitter);

Src - src/client/ioc/container.ts

Then the extension that loads second will crash.
Solution:

try {
	decorate(injectable(), EventEmitter);
} catch {}

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugIssue identified by VS Code Team member as probable bug

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions