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

Keywords Fix and Refactoring #7

Merged
merged 5 commits into from
May 8, 2019
Merged

Keywords Fix and Refactoring #7

merged 5 commits into from
May 8, 2019

Conversation

hummingly
Copy link

@hummingly hummingly commented Dec 22, 2018

  • Updates dependencies.
  • Fixes a bug where adding keywords would just overwrite/add the Subject entry.
  • Changes create_X functions to constructors that are only visible in the crate.
  • Improves allocations: removed a few clones, allocate space before inserting elements etc.
  • Puts graphicsstates tests into test module which removes dead code warning.
  • Overall made more use of the standard library.

@kaj
Copy link
Owner

kaj commented Dec 22, 2018

Thank you for this contribution! It looks good at a quick glance, but it's a lot of changes to read through, so it will take me some time before I merge it.

Copy link
Owner

@kaj kaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The most controversial thing here is probably the use of pub(crate), which was a bit new at the time, but should be considered well established now.

@kaj kaj merged commit 107c2c5 into kaj:master May 8, 2019
kaj added a commit that referenced this pull request Feb 7, 2022
* Merge PR #7 from @hummingly:
  - Updates dependencies.
  - Fixes a bug where adding keywords would just overwrite/add the Subject
    entry.
  - Changes create_X functions to constructors that are only visible in the
    crate.
  - Improves allocations: removed a few clones, allocate space before
    inserting elements etc.
  - Puts graphicsstates tests into test module which removes dead code
    warning.
  - Overall made more use of the standard library.
* Add keyword "pdf" to crate, PR #6 from @adiba
* Update to Rust edition 2021.
* Update `lazy_static` dependency.
* Use `chrono` 0.4.19 instead of `time` 0.1.
* Update travis build to use more up to date rust versions and stable
  rustfmt.
* Update clippy directive.
* Some rustfmt updates.

Thanks to @hummingly and @adiba!
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.

2 participants