Skip to content

Conversation

emilio
Copy link
Contributor

@emilio emilio commented Nov 6, 2016

This implements another feature that we need for parity with upstream bindgen.

r? @fitzgen

@emilio emilio force-pushed the dont-override-float branch 2 times, most recently from 2073bbb to 6a621b8 Compare November 6, 2016 22:00
Copy link
Member

@fitzgen fitzgen left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -63,6 +63,8 @@ Options:
--use-msvc-mangling Handle MSVC C++ ABI mangling; requires that
target be set to (i686|x86_64)-pc-win32

--no-convert-floats Don't convert floats automatically to f32/f64
Copy link
Member

Choose a reason for hiding this comment

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

NItpick: add a period to complete the sentence.

@emilio emilio force-pushed the dont-override-float branch from 6a621b8 to 1091a42 Compare November 7, 2016 22:11
@emilio
Copy link
Contributor Author

emilio commented Nov 7, 2016

@bors-servo r=fitzgen

@bors-servo
Copy link

📌 Commit 1091a42 has been approved by fitzgen

@bors-servo
Copy link

⌛ Testing commit 1091a42 with merge 6c0d065...

bors-servo pushed a commit that referenced this pull request Nov 8, 2016
Add an option to avoid converting to f32/f64 automatically float types.

This implements another feature that we need for parity with upstream bindgen.

r? @fitzgen
@bors-servo
Copy link

☀️ Test successful - status-travis

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