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

refactor: Rename (dtoa|itoa)_stream to (dtoa|itoa)_buffered + simplify signature #1339

Merged
merged 2 commits into from
Jun 14, 2020
Merged

refactor: Rename (dtoa|itoa)_stream to (dtoa|itoa)_buffered + simplify signature #1339

merged 2 commits into from
Jun 14, 2020

Conversation

MaxGraey
Copy link
Member

@MaxGraey MaxGraey commented Jun 14, 2020

For better consistency I wanted to do this a long time ago.

Not sure is it breaking changing or not due to this methods usually used internally.

  • I've read the contributing guidelines

@MaxGraey MaxGraey requested a review from dcodeIO June 14, 2020 16:00
@MaxGraey
Copy link
Member Author

cc @willemneal @jtenner Did you use dtoa_stream and itoa_stream?

@MaxGraey MaxGraey changed the title refactor: Rename (dtoa|itoa)_stream to (dtoa|itoa)_buffered refactor: Rename (dtoa|itoa)_stream to (dtoa|itoa)_buffered + simplify signature Jun 14, 2020
@jtenner
Copy link
Contributor

jtenner commented Jun 14, 2020

Not directly. I do suppose I probably should be using it, but long.js works fine for me.

Copy link
Member

@dcodeIO dcodeIO left a comment

Choose a reason for hiding this comment

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

Looks good to me! I remember that I've been using these directly in other parts of stdlib to avoid overhead, so there is a chance that users may have done this in their own programs. Suggesting to just increase the version to be safe (so most people will just skip over 0.11).

@dcodeIO dcodeIO merged commit 0e398c4 into AssemblyScript:master Jun 14, 2020
@MaxGraey MaxGraey deleted the refactor-dtoa-stream branch June 14, 2020 21:16
@github-actions
Copy link

🎉 This PR is included in version 0.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants