Move log message outside of 'if' block so I have confirmation that smoething happened

roblourens · roblourens · commit f45dffcd1fe0 · 2022-10-26T11:29:35.000-07:00
diff --git a/src/vs/workbench/contrib/notebook/browser/services/notebookExecutionStateServiceImpl.ts b/src/vs/workbench/contrib/notebook/browser/services/notebookExecutionStateServiceImpl.ts
@@ -306,8 +306,8 @@ class NotebookExecutionListeners extends Disposable {
 			if (kernel) {
 				const implementsInterrupt = kernel.implementsInterrupt;
 				const handlesToCancel = implementsInterrupt ? [...executingDeletedHandles] : [...executingDeletedHandles, ...pendingDeletedHandles];
+				this._logService.debug(`NotebookExecution#onWillAddRemoveCells, ${JSON.stringify([...handlesToCancel])}`);
 				if (handlesToCancel.length) {
-					this._logService.debug(`NotebookExecution#onWillAddRemoveCells, ${JSON.stringify([...handlesToCancel])}`);
 					kernel.cancelNotebookCellExecution(this._notebookModel.uri, handlesToCancel);
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -306,8 +306,8 @@ class NotebookExecutionListeners extends Disposable {`
`306`	`306`	`if (kernel) {`
`307`	`307`	`const implementsInterrupt = kernel.implementsInterrupt;`
`308`	`308`	`const handlesToCancel = implementsInterrupt ? [...executingDeletedHandles] : [...executingDeletedHandles, ...pendingDeletedHandles];`
	`309`	+ this._logService.debug(`NotebookExecution#onWillAddRemoveCells, ${JSON.stringify([...handlesToCancel])}`);
`309`	`310`	`if (handlesToCancel.length) {`
`310`		- this._logService.debug(`NotebookExecution#onWillAddRemoveCells, ${JSON.stringify([...handlesToCancel])}`);
`311`	`311`	`kernel.cancelNotebookCellExecution(this._notebookModel.uri, handlesToCancel);`
`312`	`312`	`}`
`313`	`313`	`}`