On Mon, Feb 23, 2009 at 23:24, <tilo@xxxxxxxxxxxxxxx> wrote: > Rationale: On a large display with many rows it would be nice to be > able to configure the heights of the views in the split view. For > example, I like to have the lower diff view be 90% of the total > view height. > > As it is, the patch lets the height of the lower view be configurable. > Is it may be better to configure the height of the upper view? I don't know, the way you have done it seems reasonable. > Any input on variable nameing, code style etc. of course welcome. Very, very clean and nicely done with the reusing of the parse/apply step code. Thanks. -- Jonas Fonseca -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
