Return result directly instead of assigning to var to return

mjbvz · mjbvz · commit 3e246720db8a · 2019-02-06T14:44:52.000-08:00
diff --git a/src/vs/workbench/common/editor/untitledEditorInput.ts b/src/vs/workbench/common/editor/untitledEditorInput.ts
@@ -95,21 +95,15 @@ export class UntitledEditorInput extends EditorInput implements IEncodingSupport
 			return null;
 		}
 
-		let description: string;
 		switch (verbosity) {
 			case Verbosity.SHORT:
-				description = this.shortDescription;
-				break;
+				return this.shortDescription;
 			case Verbosity.LONG:
-				description = this.longDescription;
-				break;
+				return this.longDescription;
 			case Verbosity.MEDIUM:
 			default:
-				description = this.mediumDescription;
-				break;
+				return this.mediumDescription;
 		}
-
-		return description;
 	}
 
 	@memoize
diff --git a/src/vs/workbench/services/files/electron-browser/remoteFileService.ts b/src/vs/workbench/services/files/electron-browser/remoteFileService.ts
@@ -221,27 +221,20 @@ export class RemoteFileService extends FileService {
 		if (!match) {
 			return undefined;
 		}
-		let res: FileOperationResult;
 		switch (match[1]) {
 			case 'EntryNotFound':
-				res = FileOperationResult.FILE_NOT_FOUND;
-				break;
+				return FileOperationResult.FILE_NOT_FOUND;
 			case 'EntryIsADirectory':
-				res = FileOperationResult.FILE_IS_DIRECTORY;
-				break;
+				return FileOperationResult.FILE_IS_DIRECTORY;
 			case 'NoPermissions':
-				res = FileOperationResult.FILE_PERMISSION_DENIED;
-				break;
+				return FileOperationResult.FILE_PERMISSION_DENIED;
 			case 'EntryExists':
-				res = FileOperationResult.FILE_MOVE_CONFLICT;
-				break;
+				return FileOperationResult.FILE_MOVE_CONFLICT;
 			case 'EntryNotADirectory':
 			default:
 				// todo
-				res = undefined;
-				break;
+				return undefined;
 		}
-		return res;
 	}
 
 	// --- stat
