[chora] yet another patch

Jon Parise jon@csh.rit.edu
Thu, 1 Mar 2001 12:15:44 -0500


On Thu, Mar 01, 2001 at 11:40:34AM +0100, Mathieu Arnold wrote:

> this morning, i needed a human readable view with more than 3 lines
> of context, so i added a few things, and a long link with 10 lines
> of context :

[snip]
 
> +     * @param nl Number of lines to be used in context and unified diff

I'd maybe change 'nl' to 'num' or something else.  I normally
associate 'nl' with 'newline' when I'm working with PHP code.

You patch looks good otherwise.

-- 
Jon Parise (jon@csh.rit.edu)  .  Rochester Inst. of Technology
http://www.csh.rit.edu/~jon/  :  Computer Science House Member