fix compile/api issue when using es6 (which is actually correct)

jrieken · jrieken · commit b0fd77e9ff3c · 2018-07-03T09:34:45.000+02:00
diff --git a/src/vs/workbench/api/node/extHostDebugService.ts b/src/vs/workbench/api/node/extHostDebugService.ts
@@ -426,9 +426,9 @@ export class ExtHostDebugService implements ExtHostDebugServiceShape {
 	private fireBreakpointChanges(added: vscode.Breakpoint[], removed: vscode.Breakpoint[], changed: vscode.Breakpoint[]) {
 		if (added.length > 0 || removed.length > 0 || changed.length > 0) {
 			this._onDidChangeBreakpoints.fire(Object.freeze({
-				added: Object.freeze<vscode.Breakpoint[]>(added),
-				removed: Object.freeze<vscode.Breakpoint[]>(removed),
-				changed: Object.freeze<vscode.Breakpoint[]>(changed)
+				added,
+				removed,
+				changed,
 			}));
 		}
 	}
diff --git a/src/vs/workbench/api/node/extHostWorkspace.ts b/src/vs/workbench/api/node/extHostWorkspace.ts
@@ -329,8 +329,8 @@ export class ExtHostWorkspace implements ExtHostWorkspaceShape {
 
 		// Events
 		this._onDidChangeWorkspace.fire(Object.freeze({
-			added: Object.freeze<vscode.WorkspaceFolder[]>(added),
-			removed: Object.freeze<vscode.WorkspaceFolder[]>(removed)
+			added,
+			removed,
 		}));
 	}
 
diff --git a/src/vs/workbench/test/electron-browser/api/extHostWorkspace.test.ts b/src/vs/workbench/test/electron-browser/api/extHostWorkspace.test.ts
@@ -528,9 +528,9 @@ suite('ExtHostWorkspace', function () {
 				assert.throws(() => {
 					(<any>e).added = [];
 				});
-				assert.throws(() => {
-					(<any>e.added)[0] = null;
-				});
+				// assert.throws(() => {
+				// 	(<any>e.added)[0] = null;
+				// });
 			} catch (error) {
 				finish(error);
 			}

Original file line number	Diff line number	Diff line change
`@@ -426,9 +426,9 @@ export class ExtHostDebugService implements ExtHostDebugServiceShape {`
`426`	`426`	`private fireBreakpointChanges(added: vscode.Breakpoint[], removed: vscode.Breakpoint[], changed: vscode.Breakpoint[]) {`
`427`	`427`	`if (added.length > 0 \|\| removed.length > 0 \|\| changed.length > 0) {`
`428`	`428`	`this._onDidChangeBreakpoints.fire(Object.freeze({`
`429`		`- added: Object.freeze<vscode.Breakpoint[]>(added),`
`430`		`- removed: Object.freeze<vscode.Breakpoint[]>(removed),`
`431`		`- changed: Object.freeze<vscode.Breakpoint[]>(changed)`
	`429`	`+ added,`
	`430`	`+ removed,`
	`431`	`+ changed,`
`432`	`432`	`}));`
`433`	`433`	`}`
`434`	`434`	`}`
Original file line number	Diff line number	Diff line change
`@@ -329,8 +329,8 @@ export class ExtHostWorkspace implements ExtHostWorkspaceShape {`
`329`	`329`
`330`	`330`	`// Events`
`331`	`331`	`this._onDidChangeWorkspace.fire(Object.freeze({`
`332`		`- added: Object.freeze<vscode.WorkspaceFolder[]>(added),`
`333`		`- removed: Object.freeze<vscode.WorkspaceFolder[]>(removed)`
	`332`	`+ added,`
	`333`	`+ removed,`
`334`	`334`	`}));`
`335`	`335`	`}`
`336`	`336`