[PATCH] Limit to 4 the number of cylinders shown in the data frame

Dirk Hohndel dirk at hohndel.org
Tue Jan 1 20:19:42 PST 2013


Linus Torvalds <torvalds at linux-foundation.org> writes:

> On Tue, Jan 1, 2013 at 8:59 AM, Salvador Cuñat <salvador.cunat at gmail.com> wrote:
>>
>> Here is the modified patch, and, off course, you were right. That
>> loop was a total nonsense mess.
>> Have extracted the nested functions, too, as I couldn't bear the idea of
>> being shooted by Linus  };-)
>
> The patch looks much better to me, although I haven't actually looked
> at what the resulting print-outs look like.

Yes, I like this. Can you send this as a clean single patch with a clean
commit message (that doesn't refer the previous patch that we rejected)
that is relative to the latest master?

I know, that feels like an extra step but I think that will give us a
more useful and consistent git history...

/D


More information about the subsurface mailing list