Messages in this thread | | | Date | Fri, 24 Mar 2006 03:02:12 +0100 | From | Patrick McHardy <> | Subject | Re: Two comments on the H.323 conntrack/NAT helper |
| |
[The hotmail address of the author doesn't work, CCed sourceforge-address]
Adrian Bunk wrote: > Two comments on the H.323 conntrack/NAT helper: > - the function prototypes in ip_nat_helper_h323.c are _ugly_, > please move them to a header file
Their ugliness is because of the current API, which cleaned up quite a lot of the surrounding code, but requires this ugliness from each helper. I would like to keep them visible as a reminder that a cleaner solution is wanted, but moving them to header files certainly sound like a good idea to eliminate the risk of prototype conflicts. But please do this for all helpers at once.
> - is there a reason for not using EXPORT_SYMBOL_GPL?
I would prefer that too. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |