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

Set curwp->w_markline after clearmark in markpara #22

Merged
merged 1 commit into from
Aug 9, 2023
Merged

Set curwp->w_markline after clearmark in markpara #22

merged 1 commit into from
Aug 9, 2023

Conversation

bscuron
Copy link
Contributor

@bscuron bscuron commented Aug 9, 2023

Fixes bug where the current markline is not reset after being cleared.

This makes it possible to now mark a region, and use apply-macro-to-region-lines since the markline is now set in the correct position:

screen.mov

Fixes bug where the current markline is not reset after being
cleared.
@troglobit
Copy link
Owner

Interesting, could you please provide a sequence of commands to reproduce the problem?

@bscuron
Copy link
Contributor Author

bscuron commented Aug 9, 2023

It happens when you press M-h to mark paragraph. Here is a screencast showing me logging the markline value after pressing M-h before the fix and then after the fix.

screen.mp4

@troglobit
Copy link
Owner

Awesome, thank you so much!

@troglobit troglobit merged commit 021b66f into troglobit:master Aug 9, 2023
4 checks passed
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