Skip to content

/ rust-smallvec

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 Array impl for [T; 0] #88

Merged
merged 1 commit into from Mar 20, 2018
Merged

Add Array impl for [T; 0] #88

merged 1 commit into from Mar 20, 2018
+12 −3

Conversation

@Vurich
Copy link
Contributor

Vurich commented Mar 20, 2018 •
edited by larsbergstrom

This seems useless but has benefits in generic contexts (plus, with smart-enough compiler optimisations this could conceivably compile to equivalent code to Vec<T>).

This change is Reviewable
@Vurich
Add Array impl for [T; 0]
Loading status checks…
cccd873
@jdm
Copy link
Member

jdm commented Mar 20, 2018

@bors-servo r+
Thanks!
@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2018

📌 Commit cccd873 has been approved by jdm
bors-servo added a commit that referenced this pull request Mar 20, 2018
@bors-servo
Auto merge of #88 - Vurich:array-zero, r=jdm
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
4749fc2 
Add Array impl for [T; 0]

This seems useless but has benefits in generic contexts (plus, with smart-enough compiler optimisations this could conceivably compile to equivalent code to `Vec<T>`).

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-smallvec/88)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2018

⌛ Testing commit cccd873 with merge 4749fc2...
@bors-servo
Copy link
Contributor

bors-servo commented Mar 20, 2018

☀️ Test successful - status-travis
Approved by: jdm
Pushing 4749fc2 to master...
@bors-servo bors-servo merged commit cccd873 into servo:master Mar 20, 2018
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Vurich
Copy link
Contributor Author

Vurich commented Mar 20, 2018

Wow, that was fast!
mbrubeck added a commit to mbrubeck/rust-smallvec that referenced this pull request Apr 6, 2018
@mbrubeck
Version 0.6.1
53617f2 
Change log for this release:

* Add `smallvec!` macro (servo#90)
* Implement `Array` trait for `[T; 0]` (servo#88)
* Add `resize` method (servo#82)
* Correct a typo in the docs (servo#81)
@mbrubeck mbrubeck mentioned this pull request Apr 6, 2018
Merged
bors-servo added a commit that referenced this pull request Apr 23, 2018
@bors-servo
Auto merge of #91 - mbrubeck:bump, r=KiChjang
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
d1a64d4 
Version 0.6.1

Change log for this release:

* Add `smallvec!` macro (#90)
* Implement `Array` trait for `[T; 0]` (#88)
* Add `resize` method (#82)
* Correct a typo in the docs (#81)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-smallvec/91)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@Vurich @jdm @bors-servo
You can’t perform that action at this time.