Labels: Type-Enhancement Priority-Medium
New issue 2008 by plj...@gmail.com: Support to automatically produce a set
of findings, not necessarily in diffs between versions
*NOTE: If you have a patch, please submit it to
What version are you running?
What's the URL of the page this enhancement relates to, if any?
Describe the enhancement and the motivation for it.
Wish to post a review request based on compiler warnings rather than diffs
between versions. This is for applying review board to existing code and to
clean it up.
What operating system are you using? What browser?
Linux server for ReviewBoard, browser currently Firefox
Please provide any additional information below.
In general, we would like to be able to produce a review request and
automatically create a set of findings in the code based on some tool.
However, we have not discovered, or found, a way to do this without trying
to create diffs and automatically produce diff comments (findings) in the
So we created a script that produces a "fake" diff-file, it simply states
there is a difference on each line our automatic tool have identified an
issue in the code. The base revision is the latest revision and the
compared version is stated to be working copy (for SVN). In order not to
make this too complicated we used an equivalent of -U 0 for the diff
unifying lines, 0 lines, only the "diffing" line is stated.
When we upload this diff to the review request, either with post-review in
RBTools or in the Web Interface, the server simply stalls and no response
A later issue still unresolved is to automatically populate the diff
locations with comments, given from the automated code analyzer.
We simply wish for the feature of submit findings in the code, without
having a diff in the code. This currently seems not possible.
If there is a way to do this it would be great and please let us know.
You received this message because you are subscribed to the Google Groups
To post to this group, send email to email@example.com.
To unsubscribe from this group, send email to
For more options, visit this group at