1
98%
master: 98%

Ran 29 Nov 2018 04:18AM UTC

Files 1

Run time 0s

Badge

Embed ▾

Committed 12 Nov 2018 07:46PM UTC coverage: 98.824%. Remained the same

Job # 216.1

Build Type

cron

travis-ci

Committed by

omus

Commit Message

Performance fix: avoid unnecessary `convert` on setindex!、copy when type is same. (#38)

* Add `convert(::Type{T}, x::T) where {T<:FD} = x`

This fixes unnecessary copies on `setindex!`, etc.

* Remove "copy constructor" after correct `convert` fix

The "copy constructor" is no longer necessary now that a no-op
`convert` is added, since by default construction falls back to convert.

To verify, after this change, the performance is still correct:
```
julia> f = FixedPointDecimals.FixedDecimal{Int128,2}(3.25)
FixedDecimal{Int128,2}(3.25)

julia> @btime x = FixedPointDecimals.FixedDecimal{Int128, 2}($f)
  0.036 ns (0 allocations: 0 bytes)
FixedDecimal{Int128,2}(3.25)
```

And `@code_native` shows this to be a simple reinterpret.

Run Details

168 of 170 relevant lines covered (98.82%)

3348.49 hits per line

JuliaMath / FixedPointDecimals.jl / 216 / 1
98%
master: 98%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 216.1

JuliaMath / FixedPointDecimals.jl / 216 / 1 98% master: 98%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 216.1

JuliaMath / FixedPointDecimals.jl / 216 / 1
98%
master: 98%

README BADGES
x