Skip to content

Commit da861be

Browse files
committed
Merge pull request mozilla#2009 from kkujala/dev
Correct gjslint warnings in builder.js.
2 parents ace05a7 + 7bbf65f commit da861be

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

external/builder/builder.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ function preprocess(inFilename, outFilename, defines) {
2727
}
2828
return null;
2929
}
30-
var writeLine = typeof outFilename === 'function' ? outFilename : function(line) {
30+
var writeLine = typeof outFilename === 'function' ? outFilename :
31+
function(line) {
3132
out += line + '\n';
3233
}
3334
function include(file) {
@@ -47,7 +48,8 @@ function preprocess(inFilename, outFilename, defines) {
4748
}
4849

4950
var s, state = 0, stack = [];
50-
var control = /^(?:\/\/|<!--)\s*#(if|else|endif|expand|include)(?:\s+(.*?)(?:-->)?$)?/;
51+
var control =
52+
/^(?:\/\/|<!--)\s*#(if|else|endif|expand|include)(?:\s+(.*?)(?:-->)?$)?/;
5153
var lineNumber = 0;
5254
while ((s = readLine()) !== null) {
5355
++lineNumber;
@@ -82,8 +84,8 @@ function preprocess(inFilename, outFilename, defines) {
8284
} else {
8385
if (state === 0) {
8486
writeLine(s);
85-
} else if(state === 3) {
86-
writeLine(s.replace(/^\/\/|^<!--|-->/g, " "));
87+
} else if (state === 3) {
88+
writeLine(s.replace(/^\/\/|^<!--|-->/g, ' '));
8789
}
8890
}
8991
}
@@ -96,11 +98,11 @@ exports.preprocess = preprocess;
9698

9799
/**
98100
* Simplifies common build steps.
99-
* @param setup
101+
* @param {object} setup
100102
* .defines defines for preprocessors
101103
* .copy array of arrays of source and destination pairs of files to copy
102104
* .preprocess array of arrays of source and destination pairs of files
103-
* run through preprocessor
105+
* run through preprocessor.
104106
*/
105107
function build(setup) {
106108
var defines = setup.defines;

0 commit comments

Comments
 (0)