add comments

alexlamsl · alexlamsl · commit 889e8798914f · 2017-02-08T12:03:36.000+08:00
diff --git a/test/compress/global_defs.js b/test/compress/global_defs.js
@@ -41,10 +41,12 @@ object: {
     }
     input: {
         function f(CONFIG) {
+            // CONFIG not global - do not replace
             return CONFIG.VALUE;
         }
         function g() {
             var CONFIG = { VALUE: 1 };
+            // CONFIG not global - do not replace
             return CONFIG.VALUE;
         }
         function h() {
@@ -78,10 +80,12 @@ expanded: {
     }
     input: {
         function f(CONFIG) {
+            // CONFIG not global - do not replace
             return CONFIG.VALUE;
         }
         function g() {
             var CONFIG = { VALUE: 1 };
+            // CONFIG not global - do not replace
             return CONFIG.VALUE;
         }
         function h() {
@@ -136,8 +140,8 @@ mixed: {
         console.log(CONFIG);
     }
     expect_warnings: [
-        'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:122,22]',
-        'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:123,22]',
-        'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:125,8]',
+        'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:126,22]',
+        'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:127,22]',
+        'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:129,8]',
     ]
 }

Original file line number	Diff line number	Diff line change
`@@ -41,10 +41,12 @@ object: {`
`41`	`41`	`}`
`42`	`42`	`input: {`
`43`	`43`	`function f(CONFIG) {`
	`44`	`+ // CONFIG not global - do not replace`
`44`	`45`	`return CONFIG.VALUE;`
`45`	`46`	`}`
`46`	`47`	`function g() {`
`47`	`48`	`var CONFIG = { VALUE: 1 };`
	`49`	`+ // CONFIG not global - do not replace`
`48`	`50`	`return CONFIG.VALUE;`
`49`	`51`	`}`
`50`	`52`	`function h() {`
`@@ -78,10 +80,12 @@ expanded: {`
`78`	`80`	`}`
`79`	`81`	`input: {`
`80`	`82`	`function f(CONFIG) {`
	`83`	`+ // CONFIG not global - do not replace`
`81`	`84`	`return CONFIG.VALUE;`
`82`	`85`	`}`
`83`	`86`	`function g() {`
`84`	`87`	`var CONFIG = { VALUE: 1 };`
	`88`	`+ // CONFIG not global - do not replace`
`85`	`89`	`return CONFIG.VALUE;`
`86`	`90`	`}`
`87`	`91`	`function h() {`
`@@ -136,8 +140,8 @@ mixed: {`
`136`	`140`	`console.log(CONFIG);`
`137`	`141`	`}`
`138`	`142`	`expect_warnings: [`
`139`		`- 'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:122,22]',`
`140`		`- 'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:123,22]',`
`141`		`- 'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:125,8]',`
	`143`	`+ 'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:126,22]',`
	`144`	`+ 'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:127,22]',`
	`145`	`+ 'WARN: global_defs CONFIG.VALUE redefined [test/compress/global_defs.js:129,8]',`
`142`	`146`	`]`
`143`	`147`	`}`