4
100%
master: 100%

Ran 04 Jul 2019 03:10PM UTC

Files 25

Run time 3s

Badge

Embed ▾

Committed 04 Jul 2019 03:08PM UTC coverage: 100.0%. Remained the same

Job # dependencies=lowest

Build Type

push

travis-ci

Committed by

BenMorel

Commit Message

Cache Money::formatTo()'s NumberFormatter for improved performance

Calling formatTo() many times spends quite some time in NumberFormatter constructor, that performs I/O, according to the profiler.
Because we usually call formatTo() many times in a row for the same locale, we can get a ~2x speedup with this fix.

The best way to tackle this in the future would be to deprecate the formatTo() method and introduce a MoneyFormatter object, that would keep a local cache of all locale/scale pairs.

Run Details

427 of 427 relevant lines covered (100.0%)

31.51 hits per line

brick / money / 308 / 4
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 308.4 (dependencies=lowest)

brick / money / 308 / 4 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 308.4 (dependencies=lowest)

brick / money / 308 / 4
100%
master: 100%

README BADGES
x