On Mon, Apr 05, 2010 at 12:52:51PM +0200, Matthias Bolte wrote:
2010/4/5 Stefan Berger <stefanb(a)us.ibm.com>:
>
> Matthias,
>
> I have a patch that does the same. Most of the non-search-replace work I
> needed to do was in nwfilter_ebiptables_driver.c where several functions
> have the virConnectPtr as parameter passing it all the way through to the
> function that used to pass it to the error reporting function. If you take
> out modification to that file from your patch, I could commit mine and it
> would save me a lot of work again.
>
> The summary of changes to this file in my patch look like this:
>
> src/nwfilter/nwfilter_ebiptables_driver.c | 762
> ++++++++++++-----------------
>
> Regards,
> Stefan
>
>
>>
>> libvir-list-bounces(a)redhat.com
>>
>> ---
>> src/conf/nwfilter_conf.c | 120 ++++++-------
>> src/conf/nwfilter_conf.h | 16 +-
>> src/conf/nwfilter_params.c | 2 +-
>> src/nwfilter/nwfilter_driver.c | 25 ++--
>> src/nwfilter/nwfilter_ebiptables_driver.c | 285 ++++++++++
>> +------------------
>> src/nwfilter/nwfilter_gentech_driver.c | 39 ++--
>> src/nwfilter/nwfilter_gentech_driver.h | 3 +-
>> tests/nwfilterxml2xmltest.c | 2 +-
>> 8 files changed, 202 insertions(+), 290 deletions(-)
>>
>
Sure, no problem. Go ahead with your patch, I'll rebase/change mine as
necessary.
Agreed, please push it.
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit
http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine
http://rpmfind.net/
http://veillard.com/ | virtualization library
http://libvirt.org/