ae_fast_decimal_formatter 0.1.0 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c8d0765a81889d9e8cb54d9961fd5bc6e7daf282e767bc10f33fb5c6660df418
-  data.tar.gz: 8f0064baa8132ac69a131d3c45a3917d1288672815a7bd280ff5c718e9e62b05
+  metadata.gz: bfad6f29472a1ffc004d85c66182f379905f04d968324acba49ded7c3bf277c9
+  data.tar.gz: b351216f297f3c692f42fd44e0ba917a3405e677c567fa9206d6d8f4cba936c3
 SHA512:
-  metadata.gz: ce7e446c20366f9b69bdf58285d6b960957ecbd96f88ca427d6d3e9e9c80cb5b0fa325b21f726544bb33b888c49aef309d16db76d1bee44a4ab357efbb757a0e
-  data.tar.gz: 55fe31c15e4a985a36cb6580685a82043407761c4c086f2da809ad6ff421a0cee776555ba712ac0190577e2abd8e8d2a08dcd017e87687095c1ddeb0651c6fb4
+  metadata.gz: d3aa9e8b0438d4f4f0dd59f286685f4d2e0954ca6b50f6aed8a02c6e819e9be2abc7886468931e2e9cf2ea0bf70525decd5364468b99b30944dc74e2e40e08b2
+  data.tar.gz: 053b1593c4b4ed4ab05cbc1cc481361a65d9dac6b05ae1ec5f03ec06c931f712ababb7665e7cfefe6d129e2d4a9a3d7717772ca9b7502a28bed1db5e044f3946

data/LICENSE.txt

@@ -0,0 +1,20 @@
+Copyright (c) 2019-2022 AppFolio, Inc
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/ae_fast_decimal_formatter.gemspec

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+require_relative 'lib/ae_fast_decimal_formatter/version'
+
+Gem::Specification.new do |spec|
+  spec.name                  = 'ae_fast_decimal_formatter'
+  spec.version               = AeFastDecimalFormatter::VERSION
+  spec.platform              = Gem::Platform::RUBY
+  spec.author                = 'AppFolio'
+  spec.email                 = 'opensource@appfolio.com'
+  spec.description           = 'Efficiently format decimal number.'
+  spec.summary               = spec.description
+  spec.homepage              = 'https://github.com/appfolio/ae_fast_decimal_formatter'
+  spec.license               = 'MIT'
+  spec.files                 = Dir['**/*'].select { |f| f[%r{^(lib/|ext/|LICENSE.txt|.*gemspec)}] }
+  spec.require_paths         = ['lib']
+  spec.extensions            = ['ext/ae_fast_decimal_formatter/extconf.rb']
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.6.3')
+
+  spec.metadata['allowed_push_host'] = 'https://rubygems.org'
+
+  spec.add_dependency('addressable', ['>= 2.3.6', '< 3'])
+  spec.add_dependency('rack', ['>= 2.2.3', '< 3'])
+end

data/ext/ae_fast_decimal_formatter/ae_fast_decimal_formatter.c

@@ -1,106 +1,59 @@
 #include <ruby.h>
 #include <float.h>
 
-VALUE c_format_decimal(double num, int precision)
-{
-  char numstr[100];
-  char str[DBL_MAX_10_EXP + 2];
-  int numi, numlen, stri, strl;
-  int maxp = 5, i;
-  char formatstr[10];
-  int digits_to_comma;
-  double rounding_factor, rounded_num;
+inline static char * long_to_formatted_string(long value, char *str, unsigned int strlen, unsigned int precision) {
+  char *p;
+  unsigned long v;
 
-  if (precision > maxp) { precision = maxp; }
-  if (precision < 0) { precision = 0; }
+  v = (value < 0) ? -value : value;
+  p = str + strlen - 1;
 
-  snprintf(formatstr, 10, "%%.%df", precision);
-  formatstr[9] = 0;
-
-  if (precision > 0) {
-    rounding_factor = pow(10.0, precision);
-    rounded_num = round(num * rounding_factor) / rounding_factor;
-  } else {
-    rounded_num = round(num);
-  }
-
-  if (precision > 0) { precision++; } // adjust for .
-
-  numlen = snprintf(numstr, 100, formatstr, rounded_num);
-  digits_to_comma = numlen - precision;
-  if (numstr[0] == '-') { digits_to_comma--; }
-
-  strl = numlen + (digits_to_comma / 3);
-  if ((digits_to_comma % 3) == 0) {
-    strl--;
-  }
-
-  if (strncmp(numstr, "-0.00000", strl) == 0) {
-    return rb_str_new("0.00000", 1 + precision);
+  // Copy all the precision digits
+  for (unsigned int i = 0; i < precision; i++) {
+    *p-- = '0' + (v % 10);
+    v /= 10;
   }
 
-  stri = strl;
-  str[stri] = 0;
-  for (i = 1; i <= (precision + 1); i++) {
-    str[stri - i] = numstr[numlen - i];
+  // Add the dot
+  if (precision > 0) {
+    *p-- = '.';
   }
 
-  stri -= (precision + 2);
-  numlen -= (precision + 1);
-  numi = 1;
+  // Copy all the digits, adding a comma every 3rd digit
+  int digits = 0;
+  do {
+    *p-- = '0' + (v % 10);
+    v /= 10;
 
-  while ((numlen - numi) >= 1) {
-    if ((numi % 3) == 0) {
-      str[stri] = ',';
-      stri--;
+    if ((++digits % 3) == 0 && v) {
+      *p-- = ',';
     }
-    str[stri] = numstr[numlen - numi];
-    stri--;
-    numi++;
-  }
+  } while(v);
 
-  if (numstr[0] == '-') {
-    str[0] = '-';
-  } else {
-    if ((numi % 3) == 0) {
-      str[1] = ',';
-    }
-    str[0] = numstr[0];
-  }
-  return rb_str_new2(str);
-}
-
-struct ae_fast_decimal_formatter {
-  double num;
-  int precision;
-};
-
-static void ae_fast_decimal_formatter_free(void *p) {}
+  // Finally, add the - if we started with a negative number
+  if (value < 0) *p-- = '-';
 
-static VALUE ae_fast_decimal_formatter_alloc(VALUE klass) {
-  VALUE obj;
-  struct ae_fast_decimal_formatter *ptr;
-
-  obj = Data_Make_Struct(klass, struct ae_fast_decimal_formatter, NULL, ae_fast_decimal_formatter_free, ptr);
-
-  ptr->num = 0.0;
-  ptr->precision = 0;
-
-  return obj;
-}
+  // And adjust the p to undo the last p--
+  p++;
 
-static VALUE ae_fast_decimal_formatter_init(VALUE self, VALUE number, VALUE precision) {
-  struct ae_fast_decimal_formatter *ptr;
-  Data_Get_Struct(self, struct ae_fast_decimal_formatter, ptr);
-  ptr->num = NUM2DBL(number);
-  ptr->precision = NUM2UINT(precision);
-  return self;
+  return p;
 }
 
-static VALUE ae_fast_decimal_formatter_format(VALUE self) {
-  struct ae_fast_decimal_formatter *ptr;
-  Data_Get_Struct(self, struct ae_fast_decimal_formatter, ptr);
-  return c_format_decimal(ptr->num, ptr->precision);
+// The function takes a value num (assumed to be fixnum) and value precision (assumed to be fixnum
+// between 0 and 5) and returns num as a formatted string.
+//
+// For example, given 100, 2 the function will output "1.00".
+// For example, given 123456, 2 the function will output "1,234.56".
+// For example, given -1234, 0 the function will output "-1,234".
+static VALUE ae_fast_decimal_formatter_format_long(VALUE self, VALUE num, VALUE precision) {
+  // The maximum length is 28 characters: 20 digits (assuming 64bits), 6 commas,
+  // negative sign, and a period. So we will always fit in 32 chars.
+  char buf[32];
+  char *numstr = long_to_formatted_string(NUM2LONG(num), buf, 32, NUM2UINT(precision));
+
+  // The rb_str_new takes a pointer to a string and a length. It does not rely on
+  // null terminated strings.
+  return rb_str_new(numstr, 32 - (int)(numstr - buf));
 }
 
 void Init_ae_fast_decimal_formatter(void) {
@@ -108,7 +61,5 @@ void Init_ae_fast_decimal_formatter(void) {
 
   cFastDecimalFormatter = rb_const_get(rb_cObject, rb_intern("AeFastDecimalFormatter"));
 
-  rb_define_alloc_func(cFastDecimalFormatter, ae_fast_decimal_formatter_alloc);
-  rb_define_method(cFastDecimalFormatter, "initialize", ae_fast_decimal_formatter_init, 2);
-  rb_define_method(cFastDecimalFormatter, "format", ae_fast_decimal_formatter_format, 0);
+  rb_define_singleton_method(cFastDecimalFormatter, "format_long", ae_fast_decimal_formatter_format_long, 2);
 }

data/ext/ae_fast_decimal_formatter/extconf.rb

@@ -1,6 +1,3 @@
 require "mkmf"
 
-abort "missing snprintf()" unless have_func "snprintf"
-abort "missing strncmp()"  unless have_func "strncmp"
-
 create_makefile "ae_fast_decimal_formatter/ae_fast_decimal_formatter"

data/lib/ae_fast_decimal_formatter/version.rb

@@ -0,0 +1,3 @@
+class AeFastDecimalFormatter
+  VERSION = '2.0.0'
+end

data/lib/ae_fast_decimal_formatter.rb

@@ -1,4 +1,23 @@
+require 'bigdecimal'
+
 class AeFastDecimalFormatter
+  def self.format(number, precision)
+    precision = 0 if precision < 0
+    precision = 5 if precision > 5
+
+    if number.is_a?(BigDecimal)
+      # With BigDecimal, we do not need to call round twice.
+      format_long((number * 10 ** precision).round, precision)
+    else
+      # It looks like we need the two sets of rounds in order to attempt
+      # to round floating numbers. For example,
+      # (148.855 * 100).round -> 14885 seems incorrect, so we try,
+      # 148.855.round(2) * 100 -> 14886 but then,
+      # (1234567890.12.round(2) * 100).to_i -> 123456789011 which is
+      # definitely not correct. Thus the two rounds.
+      format_long((number.round(precision) * 10 ** precision).round, precision)
+    end
+  end
 end
 
 require "ae_fast_decimal_formatter/ae_fast_decimal_formatter"

metadata

@@ -1,32 +1,75 @@
 --- !ruby/object:Gem::Specification
 name: ae_fast_decimal_formatter
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 2.0.0
 platform: ruby
 authors:
-- Appfolio Inc.
-autorequire: 
+- AppFolio
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-05-14 00:00:00.000000000 Z
-dependencies: []
-description: Efficiently format decimal number so that reports of millions of decimal
-  cells are fast
-email: dev@appfolio.com
+date: 2022-03-01 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: addressable
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.3.6
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.3.6
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+- !ruby/object:Gem::Dependency
+  name: rack
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.2.3
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.2.3
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
+description: Efficiently format decimal number.
+email: opensource@appfolio.com
 executables: []
 extensions:
 - ext/ae_fast_decimal_formatter/extconf.rb
 extra_rdoc_files: []
 files:
+- LICENSE.txt
+- ae_fast_decimal_formatter.gemspec
 - ext/ae_fast_decimal_formatter/ae_fast_decimal_formatter.c
 - ext/ae_fast_decimal_formatter/extconf.rb
 - lib/ae_fast_decimal_formatter.rb
-homepage: https://www.github.com/appfolio/ae_fast_decimal_formatter
+- lib/ae_fast_decimal_formatter/ae_fast_decimal_formatter.bundle
+- lib/ae_fast_decimal_formatter/version.rb
+homepage: https://github.com/appfolio/ae_fast_decimal_formatter
 licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -34,18 +77,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.3.3
-  - - "<"
-    - !ruby/object:Gem::Version
-      version: '2.7'
+      version: 2.6.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
-summary: Efficiently format decimal number
+summary: Efficiently format decimal number.
 test_files: []