[Openstack] Proposal to limit decorator usage

2012-01-17 Thread Lorin Hochstein
While going through merge proposal, I ran across this one from Mark Washenberger about limiting the number of decorators to 2 in nova code: https://review.openstack.org/2966 There's some good discussion on this in the proposal comments, but I thought it should hit the mailing list as well, in

Re: [Openstack] Proposal to limit decorator usage

2012-01-17 Thread Kevin L. Mitchell
On Tue, 2012-01-17 at 11:09 -0500, Lorin Hochstein wrote: Decorators -- A function or method should not have more than two decorators applied to it where it is defined. I'll point out that current discussion on that merge thread is favoring a different idea altogether: restrict

Re: [Openstack] Proposal to limit decorator usage

2012-01-17 Thread Sandy Walsh
+sandy.walsh=rackspace@lists.launchpad.net] on behalf of Lorin Hochstein [lo...@nimbisservices.com] Sent: Tuesday, January 17, 2012 12:09 PM To: openstack@lists.launchpad.net Subject: [Openstack] Proposal to limit decorator usage While going through merge proposal, I ran across this one from Mark