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

enable sprintf to support printing floats, let the user choose. #1974

Closed
aguaviva opened this issue Apr 25, 2016 · 6 comments
Closed

enable sprintf to support printing floats, let the user choose. #1974

aguaviva opened this issue Apr 25, 2016 · 6 comments

Comments

@aguaviva
Copy link
Contributor

Given that an ESP is as powerful as a Pentium Pro, does it make sense to remove the floating point support just to save some mem?

At a minimum we should let the user choose whether to support %f feature or not.

@TheAustrian
Copy link

I concatenate with String(floatVariable, numberOfDecimals) to print floats - I don't know if that helps in your application, though. I would not mind having %f, though.

@aguaviva
Copy link
Contributor Author

thanks and I'd also appreciate a proper fix, there a lots of confused people out there.

Even basic could print floating point numbers :) lets fix this please :)

@igrr
Copy link
Member

igrr commented Apr 25, 2016

Closing as duplicate of #1179.

@igrr igrr closed this as completed Apr 25, 2016
@aguaviva
Copy link
Contributor Author

Is anybody working in this? If anybody submits a pull request with the c implementation used in gcc, would it be accepted?

@igrr
Copy link
Member

igrr commented Apr 26, 2016

This can be implemented as part of a pending pull request: #1752.
Please follow up in the issue linked above: #1179.

@aguaviva
Copy link
Contributor Author

Thanks, this helps! BTW you guys rock!

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

No branches or pull requests

3 participants