[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] Drop iptablesContext



On 06/28/2013 12:52 AM, Roman Bogorodskiy wrote:
> iptablesContext holds only 4 pairs of iptables
> (table, chain) and there's no need to pass
> it around.
>
> This is a first step towards separating bridge_driver.c
> in platform-specific parts.
> ---
>  src/libvirt_private.syms    |   2 -
>  src/network/bridge_driver.c | 253 +++++++++++++++++--------------------------
>  src/util/viriptables.c      | 257 +++++++++++---------------------------------
>  src/util/viriptables.h      |  65 ++++-------
>  4 files changed, 183 insertions(+), 394 deletions(-)

Mostly mechanical. Visual/build ACK. I'll run some tests tomorrow, and
push it after 1.1.0 is released.


(It's just amazing how much change was required just to remove that one
thing.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]