From: Kevin Benton <blak111@gmail.com>
Date: Tue, 14 Oct 2014 06:40:36 +0000 (-0700)
Subject: Minor: remove unnecessary intermediate variable
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=50d06130e1ff2a2cfb62510b52efe1537c1068b6;p=openstack-build%2Fneutron-build.git

Minor: remove unnecessary intermediate variable

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
---

diff --git a/neutron/plugins/ml2/plugin.py b/neutron/plugins/ml2/plugin.py
index d744f19fe..73eae1832 100644
--- a/neutron/plugins/ml2/plugin.py
+++ b/neutron/plugins/ml2/plugin.py
@@ -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: