On Jan 06 2017, Alexandre Oliva <aol...@redhat.com> wrote: > On Jan 5, 2017, Andreas Schwab <sch...@linux-m68k.org> wrote: > >> On Jan 05 2017, Alexandre Oliva <aol...@redhat.com> wrote: >>> On Jan 5, 2017, Andreas Schwab <sch...@linux-m68k.org> wrote: >>> >>>> On Jan 05 2017, Alexandre Oliva <aol...@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, > >> Space after cast. > > Wow, thanks, I think I never got that one right. > > Here's what I'm installing, also breaking the now-too-long line:
Alternatively you could remove the redundant parens. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different."
