[Openstack-poc] [Bug 927650] Re: cfg: unneeded multiple inheritance
** Changed in: openstack-common Milestone: None = 2012.1 ** Changed in: openstack-common Importance: Undecided = Low ** Changed in: openstack-common Status: Fix Committed = Fix Released -- You received this bug notification because you are a member of OpenStack Common Drivers, which is the registrant for openstack-common. https://bugs.launchpad.net/bugs/927650 Title: cfg: unneeded multiple inheritance Status in openstack-common: Fix Released Bug description: From Monsyne Dragon in https://review.openstack.org/3729 File nova/openstack/common/cfg.py Line 696: class ConfigOpts(collections.Mapping, object): I don't think we really need to multiply inherit from 'object' here. esp. since collections.Mapping already inherits from object. While this doesn't affect anything atm, this kind of diamond inheritance may lead to some rather non-intuitive behavior if someone subclasses ConfigOpts in the future To manage notifications about this bug go to: https://bugs.launchpad.net/openstack-common/+bug/927650/+subscriptions ___ Mailing list: https://launchpad.net/~openstack-poc Post to : openstack-poc@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack-poc More help : https://help.launchpad.net/ListHelp
[Openstack-poc] [Bug 927650] Re: cfg: unneeded multiple inheritance
** Tags added: cfg -- You received this bug notification because you are a member of OpenStack Common Drivers, which is the registrant for openstack-common. https://bugs.launchpad.net/bugs/927650 Title: cfg: unneeded multiple inheritance Status in openstack-common: Fix Committed Bug description: From Monsyne Dragon in https://review.openstack.org/3729 File nova/openstack/common/cfg.py Line 696: class ConfigOpts(collections.Mapping, object): I don't think we really need to multiply inherit from 'object' here. esp. since collections.Mapping already inherits from object. While this doesn't affect anything atm, this kind of diamond inheritance may lead to some rather non-intuitive behavior if someone subclasses ConfigOpts in the future To manage notifications about this bug go to: https://bugs.launchpad.net/openstack-common/+bug/927650/+subscriptions ___ Mailing list: https://launchpad.net/~openstack-poc Post to : openstack-poc@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack-poc More help : https://help.launchpad.net/ListHelp
[Openstack-poc] [Bug 927650] Re: cfg: unneeded multiple inheritance
In python=2.6, collections.Mapping inherits from collections.Sized, collections.Iterable, and collections.Container which are also subclasses of object. ** Changed in: openstack-common Status: New = Confirmed -- You received this bug notification because you are a member of OpenStack Common Drivers, which is the registrant for openstack-common. https://bugs.launchpad.net/bugs/927650 Title: cfg: unneeded multiple inheritance Status in openstack-common: In Progress Bug description: From Monsyne Dragon in https://review.openstack.org/3729 File nova/openstack/common/cfg.py Line 696: class ConfigOpts(collections.Mapping, object): I don't think we really need to multiply inherit from 'object' here. esp. since collections.Mapping already inherits from object. While this doesn't affect anything atm, this kind of diamond inheritance may lead to some rather non-intuitive behavior if someone subclasses ConfigOpts in the future To manage notifications about this bug go to: https://bugs.launchpad.net/openstack-common/+bug/927650/+subscriptions ___ Mailing list: https://launchpad.net/~openstack-poc Post to : openstack-poc@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack-poc More help : https://help.launchpad.net/ListHelp
[Openstack-poc] [Bug 927650] Re: cfg: unneeded multiple inheritance
Reviewed: https://review.openstack.org/4059 Committed: http://github.com/openstack/openstack-common/commit/8f1de0c493a68eec5801037590d769e711f19127 Submitter: Jenkins Branch:master commit 8f1de0c493a68eec5801037590d769e711f19127 Author: Zhongyue Luo lzye...@gmail.com Date: Sun Feb 12 16:04:21 2012 +0800 cfg: unneeded multiple inheritance Fixed bug #927650 In python=2.6, collections.Mapping inherits from collections.Sized, collections.Iterable, and collections.Container which are also subclasses of object. Change-Id: I6238c683324127abd9fb637748a10b6bdb2961e0 ** Changed in: openstack-common Status: In Progress = Fix Committed -- You received this bug notification because you are a member of OpenStack Common Drivers, which is the registrant for openstack-common. https://bugs.launchpad.net/bugs/927650 Title: cfg: unneeded multiple inheritance Status in openstack-common: Fix Committed Bug description: From Monsyne Dragon in https://review.openstack.org/3729 File nova/openstack/common/cfg.py Line 696: class ConfigOpts(collections.Mapping, object): I don't think we really need to multiply inherit from 'object' here. esp. since collections.Mapping already inherits from object. While this doesn't affect anything atm, this kind of diamond inheritance may lead to some rather non-intuitive behavior if someone subclasses ConfigOpts in the future To manage notifications about this bug go to: https://bugs.launchpad.net/openstack-common/+bug/927650/+subscriptions ___ Mailing list: https://launchpad.net/~openstack-poc Post to : openstack-poc@lists.launchpad.net Unsubscribe : https://launchpad.net/~openstack-poc More help : https://help.launchpad.net/ListHelp