On Thu, Apr 3, 2014 at 2:48 AM, Jody Lee Bruchon <[email protected]> wrote:
> I need to review the use of last_file_modified to see if it is
> actually doing something valuable there; it has no clear comments
> and I really dislike that.

How about this: s/file_modified/modified_count/
and:

        // count_lines() is expensive.
        // Call it only if something was changed since last time
        // we were here:
        if (modified_count != last_modified_count) {
                tot = cur + count_lines(dot, end - 1) - 1;
                last_modified_count = modified_count;
        }
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to