From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v3] EAL: fix usage of printf-like functions Date: Fri, 20 Jun 2014 16:58:03 -0700 Message-ID: <20140620165803.71f68c43@nehalam.linuxnetplumber.net> References: <1403307270-28308-1-git-send-email-bruce.richardson@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Bruce Richardson Return-path: In-Reply-To: <1403307270-28308-1-git-send-email-bruce.richardson-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On Sat, 21 Jun 2014 00:34:30 +0100 Bruce Richardson wrote: > + (int)(strnlen(buf, sizeof(buf)) - sizeof("_config")), minor nit, paren not needed here (int)strnlen(buf, sizeof...) better yet change rte_snprintf() to take a size_t to make it compatiable with snprintf