jsdoc and comment cleanup

Tyriar · Tyriar · commit 36f1d703e717 · 2016-10-19T11:05:10.000-07:00
diff --git a/src/vs/workbench/services/files/node/fileService.ts b/src/vs/workbench/services/files/node/fileService.ts
@@ -458,7 +458,6 @@ export class FileService implements IFileService {
 	}
 
 	public backupFile(resource: uri, content: string): TPromise<IFileStat> {
-		// TODO: This should not backup unless necessary. Currently this is called for each file on close to ensure the files are backed up.
 		let registerResourcePromise: TPromise<void>;
 		if (resource.scheme === 'file') {
 			registerResourcePromise = this.backupService.registerResourceForBackup(resource);
diff --git a/src/vs/workbench/services/textfile/common/textFileEditorModel.ts b/src/vs/workbench/services/textfile/common/textFileEditorModel.ts
@@ -265,12 +265,10 @@ export class TextFileEditorModel extends BaseTextEditorModel implements ITextFil
 				diag('load() - created text editor model', this.resource, new Date());
 
 				if (this.restoreResource) {
-					// TODO: De-duplicate code
 					this.createTextEditorModelPromise = this.textFileService.resolveTextContent(this.restoreResource, { acceptTextOnly: true, etag: etag, encoding: this.preferredEncoding }).then((restoreContent) => {
 						return this.createTextEditorModel(restoreContent.value, content.resource).then(() => {
 							this.createTextEditorModelPromise = null;
 
-							// TODO: This does not set the dirty indicator immediately, making it look like the file is not actually dirty
 							this.setDirty(true);
 							this.toDispose.push(this.textEditorModel.onDidChangeRawContent((e: IModelContentChangedEvent) => this.onModelContentChanged(e)));
 
diff --git a/src/vs/workbench/services/textfile/common/textfiles.ts b/src/vs/workbench/services/textfile/common/textfiles.ts
@@ -322,7 +322,12 @@ export interface ITextFileService extends IDisposable {
 	 */
 	confirmSave(resources?: URI[]): ConfirmResult;
 
-	// TODO: Doc
+	/**
+	 * Backs up the provided file to a temporary directory to be used by the hot
+	 * exit feature and crash recovery.
+	 *
+	 * @param resource The resource to backup.
+	 */
 	backup(resource: URI): void;
 
 	/**

Original file line number	Diff line number	Diff line change
`@@ -458,7 +458,6 @@ export class FileService implements IFileService {`
`458`	`458`	`}`
`459`	`459`
`460`	`460`	`public backupFile(resource: uri, content: string): TPromise<IFileStat> {`
`461`		`- // TODO: This should not backup unless necessary. Currently this is called for each file on close to ensure the files are backed up.`
`462`	`461`	`let registerResourcePromise: TPromise<void>;`
`463`	`462`	`if (resource.scheme === 'file') {`
`464`	`463`	`registerResourcePromise = this.backupService.registerResourceForBackup(resource);`