Skip to content

Uplift repr(packed) fix to zerovec 0.9#5091

Merged
Manishearth merged 1 commit into
unicode-org:tmp/zerovec@0.9.6from
Manishearth:tmp/zerovec@0.9.6
Jun 24, 2024
Merged

Uplift repr(packed) fix to zerovec 0.9#5091
Manishearth merged 1 commit into
unicode-org:tmp/zerovec@0.9.6from
Manishearth:tmp/zerovec@0.9.6

Conversation

@Manishearth
Copy link
Copy Markdown
Member

Uplifts #5049 to zerovec 0.9.7

@Manishearth Manishearth requested a review from sffc as a code owner June 20, 2024 18:09
Fixes unicode-org#5039

Caused by rust-lang/rust#125360. We were
assuming that `packed` meant `C, packed` already. This is an assumption
I've seen throughout the Rust ecosystem so there may be reasons to
revert.
@Manishearth
Copy link
Copy Markdown
Member Author

We performed some out of cycle zerovec releases and we don't have a release/1.2 branch, so I just made a temporary branch as a part of this history. Once this merges I'll merge it into the main history so the commit is retained, and tag it.

Copy link
Copy Markdown
Member

@sffc sffc left a comment

Choose a reason for hiding this comment

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

Rslgtm

@Manishearth Manishearth merged commit f427697 into unicode-org:tmp/zerovec@0.9.6 Jun 24, 2024
@Manishearth Manishearth deleted the tmp/zerovec@0.9.6 branch June 24, 2024 17:34
Manishearth added a commit that referenced this pull request Jun 25, 2024
This includes CHANGELOG updates from #5079, #5080, (not #5091, it didn't
include a changelog update, we weren't doing changelogs back then)


alternative to #5109


<!--
Thank you for your pull request to ICU4X!

Reminder: try to use [Conventional
Comments](https://conventionalcomments.org/) to make comments clearer.

Please see
https://github.com/unicode-org/icu4x/blob/main/CONTRIBUTING.md for
general
information on contributing to ICU4X.
-->
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