On Thursday 2010-02-11 17:27, Patrick McHardy wrote: >>>>> I would prefer a macro over these get_target_c functions. >>>> Why so? >>> Because it avoids having a function for const and one for non-const. >> >> Well, I need two. One that returns const and one that does not; >> ip6t_get_target_c is signaturally-incompatible with ip6t_get_target. > >Yes, but you wouldn't if you'd use a macro. > >#define ip6t_get_target(e) ((void *)e + e->target_offset) That would remove the const from a const e, which is what I expressly wanted to avoid because it's already a little hairy. >> FWIW, the function is removed later on anyway ("remove remaining xt1 >> code"). > >This is patch 6/6, what do you mean with "later"? Patch 094/103 from the original submission http://markmail.org/browse/org.kernel.vger.netfilter-devel/2009-08 that I am trying to push to you all the time :) -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html