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

Skip processing if buffer not modified #98

Merged
merged 1 commit into from
Apr 5, 2021
Merged

Conversation

bootleq
Copy link
Contributor

@bootleq bootleq commented Apr 1, 2021

An possible solution to #43, can also replace #96.

An possible solution to eraserhd#43, can also replace eraserhd#96.
@eraserhd eraserhd merged commit 9e41222 into eraserhd:master Apr 5, 2021
@eraserhd
Copy link
Owner

eraserhd commented Apr 5, 2021

Thanks!

@bootleq bootleq deleted the lazy branch April 6, 2021 02:12
eraserhd added a commit that referenced this pull request Apr 30, 2021
This reverts commit f570656.

Turns out this broke things: if paren mode isn't run before other changes
are attempted, the other changes are processed in smart mode, and the
parenthesis/indent is broken, messing up the file.

The future solution is to delay paren mode until a change is made, but
it still must be run before the first change is applied to avoid breaking
the file.

See #98
@eraserhd
Copy link
Owner

I reverted this, with these notes:

Turns out this broke things: if paren mode isn't run before other changes
are attempted, the other changes are processed in smart mode, and the
parenthesis/indent is broken, messing up the file.

The future solution is to delay paren mode until a change is made, but
it still must be run before the first change is applied to avoid breaking
the file.

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.

2 participants