Re: [hackers] [vis] [PATCH] fix {, }, (, ) movements

From: Markus Teich <markus.teich_AT_stusta.mhn.de>
Date: Wed, 3 Feb 2016 18:00:05 +0100

Marc André Tanner wrote:
> On Wed, Feb 03, 2016 at 01:07:00PM +0100, Markus Teich wrote:
> > - simplify text_line_empty_{prev,next}
>
> I applied the patch, but for now left out those "simplifications". Except for
> the return value if no empty line is found these changes do not change the
> functionality, right?

Heyho Marc,

yes, at least it is the only *intended* functionality change. The simplification
is just that the state of the last scanned char is now kept in a variable
instead of in a nested if. I'm not a fan of keeping state by nesting control
structures.

--Markus
Received on Wed Feb 03 2016 - 18:00:05 CET

This archive was generated by hypermail 2.3.0 : Wed Feb 03 2016 - 18:00:14 CET