From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH iproute2] iproute: make clang happy Date: Mon, 20 Aug 2018 15:52:23 -0700 Message-ID: <20180820155223.0569b849@xeon-e3> References: <20180820214215.218127-1-mahesh@bandewar.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: netdev , Mahesh Bandewar To: Mahesh Bandewar Return-path: Received: from mail-pl0-f67.google.com ([209.85.160.67]:38017 "EHLO mail-pl0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726605AbeHUCRC (ORCPT ); Mon, 20 Aug 2018 22:17:02 -0400 Received: by mail-pl0-f67.google.com with SMTP id u11-v6so7837184plq.5 for ; Mon, 20 Aug 2018 15:59:29 -0700 (PDT) In-Reply-To: <20180820214215.218127-1-mahesh@bandewar.net> Sender: netdev-owner@vger.kernel.org List-ID: On Mon, 20 Aug 2018 14:42:15 -0700 Mahesh Bandewar wrote: > diff --git a/tc/m_ematch.c b/tc/m_ematch.c > index ace4b3dd738b..a524b520b276 100644 > --- a/tc/m_ematch.c > +++ b/tc/m_ematch.c > @@ -277,6 +277,7 @@ static int flatten_tree(struct ematch *head, struct ematch *tree) > return count; > } > > +__attribute__((format(printf, 5, 6))) > int em_parse_error(int err, struct bstr *args, struct bstr *carg, > struct ematch_util *e, char *fmt, ...) I think the printf attribute needs to go on the function prototype here: tc/m_ematch.h:extern int em_parse_error(int err, struct bstr *args, struct bstr *carg, PS: I need to take the extern of those function prototypes.