[vim/vim] Avoid infinite loops in examples. (PR #9450)

Benji Fisher

unread,

Dec 31, 2021, 9:52:21 PM12/31/21

to vim/vim, Subscribed

Another fix would be to change while counter < 40 in Line 541 to while counter > 40.

If we ever get to the decrement line, then counter is less than 40. Decrementing it makes it even smaller.

You can view, comment on, or merge this pull request online at:

https://github.com/vim/vim/pull/9450

Commit Summary

34bbbdb Avoid infinite loops in examples.

File Changes

(1 file)

M runtime/doc/usr_41.txt (2)

Patch Links:

—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you are subscribed to this thread.

Yegappan Lakshmanan

unread,

Dec 31, 2021, 11:12:30 PM12/31/21

to vim_dev, reply+ACY5DGBSMELIB5N3IN...@reply.github.com

Hi Benji,

Welcome back. Happy to see a contribution from you after a long time.

Regards,

Yegappan

vim-dev ML

unread,

Dec 31, 2021, 11:12:47 PM12/31/21

to vim/vim, vim-dev ML, Your activity

Hi Benji,

Welcome back. Happy to see a contribution from you after a long time.

Regards,
Yegappan

On Fri, Dec 31, 2021 at 6:52 PM Benji Fisher ***@***.***>
wrote:

> Another fix would be to change while counter < 40 in Line 541 to while
> counter > 40.
>
> If we ever get to the decrement line, then counter is less than 40.
> Decrementing it makes it even smaller.

> ------------------------------

> You can view, comment on, or merge this pull request online at:
>
> https://github.com/vim/vim/pull/9450
>
>

—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you are subscribed to this thread.

Bram Moolenaar

unread,

Jan 1, 2022, 6:22:46 AM1/1/22

to vim/vim, vim-dev ML, Comment

Thanks for the fix. Let me also make the conditions more sensible.

—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you commented.

Bram Moolenaar

unread,

Jan 1, 2022, 6:22:47 AM1/1/22

to vim/vim, vim-dev ML, Comment

Closed #9450.

—
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.

You are receiving this because you commented.

Reply all

Reply to author

Forward