Vitor Antunes wrote: > 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=C3=A4r 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 =3D ml_get_curline(); > > > > ai_col =3D (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 =3D get_whitespace_line_start(curwin->w_cursor.lnum); > > > > > + if (curbuf->b_op_start.col > ind_pre && ind_post > ind_pre) > > > > > + { > > > > > + bd.textcol +=3D ind_post - ind_pre; > > > > > + bd.start_vcol +=3D 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?
I only see a few lines of code here. Is there a patch (with a test)? -- If someone questions your market projections, simply point out that your target market is "People who are nuts" and "People who will buy any damn thing". Nobody is going to tell you there aren't enough of those people to go around. (Scott Adams - The Dilbert principle) /// Bram Moolenaar -- b...@moolenaar.net -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.