Skip to content

Conversation

@jethrogb
Copy link
Contributor

@jethrogb jethrogb commented Feb 1, 2019

It was 0-indexed but should be 1-indexed. This PR just removes the duplicate code and re-exports the internal enum.

Fixes fortanix/rust-sgx#88

r? @joshtriplett

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 1, 2019
@joshtriplett
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 3, 2019

📌 Commit a90b23f has been approved by joshtriplett

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 3, 2019
@bors
Copy link
Collaborator

bors commented Feb 3, 2019

⌛ Testing commit a90b23f with merge 2966fbc...

bors added a commit that referenced this pull request Feb 3, 2019
Fix `std::os::fortanix_sgx::usercalls::raw::UsercallNrs`

It was 0-indexed but should be 1-indexed. This PR just removes the duplicate code and re-exports the internal enum.

Fixes fortanix/rust-sgx#88

r? @joshtriplett
@coronelargie
Copy link

greats

@bors
Copy link
Collaborator

bors commented Feb 3, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: joshtriplett
Pushing 2966fbc to master...

@bors bors merged commit a90b23f into rust-lang:master Feb 3, 2019
@workingjubilee workingjubilee added the O-SGX Target: SGX label Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

O-SGX Target: SGX S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

std::os::fortanix_sgx::usercalls::raw::UsercallNrs wrong

6 participants