Code review comment for lp:~mwhudson/launchpad/vostok-traverse-distro

Revision history for this message
Tim Penhey (thumper) wrote :

On Fri, 30 Jul 2010 16:04:43 you wrote:
> Review: Needs Fixing
>
> This change is faulty:
>
> - 'Unrecognized branch type: %r' % (branch_type,))
> + 'Unrecognized branch type: %r' % branch_type)
>
> if branch_type is a tuple with length != 1, it will blow up in your
> version, and for a error code path thats undesirable. I haven't done a
> full review - was glancing in email and this jumped out at me.

Yeah, but branch_type is always a BranchType enum value, which is why I let
this slide.

« Back to merge proposal