309
100%
master: 100%

Ran 04 Jul 2019 03:13PM UTC

Jobs 4

Files 25

Run time 19s

Badge

Embed ▾

pending completion

Build # 309

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%)

125.91 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	309.1	04 Jul 2019 03:13PM UTC	100.0	Travis Job 309.1
2	309.2	04 Jul 2019 03:13PM UTC	100.0	Travis Job 309.2
3	309.3	04 Jul 2019 03:13PM UTC	100.0	Travis Job 309.3
4	309.4 (dependencies=lowest)	04 Jul 2019 03:13PM UTC	100.0	Travis Job 309.4

Source Files on build 309

Detailed source file information is not available for this build.

brick / money / 309
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 309

brick / money / 309 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 309

brick / money / 309
100%
master: 100%

README BADGES
x