On Mar 22, 2013, at 12:09 PM, Brian Miller wrote: > Remove an extra ')'. > --- > pox/openflow/libopenflow_01.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/pox/openflow/libopenflow_01.py b/pox/openflow/libopenflow_01.py > index 4905465..f81eb0c 100644 > --- a/pox/openflow/libopenflow_01.py > +++ b/pox/openflow/libopenflow_01.py > @@ -668,7 +668,7 @@ class ofp_phy_port (ofp_base): > def _validate (self): > if isinstance(self.hw_addr, bytes) and len(self.hw_addr) == 6: > pass > - elif not isinstance(self.hw_addr, EthAddr)): > + elif not isinstance(self.hw_addr, EthAddr): > return "hw_addr is not a valid format" > if len(self.name) > OFP_MAX_PORT_NAME_LEN: > return "name is too long"
Thanks. This had actually been fixed in betta at this point; it just hadn't been merged yet since I haven't been working on carp again. I just did a merge to catch carp back up. (carp should really be consistently ahead of betta, I'm just a bit slow in making that actually true.) -- Murphy
