• 
      

    Protect against uninitialized variable or None value in two error cases in post-review 0.2RC1

    Review Request #1427 — Created Feb. 22, 2010 and submitted

    Information

    RBTools

    Reviewers

    Line 128:  'err' in self.rsp  would trigger "TypeError: argument of type 'NoneType' is not iterable"  if rsp in None
    
    Line 419: rsp is not initialized but is used at lines 427 and 432 (fall through).    I also moved the last debug message into an else: under try: as a Review is not created in the fall through case.
    Verified both error cases.  Verified that a review request is correctly updated if --diff-file option is used with a changenum but no review id.
    chipx86
    1. Thanks. Committed to master (r84366db)
    2.