|Summary:||Could review tool include style failure info?|
|Product:||WebKit||Reporter:||Oliver Hunt <oliver>|
|Component:||Tools / Tests||Assignee:||Nobody <webkit-unassigned>|
|Severity:||Normal||CC:||abarth, eric, ojan|
|Version:||528+ (Nightly build)|
|OS:||OS X 10.5|
Description Oliver Hunt 2010-11-04 20:38:49 PDT
I was just reviewing a patch that made the style bot go red for bogus reasons, it was annoying to have to go away and look at a separate page to identify whether the errors were indeed correct. It would be awesome if the style failures could be displayed inline in the review tool
Comment 1 Eric Seidel (no email) 2010-11-04 20:40:12 PDT
Woh, crazy talk. Should be quite easy to do, I think.
Comment 2 Adam Barth 2010-11-05 10:20:28 PDT
Yeah, I was planning to do this originally, but never got around to it. We just need to write a parser for the style queue output, which shouldn't be that hard.