Skip to content
This repository was archived by the owner on Oct 15, 2020. It is now read-only.

Commit ed29231

Browse files
committed
chakrashim,lib,test: fix lint errors
* Fixed a bunch of issues with whitespace around comments * Ignored lint issues in `test-util-inspect.js` that seemed to be false positives.
1 parent 27685b5 commit ed29231

File tree

9 files changed

+108
-120
lines changed

9 files changed

+108
-120
lines changed

deps/chakrashim/lib/chakra_shim.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454
};
5555

5656
StackFrame.prototype.getTypeName = function() {
57-
//TODO : Fix this
57+
// TODO : Fix this
5858
return this.functionName;
5959
};
6060

lib/assert.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ const meta = [
5454

5555
const escapeFn = (str) => meta[str.charCodeAt(0)];
5656

57-
const trace_mgr = require('trace_mgr'); //ENABLE_TTD
57+
const trace_mgr = require('trace_mgr'); // ENABLE_TTD
5858

5959
const ERR_DIFF_DEACTIVATED = 0;
6060
const ERR_DIFF_NOT_EQUAL = 1;
@@ -83,7 +83,7 @@ function innerFail(obj) {
8383
}
8484

8585
function fail(actual, expected, message, operator, stackStartFn) {
86-
trace_mgr.emitTrace('emitOnAssert'); //ENABLE_TTD
86+
trace_mgr.emitTrace('emitOnAssert'); // ENABLE_TTD
8787

8888
const argsLen = arguments.length;
8989

lib/console.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ const {
3636
isTypedArray, isSet, isMap, isSetIterator, isMapIterator,
3737
} = util.types;
3838
const kCounts = Symbol('counts');
39-
const trace_mgr = require('trace_mgr'); //ENABLE_TTD
39+
const trace_mgr = require('trace_mgr'); // ENABLE_TTD
4040

4141
const {
4242
keys: ObjectKeys,
@@ -166,15 +166,15 @@ Console.prototype.warn = function warn(...args) {
166166
this._stderrErrorHandler,
167167
this[kGroupIndent]);
168168

169-
trace_mgr.emitTrace('emitOnLogWarn'); //ENABLE_TTD
169+
trace_mgr.emitTrace('emitOnLogWarn'); // ENABLE_TTD
170170
};
171171
Console.prototype.error = function error(...args) {
172172
write(this._ignoreErrors,
173173
this._stderr,
174174
util.format.apply(null, args),
175175
this._stderrErrorHandler,
176176
this[kGroupIndent]);
177-
trace_mgr.emitTrace('emitOnLogError'); //ENABLE_TTD
177+
trace_mgr.emitTrace('emitOnLogError'); // ENABLE_TTD
178178
};
179179

180180
Console.prototype.dir = function dir(object, options) {

lib/internal/bootstrap/node.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@
4949
}
5050

5151

52-
5352
setupV8();
5453
setupProcessICUVersions();
5554

@@ -86,7 +85,7 @@
8685

8786
NativeModule.require('internal/trace_events_async_hooks').setup();
8887
NativeModule.require('internal/inspector_async_hook').setup();
89-
NativeModule.require('trace_mgr'); //ENABLE_TTD;
88+
NativeModule.require('trace_mgr'); // ENABLE_TTD;
9089

9190
_process.setupChannel();
9291
_process.setupRawDebug();
@@ -455,7 +454,7 @@
455454
if (exceptionHandlerState.captureFn !== null) {
456455
exceptionHandlerState.captureFn(er);
457456
} else if (!process.emit('uncaughtException', er)) {
458-
//ENABLE_TTD
457+
// ENABLE_TTD
459458
NativeModule.require('trace_mgr').onUncaughtExceptionHandler();
460459

461460
// If someone handled it, then great. otherwise, die in C++ land

lib/internal/process.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ function setupKillAndExit() {
164164
process.emit('exit', process.exitCode || 0);
165165
}
166166

167-
require('trace_mgr').onExitHandler(process.exitCode || 0); //ENABLE_TTD
167+
require('trace_mgr').onExitHandler(process.exitCode || 0); // ENABLE_TTD
168168
process.reallyExit(process.exitCode || 0);
169169
};
170170

@@ -214,10 +214,10 @@ function setupSignalHandlers() {
214214
wrap.unref();
215215

216216
if (type === 'SIGINT' && global.enabledDiagnosticsTrace) {
217-
//TODO: this is probably a little over complicated
217+
// TODO: this is probably a little over complicated
218218
// -- should unify this and ttdSigIntHandler setup below
219219
// -- ttdSigIntHandler will emit event after it runs
220-
wrap.onsignal = function() { }; //ENABLE_TTD
220+
wrap.onsignal = function() { }; // ENABLE_TTD
221221
} else {
222222
wrap.onsignal = process.emit.bind(process, type, type);
223223
}
@@ -240,7 +240,7 @@ function setupSignalHandlers() {
240240
}
241241
});
242242

243-
//ENABLE_TTD
243+
// ENABLE_TTD
244244
if (global.enabledDiagnosticsTrace) {
245245
(function() {
246246
const type = 'SIGINT';

0 commit comments

Comments
 (0)