Status: New
Owner:
Labels: Type-Defect Priority-Medium
New issue 2014 by timw.at@gmail.com: get_interesting_headers raises
IndexError with C++ end-of-file whitespace change
http://code.google.com/p/reviewboard/issues/detail?id=2014
What version are you running?
1.5.4
What's the URL of the page containing the problem?
http://example.com/r/4976/diff/
What steps will reproduce the problem?
1. Create a non-trivial (more than a few dozen lines) C++ source file with
two newlines after the last function definition (revision 1)
2. Change the above file to add or remove newlines to the end of the file
3. Post a diff covering the change from step 2 to reviewboard
4. Attempt to view the diff (View Diff link)
What is the expected output? What do you see instead?
Expected: the diff viewer
Actual: IndexError in get_interesting_headers, where it accesses 'lines'
What operating system are you using? What browser?
Ubuntu. Any browser.
Please provide any additional information below.
I fixed the problem locally by wrapping 'if is_modified_file: / ... / else:
/ ...' with 'try: / ... / except IndexError: / raise StopIteration'.
(Sorry, I don't have a clean diff against 1.5.4 to post)
Based on the surrounding code and method names, the code in question is
trying to get a context function name.
If your diff is trivial (only this file), the whole view-diff page refuses
to load. If the diff is non-trivial, the asynchronous diff fetching will
only fail for files with that type of change--however, it will prevent
viewing any diff chunks for that file.
--
You received this message because you are subscribed to the Google Groups
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@googlegroups.com.
To unsubscribe from this group, send email to
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/reviewboard-issues?hl=en.
