Comment #1 on issue 1345 by lonico: post-review does not process a  
*pending* Perforce changeset in v0.2 beta3
http://code.google.com/p/reviewboard/issues/detail?id=1345

I should have gone a bit further.

The last else is misplaced in the diffs below.  It should be at the same  
level as the
for.

Thank you
Laurent


-        # Get the file list
-        for line_num, line in enumerate(description):
-            if 'Affected files ...' in line:
-                break
          else:
-            # Got to the end of all the description lines and didn't find
-            # what we were looking for.
-            die("Couldn't find any affected files for this change.")
+            # Get the file list
+            for line_num, line in enumerate(description):
+                if 'Affected files ...' in line:
+                    break
+                else:
+                    # Got to the end of all the description lines and  
didn't find
+                    # what we were looking for.
+                    die("Couldn't find any affected files for this  
change.")


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to 
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard-issues?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to