diff options
author | Christian Hammond <chipx86@chipx86.com> | 2009-11-03 01:33:50 -0800 |
---|---|---|
committer | Christian Hammond <chipx86@chipx86.com> | 2009-11-03 01:33:50 -0800 |
commit | 3bd73dda33fb0669627b642cec6bf1d26b966263 (patch) | |
tree | f5a6ec89db640248e055d4a404f370186ce0d8a5 /AUTHORS | |
parent | 5985a3192e52832a5bfb05d7071cb00d93e17c63 (diff) |
Fix post-review error reporting when using --diff-only.
Running post-review --diff-only may throw exceptions when encountering
errors, instead of gracefully handlin them. A simple case of this is to
run with --diff-only without having a post-review login cookie, and instead
of a login prompt, post-review will crash with a KeyError.
This change fixes this reporting by not special-casing the --diff-only
parameter in the error reporting. We were only handling errors if that
option was not set.
Patch by Eric Huss.
Reviewed at http://reviews.reviewboard.org/r/1185/
Diffstat (limited to 'AUTHORS')
0 files changed, 0 insertions, 0 deletions