[webkit-dev] Show more context in diffs

Jeremy Orlow jorlow at chromium.org
Thu Jul 2 17:48:34 PDT 2009

On Thu, Jul 2, 2009 at 4:37 PM, Maciej Stachowiak <mjs at apple.com> wrote:

> On Jul 2, 2009, at 4:33 PM, Julie Parent wrote:
> When viewing patches on bugs.webkit.org, either with the review tool or
> prettydiff, I regularly find the 3 lines of context provided to
> be insufficient to really understand the code change.  This may be because I
> am new to the code base, or because I am used to review tools like Rietveld,
> but I find that every time I want to look at a patch, I have to open the
> file side by side with the patch window to fully understand the change.
>  This is very annoying and often leads me to just give up and not bother
> fully understanding the code.
> Proposal: I'd like to change svn-create-patch to increase the number of
> lines of context provided (suggestions for how many are welcome).  One
> downside to doing this of course would be that patch sizes would increase.
> I don't think it's just because you're (relatively) new. I often wish for
> more context myself.

I would definitely find this useful as well.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20090702/4a29138a/attachment.html>

More information about the webkit-dev mailing list