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
What version are you running?
What's the URL of the page containing the problem?
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
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