2009/8/5 Luca Bigliardi <[email protected]>: > --- a/lib/config.py > +++ b/lib/config.py > + def _UnlockedGetNodePrimaryIP(self, node_name): > + return self._config_data.nodes[node_name].primary_ip > + > + > + def _UnlockedGetNodeSecondaryIP(self, node_name): > + return self._config_data.nodes[node_name].secondary_ip > + > +
Why not combine the two into one function returning a tuple? There should only be one empty line between functions at class level. Two empty lines at module level. Regards, Michael
