From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33947) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gMvxZ-00085d-52 for qemu-devel@nongnu.org; Wed, 14 Nov 2018 09:16:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gMvxW-0004JU-Pt for qemu-devel@nongnu.org; Wed, 14 Nov 2018 09:16:00 -0500 Received: from mx1.redhat.com ([209.132.183.28]:38390) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gMvxU-0004Gb-Rg for qemu-devel@nongnu.org; Wed, 14 Nov 2018 09:15:58 -0500 Date: Wed, 14 Nov 2018 14:15:15 +0000 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Message-ID: <20181114141515.GI19298@redhat.com> Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= References: <20181114123643.24091-1-marcandre.lureau@redhat.com> <20181114123643.24091-38-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20181114123643.24091-38-marcandre.lureau@redhat.com> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH for-3.2 37/41] slirp: replace ARRAY_SIZE with G_N_ELEMENTS List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?utf-8?Q?Marc-Andr=C3=A9?= Lureau Cc: qemu-devel@nongnu.org, samuel.thibault@ens-lyon.org, rjones@redhat.com, stefanha@redhat.com, renzo@cs.unibo.it On Wed, Nov 14, 2018 at 04:36:39PM +0400, Marc-Andr=C3=A9 Lureau wrote: > Do not require QEMU macro. As with the endian thing, we should really purge ARRAY_SIZE from the entire QEMU codebase and just defer to glib. ARRAY_SIZE made sense before glib was a hard dep in QEMU, but there's no reason beyond bike shed colour choice to preserve it now. I'm fine with this patch being targetted to slirp to avoid getting dragged into a broader discussion, but I'd encourage the global cleanup. I've personally used G_N_ELEMENTS in all QEMU code I've been writing since its more familiar from glib than a QEMU specific macro name. Reviewed-by: Daniel P. Berrang=C3=A9 Regards, Daniel --=20 |: https://berrange.com -o- https://www.flickr.com/photos/dberran= ge :| |: https://libvirt.org -o- https://fstop138.berrange.c= om :| |: https://entangle-photo.org -o- https://www.instagram.com/dberran= ge :|