Skip to content

Commit 5c4e451

Browse files
committed
Abstracted out common code into innerTransform
As per feedback brought common logic from both `transform` and `transformAsObject` into its own `innerTransform` function.
1 parent 4ecde42 commit 5c4e451

1 file changed

Lines changed: 17 additions & 16 deletions

File tree

main.js

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -6,34 +6,35 @@ var Buffer = require('buffer').Buffer;
66

77
module.exports = {
88
transform: function(input, options) {
9-
options = options || {};
10-
var visitorList = getVisitors(options.harmony);
11-
var result = transform(visitorList, input, options);
12-
var output = result.code;
13-
if (options.sourceMap) {
9+
var output = innerTransform(input, options);
10+
var result = output.code;
11+
if (options && options.sourceMap) {
1412
var map = inlineSourceMap(
15-
result.sourceMap,
13+
output.sourceMap,
1614
input,
1715
options.sourceFilename
1816
);
19-
output += '\n' + map;
17+
result += '\n' + map;
2018
}
21-
return output;
19+
return result;
2220
},
2321
transformAsObject: function(input, options) {
24-
options = options || {};
25-
var visitorList = getVisitors(options.harmony);
26-
var resultRaw = transform(visitorList, input, options);
27-
var result = {
28-
code: resultRaw.code
29-
};
30-
if (options.sourceMap) {
31-
result.sourceMap = resultRaw.sourceMap;
22+
var output = innerTransform(input, options);
23+
var result = {};
24+
result.code = output.code;
25+
if (options && options.sourceMap) {
26+
result.sourceMap = output.sourceMap;
3227
}
3328
return result;
3429
}
3530
};
3631

32+
function innerTransform(input, options) {
33+
options = options || {};
34+
var visitorList = getVisitors(options.harmony);
35+
return transform(visitorList, input, options);
36+
}
37+
3738
function getVisitors(harmony) {
3839
if (harmony) {
3940
return visitors.getAllVisitors();

0 commit comments

Comments
 (0)