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

Add from_const_with_len_unchecked #329

Merged
merged 1 commit into from
Jan 13, 2024
Merged

Add from_const_with_len_unchecked #329

merged 1 commit into from
Jan 13, 2024

Conversation

Expyron
Copy link

@Expyron Expyron commented Nov 18, 2023

Fixes #301

I made the method unsafe to be able to use the existing const_new feature without raising its MSRV to 1.57, which would have been necessary for const-panicking.

This is v1-only. v2 already has const from_buf_and_len_unchecked, so this use-case is already covered there.

@mbrubeck mbrubeck merged commit c638a0b into servo:v1 Jan 13, 2024
@mbrubeck
Copy link
Collaborator

Sorry for the delay. This has been published as smallvec 1.12.0.

maxdeviant referenced this pull request in zed-industries/zed Jul 25, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [smallvec](https://github.com/servo/rust-smallvec) |
workspace.dependencies | minor | `1.11.1` -> `1.13.2` |

---

### Release Notes

<details>
<summary>servo/rust-smallvec (smallvec)</summary>

###
[`v1.13.2`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.2)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.13.1...v1.13.2)

#### What's Changed

- Add more tests for UB by
[@&#8203;workingjubilee](https://github.com/workingjubilee) in
[https://github.com/servo/rust-smallvec/pull/346](https://github.com/servo/rust-smallvec/pull/346)
- Fix UB on out-of-bounds insert() by
[@&#8203;mbrubeck](https://github.com/mbrubeck) in
[https://github.com/servo/rust-smallvec/pull/345](https://github.com/servo/rust-smallvec/pull/345)

**Full Changelog**:
servo/rust-smallvec@v1.13.1...v1.13.2

###
[`v1.13.1`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.1)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.13.0...v1.13.1)

- Remove the optional `get-size` feature, to avoid a cyclic dependency
([#&#8203;335](https://github.com/servo/rust-smallvec/issues/335)).

**Full Changelog**:
servo/rust-smallvec@v1.13.0...v1.13.1

###
[`v1.13.0`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.0)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.12.0...v1.13.0)

#### What's Changed

- Impl get_size::GetSize (behind feature flag) by
[@&#8203;amandasaurus](https://github.com/amandasaurus) in
[https://github.com/servo/rust-smallvec/pull/335](https://github.com/servo/rust-smallvec/pull/335)

**Full Changelog**:
servo/rust-smallvec@v1.12.0...v1.13.0

###
[`v1.12.0`](https://github.com/servo/rust-smallvec/releases/tag/v1.12.0)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.11.2...v1.12.0)

#### What's Changed

- Add `from_const_with_len_unchecked` by
[@&#8203;Expyron](https://github.com/Expyron) in
[https://github.com/servo/rust-smallvec/pull/329](https://github.com/servo/rust-smallvec/pull/329)

#### New Contributors

- [@&#8203;Expyron](https://github.com/Expyron) made their first
contribution in
[https://github.com/servo/rust-smallvec/pull/329](https://github.com/servo/rust-smallvec/pull/329)

**Full Changelog**:
servo/rust-smallvec@v1.11.2...v1.12.0

###
[`v1.11.2`](https://github.com/servo/rust-smallvec/releases/tag/v1.11.2)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.11.1...v1.11.2)

#### What's Changed

- Automated testing improvements by
[@&#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/322](https://github.com/servo/rust-smallvec/pull/322)
and
[https://github.com/servo/rust-smallvec/pull/326](https://github.com/servo/rust-smallvec/pull/326)
- fix: don't special-case `doc` for `feature = "const_generics"` by
[@&#8203;mkroening](https://github.com/mkroening) in
[https://github.com/servo/rust-smallvec/pull/328](https://github.com/servo/rust-smallvec/pull/328)
- Code cleanup by [@&#8203;emilio](https://github.com/emilio) in
[https://github.com/servo/rust-smallvec/pull/316](https://github.com/servo/rust-smallvec/pull/316)
and [@&#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/323](https://github.com/servo/rust-smallvec/pull/323)
- Minor tweaks to doc formatting. by
[@&#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/318](https://github.com/servo/rust-smallvec/pull/318)

#### New Contributors

- [@&#8203;mkroening](https://github.com/mkroening) made their first
contribution in
[https://github.com/servo/rust-smallvec/pull/328](https://github.com/servo/rust-smallvec/pull/328)

**Full Changelog**:
servo/rust-smallvec@v1.11.1...v1.11.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 3pm on Wednesday" in timezone
America/New_York, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

Release Notes:

- N/A

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
CharlesChen0823 referenced this pull request in CharlesChen0823/zed Jul 29, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [smallvec](https://github.com/servo/rust-smallvec) |
workspace.dependencies | minor | `1.11.1` -> `1.13.2` |

---

### Release Notes

<details>
<summary>servo/rust-smallvec (smallvec)</summary>

###
[`v1.13.2`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.2)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.13.1...v1.13.2)

#### What's Changed

- Add more tests for UB by
[@&zed-industries#8203;workingjubilee](https://github.com/workingjubilee) in
[https://github.com/servo/rust-smallvec/pull/346](https://github.com/servo/rust-smallvec/pull/346)
- Fix UB on out-of-bounds insert() by
[@&zed-industries#8203;mbrubeck](https://github.com/mbrubeck) in
[https://github.com/servo/rust-smallvec/pull/345](https://github.com/servo/rust-smallvec/pull/345)

**Full Changelog**:
servo/rust-smallvec@v1.13.1...v1.13.2

###
[`v1.13.1`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.1)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.13.0...v1.13.1)

- Remove the optional `get-size` feature, to avoid a cyclic dependency
([#&zed-industries#8203;335](https://github.com/servo/rust-smallvec/issues/335)).

**Full Changelog**:
servo/rust-smallvec@v1.13.0...v1.13.1

###
[`v1.13.0`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.0)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.12.0...v1.13.0)

#### What's Changed

- Impl get_size::GetSize (behind feature flag) by
[@&zed-industries#8203;amandasaurus](https://github.com/amandasaurus) in
[https://github.com/servo/rust-smallvec/pull/335](https://github.com/servo/rust-smallvec/pull/335)

**Full Changelog**:
servo/rust-smallvec@v1.12.0...v1.13.0

###
[`v1.12.0`](https://github.com/servo/rust-smallvec/releases/tag/v1.12.0)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.11.2...v1.12.0)

#### What's Changed

- Add `from_const_with_len_unchecked` by
[@&zed-industries#8203;Expyron](https://github.com/Expyron) in
[https://github.com/servo/rust-smallvec/pull/329](https://github.com/servo/rust-smallvec/pull/329)

#### New Contributors

- [@&zed-industries#8203;Expyron](https://github.com/Expyron) made their first
contribution in
[https://github.com/servo/rust-smallvec/pull/329](https://github.com/servo/rust-smallvec/pull/329)

**Full Changelog**:
servo/rust-smallvec@v1.11.2...v1.12.0

###
[`v1.11.2`](https://github.com/servo/rust-smallvec/releases/tag/v1.11.2)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.11.1...v1.11.2)

#### What's Changed

- Automated testing improvements by
[@&zed-industries#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/322](https://github.com/servo/rust-smallvec/pull/322)
and
[https://github.com/servo/rust-smallvec/pull/326](https://github.com/servo/rust-smallvec/pull/326)
- fix: don't special-case `doc` for `feature = "const_generics"` by
[@&zed-industries#8203;mkroening](https://github.com/mkroening) in
[https://github.com/servo/rust-smallvec/pull/328](https://github.com/servo/rust-smallvec/pull/328)
- Code cleanup by [@&zed-industries#8203;emilio](https://github.com/emilio) in
[https://github.com/servo/rust-smallvec/pull/316](https://github.com/servo/rust-smallvec/pull/316)
and [@&zed-industries#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/323](https://github.com/servo/rust-smallvec/pull/323)
- Minor tweaks to doc formatting. by
[@&zed-industries#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/318](https://github.com/servo/rust-smallvec/pull/318)

#### New Contributors

- [@&zed-industries#8203;mkroening](https://github.com/mkroening) made their first
contribution in
[https://github.com/servo/rust-smallvec/pull/328](https://github.com/servo/rust-smallvec/pull/328)

**Full Changelog**:
servo/rust-smallvec@v1.11.1...v1.11.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 3pm on Wednesday" in timezone
America/New_York, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

Release Notes:

- N/A

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
kevmo314 referenced this pull request in kevmo314/zed Jul 29, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [smallvec](https://github.com/servo/rust-smallvec) |
workspace.dependencies | minor | `1.11.1` -> `1.13.2` |

---

### Release Notes

<details>
<summary>servo/rust-smallvec (smallvec)</summary>

###
[`v1.13.2`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.2)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.13.1...v1.13.2)

#### What's Changed

- Add more tests for UB by
[@&zed-industries#8203;workingjubilee](https://github.com/workingjubilee) in
[https://github.com/servo/rust-smallvec/pull/346](https://github.com/servo/rust-smallvec/pull/346)
- Fix UB on out-of-bounds insert() by
[@&zed-industries#8203;mbrubeck](https://github.com/mbrubeck) in
[https://github.com/servo/rust-smallvec/pull/345](https://github.com/servo/rust-smallvec/pull/345)

**Full Changelog**:
servo/rust-smallvec@v1.13.1...v1.13.2

###
[`v1.13.1`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.1)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.13.0...v1.13.1)

- Remove the optional `get-size` feature, to avoid a cyclic dependency
([#&zed-industries#8203;335](https://github.com/servo/rust-smallvec/issues/335)).

**Full Changelog**:
servo/rust-smallvec@v1.13.0...v1.13.1

###
[`v1.13.0`](https://github.com/servo/rust-smallvec/releases/tag/v1.13.0)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.12.0...v1.13.0)

#### What's Changed

- Impl get_size::GetSize (behind feature flag) by
[@&zed-industries#8203;amandasaurus](https://github.com/amandasaurus) in
[https://github.com/servo/rust-smallvec/pull/335](https://github.com/servo/rust-smallvec/pull/335)

**Full Changelog**:
servo/rust-smallvec@v1.12.0...v1.13.0

###
[`v1.12.0`](https://github.com/servo/rust-smallvec/releases/tag/v1.12.0)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.11.2...v1.12.0)

#### What's Changed

- Add `from_const_with_len_unchecked` by
[@&zed-industries#8203;Expyron](https://github.com/Expyron) in
[https://github.com/servo/rust-smallvec/pull/329](https://github.com/servo/rust-smallvec/pull/329)

#### New Contributors

- [@&zed-industries#8203;Expyron](https://github.com/Expyron) made their first
contribution in
[https://github.com/servo/rust-smallvec/pull/329](https://github.com/servo/rust-smallvec/pull/329)

**Full Changelog**:
servo/rust-smallvec@v1.11.2...v1.12.0

###
[`v1.11.2`](https://github.com/servo/rust-smallvec/releases/tag/v1.11.2)

[Compare
Source](https://github.com/servo/rust-smallvec/compare/v1.11.1...v1.11.2)

#### What's Changed

- Automated testing improvements by
[@&zed-industries#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/322](https://github.com/servo/rust-smallvec/pull/322)
and
[https://github.com/servo/rust-smallvec/pull/326](https://github.com/servo/rust-smallvec/pull/326)
- fix: don't special-case `doc` for `feature = "const_generics"` by
[@&zed-industries#8203;mkroening](https://github.com/mkroening) in
[https://github.com/servo/rust-smallvec/pull/328](https://github.com/servo/rust-smallvec/pull/328)
- Code cleanup by [@&zed-industries#8203;emilio](https://github.com/emilio) in
[https://github.com/servo/rust-smallvec/pull/316](https://github.com/servo/rust-smallvec/pull/316)
and [@&zed-industries#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/323](https://github.com/servo/rust-smallvec/pull/323)
- Minor tweaks to doc formatting. by
[@&zed-industries#8203;waywardmonkeys](https://github.com/waywardmonkeys) in
[https://github.com/servo/rust-smallvec/pull/318](https://github.com/servo/rust-smallvec/pull/318)

#### New Contributors

- [@&zed-industries#8203;mkroening](https://github.com/mkroening) made their first
contribution in
[https://github.com/servo/rust-smallvec/pull/328](https://github.com/servo/rust-smallvec/pull/328)

**Full Changelog**:
servo/rust-smallvec@v1.11.1...v1.11.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 3pm on Wednesday" in timezone
America/New_York, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

Release Notes:

- N/A

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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