Hi Bram,

On Friday, 12 May 2017 15:37:25 UTC+1, Vitor Antunes  wrote:
> On Tuesday, 14 March 2017 10:26:15 UTC, Christian Brabandt  wrote:
> > Hi Bram!
> >
> > On Do, 09 Mär 2017, Bram Moolenaar wrote:
> >
> > > This can use skipwhite().  There are several other places where this
> > > computation is done, e.g. in misc1.c, open_line():
> > >
> > >   p = ml_get_curline();
> > >   ai_col = (colnr_T)(skipwhite(p) - p);
> > >
> > > Similar one a bit further down.
> > >
> > > Would be good to put this function close to skipwhite() to make it
> > > easeri to find.
> >
> > Attached patch does that.
> >
> > >
> > > > +    /* if indent kicked in, the firstline might have changed
> > > > +     * but only do that, if the indent actually increased */
> > > > +    ind_post = get_whitespace_line_start(curwin->w_cursor.lnum);
> > > > +    if (curbuf->b_op_start.col > ind_pre && ind_post > ind_pre)
> > > > +    {
> > > > +       bd.textcol += ind_post - ind_pre;
> > > > +       bd.start_vcol += ind_post - ind_pre;
> > > > +    }
> > >
> > > Why only for when the indent increases?
> >
> > To be on the safe side. Currently blockwise editing only allows for
> > actually adding text. This expectation isn't documented yet, so I
> > documented it.
> >
> > Best,
> > Christian
>
> Hi Bram and Christian,
>
> As far as I can tell, this patch was still not merged. Is there
> anything I can help with in order to have this into Git HEAD?
>
> Thanks in advance and best regards,
> Vitor

Is there anything still missing in Christian's patches?

Thanks,
Vitor

-- 
-- 
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.

Raspunde prin e-mail lui