]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Minor: remove unnecessary intermediate variable
authorKevin Benton <blak111@gmail.com>
Tue, 14 Oct 2014 06:40:36 +0000 (23:40 -0700)
committerKevin Benton <blak111@gmail.com>
Tue, 14 Oct 2014 06:40:36 +0000 (23:40 -0700)
Removes an unnecessary intermediary variable and an
unnecessary list extend operation that implied previous
list members where there weren't any. There should be no
functional change. This just improves readability slightly.

Change-Id: Ice412c29be083d82e055cc0bc45ed8b97c7628d0

neutron/plugins/ml2/plugin.py

index d744f19fe1f32a9b8a14b5cee7d7ff95622a37ec..73eae18324dc39290b7cf41738273ab97d7c00ad 100644 (file)
@@ -1002,8 +1002,8 @@ class Ml2Plugin(db_base_plugin_v2.NeutronDbPluginV2,
                 mech_context = driver_context.PortContext(self, context, port,
                                                           network, binding)
                 if is_dvr_enabled and utils.is_dvr_serviced(device_owner):
-                    router_info = l3plugin.dvr_deletens_if_no_port(context, id)
-                    removed_routers += router_info
+                    removed_routers = l3plugin.dvr_deletens_if_no_port(
+                        context, id)
                 self.mechanism_manager.delete_port_precommit(mech_context)
                 self._delete_port_security_group_bindings(context, id)
             if l3plugin: