Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 7e03429

Browse filesBrowse files
aozgaaaozgaa
committed
enforce triple-equals
1 parent 9bfba73 commit 7e03429
Copy full SHA for 7e03429

29 files changed

+51-50Lines changed: 51 additions & 50 deletions
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎src/compiler/checker.ts‎

Copy file name to clipboardExpand all lines: src/compiler/checker.ts
+2-2Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,7 @@ namespace ts {
338338
TypeofNEHostObject = 1 << 13, // typeof x !== "xxx"
339339
EQUndefined = 1 << 14, // x === undefined
340340
EQNull = 1 << 15, // x === null
341-
EQUndefinedOrNull = 1 << 16, // x == undefined / x == null
341+
EQUndefinedOrNull = 1 << 16, // x === undefined / x === null
342342
NEUndefined = 1 << 17, // x !== undefined
343343
NENull = 1 << 18, // x !== null
344344
NEUndefinedOrNull = 1 << 19, // x != undefined / x != null
@@ -17506,7 +17506,7 @@ namespace ts {
1750617506
function checkObjectTypeForDuplicateDeclarations(node: TypeLiteralNode | InterfaceDeclaration) {
1750717507
const names = createMap<boolean>();
1750817508
for (const member of node.members) {
17509-
if (member.kind == SyntaxKind.PropertySignature) {
17509+
if (member.kind === SyntaxKind.PropertySignature) {
1751017510
let memberName: string;
1751117511
switch (member.name.kind) {
1751217512
case SyntaxKind.StringLiteral:
Collapse file

‎src/compiler/moduleNameResolver.ts‎

Copy file name to clipboardExpand all lines: src/compiler/moduleNameResolver.ts
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -378,7 +378,7 @@ namespace ts {
378378
directoryPathMap.set(parent, result);
379379
current = parent;
380380

381-
if (current == commonPrefix) {
381+
if (current === commonPrefix) {
382382
break;
383383
}
384384
}
Collapse file

‎src/compiler/program.ts‎

Copy file name to clipboardExpand all lines: src/compiler/program.ts
+2-2Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -557,7 +557,7 @@ namespace ts {
557557
// combine results of resolutions and predicted results
558558
let j = 0;
559559
for (let i = 0; i < result.length; i++) {
560-
if (result[i] == predictedToResolveToAmbientModuleMarker) {
560+
if (result[i] === predictedToResolveToAmbientModuleMarker) {
561561
result[i] = undefined;
562562
}
563563
else {
@@ -1364,7 +1364,7 @@ namespace ts {
13641364

13651365
// If the file was previously found via a node_modules search, but is now being processed as a root file,
13661366
// then everything it sucks in may also be marked incorrectly, and needs to be checked again.
1367-
if (file && sourceFilesFoundSearchingNodeModules.get(file.path) && currentNodeModulesDepth == 0) {
1367+
if (file && sourceFilesFoundSearchingNodeModules.get(file.path) && currentNodeModulesDepth === 0) {
13681368
sourceFilesFoundSearchingNodeModules.set(file.path, false);
13691369
if (!options.noResolve) {
13701370
processReferencedFiles(file, isDefaultLib);
Collapse file

‎src/compiler/tsc.ts‎

Copy file name to clipboardExpand all lines: src/compiler/tsc.ts
+3-3Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ namespace ts {
3030
}
3131

3232
function reportEmittedFiles(files: string[]): void {
33-
if (!files || files.length == 0) {
33+
if (!files || files.length === 0) {
3434
return;
3535
}
3636

@@ -282,7 +282,7 @@ namespace ts {
282282
// When the configFileName is just "tsconfig.json", the watched directory should be
283283
// the current directory; if there is a given "project" parameter, then the configFileName
284284
// is an absolute file name.
285-
directory == "" ? "." : directory,
285+
directory === "" ? "." : directory,
286286
watchedDirectoryChanged, /*recursive*/ true);
287287
}
288288
}
@@ -334,7 +334,7 @@ namespace ts {
334334
// When the configFileName is just "tsconfig.json", the watched directory should be
335335
// the current directory; if there is a given "project" parameter, then the configFileName
336336
// is an absolute file name.
337-
directory == "" ? "." : directory,
337+
directory === "" ? "." : directory,
338338
watchedDirectoryChanged, /*recursive*/ true);
339339
};
340340
}
Collapse file

‎src/compiler/utilities.ts‎

Copy file name to clipboardExpand all lines: src/compiler/utilities.ts
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4658,7 +4658,7 @@ namespace ts {
46584658
*/
46594659
export function getParseTreeNode<T extends Node>(node: Node, nodeTest?: (node: Node) => node is T): T;
46604660
export function getParseTreeNode(node: Node, nodeTest?: (node: Node) => boolean): Node {
4661-
if (node == undefined || isParseTreeNode(node)) {
4661+
if (node === undefined || isParseTreeNode(node)) {
46624662
return node;
46634663
}
46644664

Collapse file

‎src/harness/fourslash.ts‎

Copy file name to clipboardExpand all lines: src/harness/fourslash.ts
+4-4Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2275,13 +2275,13 @@ namespace FourSlash {
22752275

22762276
public verifyImportFixAtPosition(expectedTextArray: string[], errorCode?: number) {
22772277
const ranges = this.getRanges();
2278-
if (ranges.length == 0) {
2278+
if (ranges.length === 0) {
22792279
this.raiseError("At least one range should be specified in the testfile.");
22802280
}
22812281

22822282
const codeFixes = this.getCodeFixActions(this.activeFile.fileName, errorCode);
22832283

2284-
if (!codeFixes || codeFixes.length == 0) {
2284+
if (!codeFixes || codeFixes.length === 0) {
22852285
this.raiseError("No codefixes returned.");
22862286
}
22872287

@@ -2646,7 +2646,7 @@ namespace FourSlash {
26462646
private assertItemInCompletionList(items: ts.CompletionEntry[], name: string, text?: string, documentation?: string, kind?: string, spanIndex?: number) {
26472647
for (const item of items) {
26482648
if (item.name === name) {
2649-
if (documentation != undefined || text !== undefined) {
2649+
if (documentation !== undefined || text !== undefined) {
26502650
const details = this.getCompletionEntryDetails(item.name);
26512651

26522652
if (documentation !== undefined) {
@@ -2899,7 +2899,7 @@ ${code}
28992899
}
29002900
// TODO: should be '==='?
29012901
}
2902-
else if (line == "" || lineLength === 0) {
2902+
else if (line === "" || lineLength === 0) {
29032903
// Previously blank lines between fourslash content caused it to be considered as 2 files,
29042904
// Remove this behavior since it just causes errors now
29052905
}
Collapse file

‎src/harness/harness.ts‎

Copy file name to clipboardExpand all lines: src/harness/harness.ts
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1933,7 +1933,7 @@ namespace Harness {
19331933
}
19341934

19351935
const parentDirectory = IO.directoryName(dirName);
1936-
if (parentDirectory != "") {
1936+
if (parentDirectory !== "") {
19371937
createDirectoryStructure(parentDirectory);
19381938
}
19391939
IO.createDirectory(dirName);
Collapse file

‎src/harness/projectsRunner.ts‎

Copy file name to clipboardExpand all lines: src/harness/projectsRunner.ts
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ class ProjectRunner extends RunnerBase {
112112
// Replace the disk specific path into the project root path
113113
url = url.substr(diskProjectPath.length);
114114
// TODO: should be '!=='?
115-
if (url.charCodeAt(0) != ts.CharacterCodes.slash) {
115+
if (url.charCodeAt(0) !== ts.CharacterCodes.slash) {
116116
url = "/" + url;
117117
}
118118
}
Collapse file

‎src/harness/sourceMapRecorder.ts‎

Copy file name to clipboardExpand all lines: src/harness/sourceMapRecorder.ts
+4-4Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,11 @@ namespace Harness.SourceMapRecorder {
5050
return true;
5151
}
5252

53-
if (sourceMapMappings.charAt(decodingIndex) == ",") {
53+
if (sourceMapMappings.charAt(decodingIndex) === ",") {
5454
return true;
5555
}
5656

57-
if (sourceMapMappings.charAt(decodingIndex) == ";") {
57+
if (sourceMapMappings.charAt(decodingIndex) === ";") {
5858
return true;
5959
}
6060

@@ -117,15 +117,15 @@ namespace Harness.SourceMapRecorder {
117117
}
118118

119119
while (decodingIndex < sourceMapMappings.length) {
120-
if (sourceMapMappings.charAt(decodingIndex) == ";") {
120+
if (sourceMapMappings.charAt(decodingIndex) === ";") {
121121
// New line
122122
decodeOfEncodedMapping.emittedLine++;
123123
decodeOfEncodedMapping.emittedColumn = 1;
124124
decodingIndex++;
125125
continue;
126126
}
127127

128-
if (sourceMapMappings.charAt(decodingIndex) == ",") {
128+
if (sourceMapMappings.charAt(decodingIndex) === ",") {
129129
// Next entry is on same line - no action needed
130130
decodingIndex++;
131131
continue;
Collapse file

‎src/harness/unittests/tsserverProjectSystem.ts‎

Copy file name to clipboardExpand all lines: src/harness/unittests/tsserverProjectSystem.ts
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3144,7 +3144,7 @@ namespace ts.projectSystem {
31443144
checkNumberOfInferredProjects(projectService, 1);
31453145

31463146
const configuredProject = projectService.configuredProjects[0];
3147-
assert.isTrue(configuredProject.getFileNames().length == 0);
3147+
assert.isTrue(configuredProject.getFileNames().length === 0);
31483148

31493149
const inferredProject = projectService.inferredProjects[0];
31503150
assert.isTrue(inferredProject.containsFile(<server.NormalizedPath>file1.path));

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.