[webkit-dev] Changes to line-by-line code reviews

Oliver Hunt oliver at apple.com
Sun Aug 29 12:01:43 PDT 2010


It would be nice if you could select a block of code and have the comment be for that block -- last i looked the line by line review basically loses context for reviews as it pushes the comments to the bottom and only includes a single line of context.

--Oliver

On Aug 29, 2010, at 11:13 AM, Adam Barth wrote:

> Based on some feedback, I'm going to try to improve the line-by-line
> review tool.  I've landed the first iteration of the new design, which
> should be usable and have roughly the same functionality as the old
> design.  I'll be adding new features shortly.
> 
> The main difference is you now access the line-by-line review feature
> using the "Formatted Diff" link in bugs.webkit.org instead of the
> "Review Patch" link.  I made this change so that folks who like the
> old "Review Patch" tool won't be bothered by the new tool.  If you
> have feature requests, let me know.  I'll post an update once the tool
> is awesomified.
> 
> Thanks,
> Adam
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev



More information about the webkit-dev mailing list