rename variables for better readability (#1863)
authorAlex Lam S.L <alexlamsl@gmail.com>
Tue, 2 May 2017 12:47:10 +0000 (20:47 +0800)
committerGitHub <noreply@github.com>
Tue, 2 May 2017 12:47:10 +0000 (20:47 +0800)
lib/compress.js

index e5a7af2..931b8d6 100644 (file)
@@ -3703,7 +3703,7 @@ merge(Compressor.prototype, {
                     var init = fixed.evaluate(compressor);
                     if (init !== fixed) {
                         init = make_node_from_constant(init, fixed);
-                        var value = init.optimize(compressor).print_to_string().length;
+                        var value_length = init.optimize(compressor).print_to_string().length;
                         var fn;
                         if (has_symbol_ref(fixed)) {
                             fn = function() {
@@ -3711,18 +3711,18 @@ merge(Compressor.prototype, {
                                 return result === init ? result.clone(true) : result;
                             };
                         } else {
-                            value = Math.min(value, fixed.print_to_string().length);
+                            value_length = Math.min(value_length, fixed.print_to_string().length);
                             fn = function() {
                                 var result = best_of_expression(init.optimize(compressor), fixed);
                                 return result === init || result === fixed ? result.clone(true) : result;
                             };
                         }
-                        var name = d.name.length;
+                        var name_length = d.name.length;
                         var overhead = 0;
                         if (compressor.option("unused") && (!d.global || compressor.toplevel(d))) {
-                            overhead = (name + 2 + value) / d.references.length;
+                            overhead = (name_length + 2 + value_length) / d.references.length;
                         }
-                        d.should_replace = value <= name + overhead ? fn : false;
+                        d.should_replace = value_length <= name_length + overhead ? fn : false;
                     } else {
                         d.should_replace = false;
                     }