Simplify.

d3.v2.js

@@ -606,16 +606,12 @@
   if (d3_format_grouping) {
     var d3_format_groupingLength = d3_format_grouping.length;
     d3_format_group = function(value) {
-      var i = value.lastIndexOf("."), f = i >= 0 ? "." + value.substring(i + 1) : (i = value.length, "");
+      var i = value.lastIndexOf("."), f = i >= 0 ? "." + value.substring(i + 1) : (i = value.length, ""), t = [], j = 0, g = d3_format_grouping[0];
-      if (d3_format_grouping) {
+      while (i > 0 && g > 0) {
-        var t = [], j = 0, g = d3_format_grouping[0];
+        t.unshift(value.substring(i -= g, i + g));
-        while (i > 0 && g > 0) {
+        g = d3_format_grouping[j = (j + 1) % d3_format_groupingLength];
-          t.unshift(value.substring(i -= g, i + g));
-          g = d3_format_grouping[j = (j + 1) % d3_format_groupingLength];
-        }
-        return t.join(d3_format_thousandsSeparator || "") + f;
       }
-      return value.substring(0, i) + f;
+      return t.join(d3_format_thousandsSeparator || "") + f;
     };
   }
   var d3_formatPrefixes = [ "y", "z", "a", "f", "p", "n", "μ", "m", "", "k", "M", "G", "T", "P", "E", "Z", "Y" ].map(d3_formatPrefix);

src/core/format.js

@@ -96,17 +96,14 @@ if (d3_format_grouping) {
   var d3_format_groupingLength = d3_format_grouping.length;
   d3_format_group = function(value) {
     var i = value.lastIndexOf("."),
-        f = i >= 0 ? "." + value.substring(i + 1) : (i = value.length, "");
+        f = i >= 0 ? "." + value.substring(i + 1) : (i = value.length, ""),
-    if (d3_format_grouping) {
+        t = [],
-      var t = [],
+        j = 0,
-          j = 0,
+        g = d3_format_grouping[0];
-          g = d3_format_grouping[0];
+    while (i > 0 && g > 0) {
-      while (i > 0 && g > 0) {
+      t.unshift(value.substring(i -= g, i + g));
-        t.unshift(value.substring(i -= g, i + g));
+      g = d3_format_grouping[j = (j + 1) % d3_format_groupingLength];
-        g = d3_format_grouping[j = (j + 1) % d3_format_groupingLength];
-      }
-      return t.join(d3_format_thousandsSeparator || "") + f;
     }
-    return value.substring(0, i) + f;
+    return t.join(d3_format_thousandsSeparator || "") + f;
   };
 }