Fix UnboundLocalError when posting Perforce paths.

Review Request #5834 — Created May 16, 2014 and submitted

anselina
RBTools
master
d132fc4...
rbtools

Posting Perforce paths using rbt post would sometimes result in an
UnboundLocalError because the local variable new_path was referenced before
assignment. This is a trivial fix, changing the argument from new_path to
old_path.

  • Ran unit tests.
  • Ran rbt post //p4-test2/foo@2,@3 (which previously resulted in the error),
    and saw that the review request was created successfully.
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        rbtools/clients/perforce.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        rbtools/clients/perforce.py
      Ignored Files:
    
    
  2. 
      
david
  1. Ship It!

  2. 
      
anselina
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.6.x (91666ee)
Loading...