Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore v0.16.0 id_pat #454

Open
wants to merge 1 commit into
base: impl-reader-seek
Choose a base branch
from
Open

Restore v0.16.0 id_pat #454

wants to merge 1 commit into from

Conversation

wcampbell0x2a
Copy link
Collaborator

  • When moving to the Read powered impl, I forced the user to use the same type as the id when reading into an id_pat With the use of Seek, we can now fix that and just seek back to the position before the id was read in the case of id_pat

Copy link

Benchmark for 22221b6

Click to view benchmark
Test Base PR %
deku_read_bits 1281.4±11.79ns 1247.9±26.80ns -2.61%
deku_read_byte 3.4±0.07ns 3.4±0.06ns 0.00%
deku_read_enum 2.6±0.04ns 2.6±0.07ns 0.00%
deku_read_vec 35.5±2.16ns 36.8±0.95ns +3.66%
deku_write_bits 173.6±6.01ns 167.2±9.96ns -3.69%
deku_write_byte 24.0±0.36ns 24.3±1.73ns +1.25%
deku_write_enum 22.6±0.37ns 22.6±0.38ns 0.00%
deku_write_vec 415.8±6.26ns 417.1±3.39ns +0.31%

* When moving to the Read powered impl, I forced the user to
  use the same type as the id when reading into an id_pat
  With the use of Seek, we can now fix that and just seek
  back to the position before the id was read in the case of id_pat
Copy link

Benchmark for 556c1e7

Click to view benchmark
Test Base PR %
deku_read_bits 1273.3±19.06ns 1201.5±15.42ns -5.64%
deku_read_byte 3.4±0.09ns 3.4±0.12ns 0.00%
deku_read_enum 2.6±0.04ns 2.6±0.04ns 0.00%
deku_read_vec 34.9±0.57ns 36.4±0.60ns +4.30%
deku_write_bits 183.4±4.70ns 165.6±4.03ns -9.71%
deku_write_byte 24.0±0.24ns 24.1±0.49ns +0.42%
deku_write_enum 22.7±0.45ns 22.7±0.34ns 0.00%
deku_write_vec 420.2±7.31ns 414.7±6.46ns -1.31%

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant