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

(see #274) fix AutoPairsFastWrap: remove wrong \ inside [] #269

Closed
wants to merge 2 commits into from

Conversation

tomtomjhj
Copy link

@tomtomjhj tomtomjhj commented Oct 2, 2019

'\v[\{\[\(\<]' matches '\' because the collection pattern (:help /[]) matches any single char in the collection even though \v is used. Because of this, AutoPairsFastWrap() on texts like {\bar{|}\asdf} didn't work.

This patch removes such wrong \s.

Squashed commit of the following:

commit 17bc0d0
Author: Léopold Mebazaa <[email protected]>
Date:   Fri Oct 11 22:30:23 2019 -0400

    Return to sanity

commit b1c7fea
Author: Léopold Mebazaa <[email protected]>
Date:   Fri Oct 11 22:28:58 2019 -0400

    Added logging for debug

commit 562a72e
Author: Léopold Mebazaa <[email protected]>
Date:   Fri Oct 11 18:57:41 2019 -0400

    Hypothesis correct, turns out

commit c126e3e
Author: Léopold Mebazaa <[email protected]>
Date:   Fri Oct 11 18:53:32 2019 -0400

    hypothesis 1: multiline is culprit

commit 64966a9
Author: Léopold Mebazaa <[email protected]>
Date:   Fri Oct 11 18:36:24 2019 -0400

    Added log to debug AutoPairDelete()]

commit 5f254b3
Author: Léopold Mebazaa <[email protected]>
Date:   Fri Oct 11 12:51:49 2019 -0400

    Update auto-pairs.vim
@tomtomjhj tomtomjhj closed this Oct 15, 2019
@tomtomjhj
Copy link
Author

closed in favor of #274 because I selected the wrong branch

@tomtomjhj tomtomjhj changed the title fix AutoPairsFastWrap: remove wrong \ inside [] (see #274) fix AutoPairsFastWrap: remove wrong \ inside [] Oct 15, 2019
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.

1 participant