<div>On Thu, Jul 2, 2009 at 4:37 PM, Maciej Stachowiak <span dir="ltr">&lt;<a href="mailto:mjs@apple.com">mjs@apple.com</a>&gt;</span> wrote:</div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div style="word-wrap:break-word"><div class="im"><br><div><div>On Jul 2, 2009, at 4:33 PM, Julie Parent wrote:</div><br><blockquote type="cite"><div class="gmail_quote"><div><div><span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse">When viewing patches on <a href="http://bugs.webkit.org/" style="color:rgb(0, 101, 204)" target="_blank">bugs.webkit.org</a>, 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.<div>
 <br></div><div>Proposal: I&#39;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.</div>
 </span></div></div></div></blockquote></div><br></div><div>I don&#39;t think it&#39;s just because you&#39;re (relatively) new. I often wish for more context myself.</div></div></blockquote><div><br></div><div>I would definitely find this useful as well.</div>
</div></div>