1
77%
master: 77%

Ran 10 Apr 2024 10:21AM UTC

Files 361

Run time 15s

Badge

Embed ▾

Committed 09 Apr 2024 09:06PM UTC coverage: 81.402% (-0.02%) from 81.42%

Job # #37743.1

Build Type

push

local

Committed by

web-flow

Commit Message

Improve performance of `ncodeunits(::Char)` (#54001)

This improves performance of `ncodeunits(::Char)` by simply counting the
number of non-zero bytes (except for `\0`, which is encoded as all zero
bytes). For a performance comparison, see [this gist](
https://gist.github.com/Seelengrab/ebb02d4b8d754700c2869de8daf88cad);
there's an up to 10x improvement here for collections of `Char`, with a
minor improvement for single `Char` (with much smaller spread). The
version in this PR is called `nbytesencoded` in the benchmarks.

Correctness has been verified with Supposition.jl, using the existing
implementation as an oracle:

```julia
julia> using Supposition

julia> const chars = Data.Characters()

julia> @check max_examples=1_000_000 function bytesenc(i=Data.Integers{UInt32}())
           c = reinterpret(Char, i)
           ncodeunits(c) == nbytesdiv(c)
       end;
Test Summary: | Pass  Total  Time
bytesenc      |    1      1  1.0s

julia> ncodeunits('\0') == nbytesencoded('\0')
true
```

Let's see if CI agrees!

Notably, neither the existing nor the new implementation check whether
the given `Char` is valid or not, since the only thing that matters is
how many bytes are written out.

---------

Co-authored-by: Sukera <Seelengrab@users.noreply.github.com>

Run Details

70709 of 86864 relevant lines covered (81.4%)

15630252.81 hits per line

JuliaLang / julia / #37743 / 1
77%
master: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #37743.1

JuliaLang / julia / #37743 / 1 77% master: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job #37743.1

JuliaLang / julia / #37743 / 1
77%
master: 77%

README BADGES
x