This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [bootstrap-O1] change value type to avoid sprintf buffer size warning

From: Alexandre Oliva <aoliva at redhat dot com>
To: Andreas Schwab <schwab at linux-m68k dot org>
Cc: Martin Sebor <msebor at gmail dot com>, Jeff Law <law at redhat dot com>, gcc-patches at gcc dot gnu dot org
Date: Thu, 05 Jan 2017 19:00:07 -0200
Subject: Re: [bootstrap-O1] change value type to avoid sprintf buffer size warning
Authentication-results: sourceware.org; auth=none
References: <orwpec3e5y.fsf@lxoliva.fsfla.org> <0f06b93a-da0c-fbbd-95e3-e7b08e9ca713@redhat.com> <54c58729-fc8e-a3f9-a5c5-ecf678006143@gmail.com> <ortw9exiao.fsf_-_@lxoliva.fsfla.org> <m260ltkj09.fsf@linux-m68k.org>

On Jan  5, 2017, Andreas Schwab <schwab@linux-m68k.org> wrote:

> On Jan 05 2017, Alexandre Oliva <aoliva@redhat.com> wrote:
>> -      sprintf (xname, "<U%4x>", ((unsigned)((uintptr_t)(t) & 0xffff)));
>> +      sprintf (xname, "<U%4hx>", ((unsigned short)((uintptr_t)(t) & 0xffff)));

> Please fix the spacing while you are at it.

Err...  I sure would, if I knew what fix you had in mind.  Care to share
your thoughts?  Thanks,

-- 
Alexandre Oliva, freedom fighter    http://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/   FSF Latin America board member
Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer

Follow-Ups:
- Re: [bootstrap-O1] change value type to avoid sprintf buffer size warning
  - From: Andreas Schwab

References:
- [bootstrap-O1] enlarge sprintf output buffer to avoid warning
  - From: Alexandre Oliva
- Re: [bootstrap-O1] enlarge sprintf output buffer to avoid warning
  - From: Jeff Law
- Re: [bootstrap-O1] enlarge sprintf output buffer to avoid warning
  - From: Martin Sebor
- [bootstrap-O1] change value type to avoid sprintf buffer size warning (was: Re: [bootstrap-O1] enlarge sprintf output buffer to avoid warning)
  - From: Alexandre Oliva
- Re: [bootstrap-O1] change value type to avoid sprintf buffer size warning
  - From: Andreas Schwab

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]