From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Engelhardt Subject: Re: [PATCH 6/6] netfilter: xtables: add const qualifiers Date: Thu, 11 Feb 2010 17:22:06 +0100 (CET) Message-ID: References: <1265823547-10358-1-git-send-email-jengelh@medozas.de> <1265823547-10358-7-git-send-email-jengelh@medozas.de> <4B73C740.5020003@trash.net> <4B742B21.9040300@trash.net> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Cc: netfilter-devel@vger.kernel.org To: Patrick McHardy Return-path: Received: from borg.medozas.de ([188.40.89.202]:37902 "EHLO borg.medozas.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756175Ab0BKQWI (ORCPT ); Thu, 11 Feb 2010 11:22:08 -0500 In-Reply-To: <4B742B21.9040300@trash.net> Sender: netfilter-devel-owner@vger.kernel.org List-ID: On Thursday 2010-02-11 17:06, Patrick McHardy wrote: >Jan Engelhardt wrote: >> On Thursday 2010-02-11 10:00, Patrick McHardy wrote: >> >>> Jan Engelhardt wrote: >>>> +static inline const struct ip6t_entry_target * >>>> +ip6t_get_target_c(const struct ip6t_entry *e) >>>> +{ >>>> + return ip6t_get_target((struct ip6t_entry *)e); >>>> +} >>>> + >>> 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. FWIW, the function is removed later on anyway ("remove remaining xt1 code").