Skip to content

Properly support no_std#312

Closed
Manishearth wants to merge 1 commit intomasterfrom
no-std
Closed

Properly support no_std#312
Manishearth wants to merge 1 commit intomasterfrom
no-std

Conversation

@Manishearth
Copy link
Copy Markdown
Member

@Manishearth Manishearth commented Jan 9, 2019

num-traits isn't no-std by default, and we use a bunch of stuff from it that requires std

r? @SimonSapin


This change is Reviewable

}
}

#[cfg(feature = "std")]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why can't this use FloatCore?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

no trigonometry in FloatCore

@bors-servo
Copy link
Copy Markdown
Contributor

☔ The latest upstream changes (presumably #314) made this pull request unmergeable. Please resolve the merge conflicts.

@nical
Copy link
Copy Markdown
Contributor

nical commented Jul 4, 2019

@Manishearth other than rebasing, is there anything keeping us from going through with this?

@Manishearth
Copy link
Copy Markdown
Member Author

Rebasing and review

@nical
Copy link
Copy Markdown
Contributor

nical commented Nov 25, 2019

Fixed in #372

@nical nical closed this Nov 25, 2019
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.

4 participants