reviewboard/rbtools — comment on commit 91ca2eb5f037df0756debf0a442ed14fba9e6d1e
File:  line 0
By inorton:

This chance should actually be:

not self.has_valid_cookie() and
not (options.username and options.password)):

Otherwise --diff-filename=- fails when you supply the username and password as command line arguments