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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimize removeTrailingWhitespace function #140

Merged
merged 1 commit into from Dec 16, 2016

Conversation

Projects
None yet
1 participant
@maxbrunsfeld
Copy link
Contributor

maxbrunsfeld commented Dec 16, 2016

This code was very inefficient; it caused saving a large file to take forever, even if there was no trailing whitespace.

馃崘 'd with @nathansobo

/cc @ungb @Ben3eeE

Optimize removeTrailingWhitespace function
Signed-off-by: Nathan Sobo <nathan@github.com>

@maxbrunsfeld maxbrunsfeld merged commit 63074fd into master Dec 16, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@maxbrunsfeld maxbrunsfeld deleted the mb-ns-optimize branch Dec 16, 2016

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