Christian Brabandt wrote:
> On Do, 12 Jun 2014, Bram Moolenaar wrote:
>
> > Looks like somehow the data structures allocated for the screen don't
> > match with the screen size. That's weird.
>
> It's not the screen size, that seems to be correct.
> The problem is, set ls=2. This causes a recursive call to
> win_new_height() and therefore does not adjust the window height
> correctly.
>
> This patch fixes it for me:
> diff --git a/src/window.c b/src/window.c
> --- a/src/window.c
> +++ b/src/window.c
> @@ -5650,7 +5650,10 @@ win_new_height(wp, height)
> if (wp->w_height > 0)
> {
> if (wp == curwin)
> - validate_cursor(); /* w_wrow needs to be valid */
> + /* w_wrow needs to be valid, might call win_new_height recursively...
> */
> + validate_cursor();
> + if (wp->w_height != prev_height)
> + return; /* recursive call already changed window size */
> if (wp->w_wrow != wp->w_prev_fraction_row)
> set_fraction(wp);
> }
That is weird. I still can't reproduce it. This does make more sense
than the check for the NULL pointer.
--
DINGO: You must spank her well and after you have spanked her you
may deal with her as you like and then ... spank me.
AMAZING: And spank me!
STUNNER: And me.
LOVELY: And me.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
