From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 2212 invoked by alias); 2 Aug 2014 17:21:25 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 2199 invoked by uid 89); 2 Aug 2014 17:21:24 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.4 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 X-HELO: smtp.gentoo.org Received: from smtp.gentoo.org (HELO smtp.gentoo.org) (140.211.166.183) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Sat, 02 Aug 2014 17:21:23 +0000 Received: from vapier.localnet (localhost [127.0.0.1]) by smtp.gentoo.org (Postfix) with ESMTP id 8D9B233FF0A; Sat, 2 Aug 2014 17:21:21 +0000 (UTC) From: Mike Frysinger To: Chen Gang , gdb-patches@sourceware.org Cc: eager@eagercon.com Subject: Re: [PATCH] sim/microblaze/interp.c: Use long int format instead of int to avoid compiling warnings Date: Sat, 02 Aug 2014 17:21:00 -0000 Message-ID: <1444359.20hJbUDvYc@vapier> User-Agent: KMail/4.13.1 (Linux/3.14.2; KDE/4.13.1; x86_64; ; ) In-Reply-To: <53B1FD76.7030500@gmail.com> References: <53B1FD76.7030500@gmail.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart1581296.7BIXPfXnNS"; micalg="pgp-sha1"; protocol="application/pgp-signature" X-IsSubscribed: yes X-SW-Source: 2014-08/txt/msg00030.txt.bz2 --nextPart1581296.7BIXPfXnNS Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="us-ascii" Content-length: 543 On Tue 01 Jul 2014 08:14:46 Chen Gang wrote: sim patches only go to gdb-patches, so don't include cgen/binutils in the=20 future please > --- a/sim/microblaze/interp.c > +++ b/sim/microblaze/interp.c >=20 > if (len > (int) sizeof (unsigned long)) > printf ("That operation is not available on integers of more than " > - "%d bytes.", sizeof (unsigned long)); > + "%ld bytes.", sizeof (unsigned long)); unfortunately, i don't believe this is correct. pretty sure the right answ= er=20 is %zu as sizeof() is a size_t. -mike= --nextPart1581296.7BIXPfXnNS Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit Content-length: 819 -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABAgAGBQJT3R4UAAoJEEFjO5/oN/WBqnsP/1XJ8miO+gt7f1He5t+TH/cD t+KBuDIDkENbzt+CYe+2YABy0admTFXv4sH9XXKB4pyw36CKhfPIugp4JMTPtZzr YtaHnCnNZdkQOysqUtg4rI5TBzdrRUP2uTowk9xuNcWkjxgQoELppz3mQQHTY5a8 62oiNFnjcWZNAfCP22S5wcTk8aqgRmaFxtuRmMo+5bOra/66ByUAa5CKE80BrESf ypXIhobuO9VexahJWAbP5Etx3mY3/8rolD3IjUo02+D0xRHZqxb5Icjwm7MADfxC entOxPBD05drPv+TcVkCHt4h5p3Fvsgx1SikLUZQLlpxVKtxjRgcPlPs59bUxLVz JNULrfE39hDpXgAkzwx6ids5w1h+18b09nG1yj/LRUG/1g5GTh/SKFjo94suo/s2 cUGZQdEfizkfkjIs8vgeigH8T3rNYHdianbalSDafqb7NupORidwsSV+BEBt3wXy jCPu8Q6KWZucbg/W27VEUUJTlcQBPpuWguxmiQawRpbPEM7WYFRSCK8k7DLhQsPU k3QZfs9XbMzp5qvlQov9AoJYO455Jh2dPWjo9rQWiVoPl+fOUFs/WaMajFl/3h6L xmaZHQPSs4/0MH6vxbXR0M43CJteFbZMXprE3ZEerHT6VPva9SLMrTYXgaM7qNij oA0LPPcQTmzQ+um2QFmy =y47X -----END PGP SIGNATURE----- --nextPart1581296.7BIXPfXnNS--