Re: h8300-elf build broken

2012-05-10 Thread DJ Delorie
style nits: It should be size_t(__len - __pos), and not (size_t)(__len - __pos). Same for the other hunk. Patch OK with those changes. Committed that way. Thanks! Ok for 4.7 branch as well? * include/bits/random.tcc (seed_seq::generate): Cast max() operands to size_t to

Re: h8300-elf build broken

2012-05-10 Thread Gabriel Dos Reis
On Thu, May 10, 2012 at 3:33 PM, DJ Delorie d...@redhat.com wrote: style nits: It should be size_t(__len - __pos), and not (size_t)(__len - __pos).  Same for the other hunk.  Patch OK with those changes. Committed that way.  Thanks!  Ok for 4.7 branch as well? yes, it is. Thanks, -- Gaby

Re: h8300-elf build broken

2012-05-10 Thread DJ Delorie
Committed that way.  Thanks!  Ok for 4.7 branch as well? yes, it is. Thanks, Done!

Re: h8300-elf build broken

2012-05-09 Thread Richard Guenther
On Wed, May 9, 2012 at 1:24 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Tue, May 8, 2012 at 5:14 PM, DJ Delorie d...@redhat.com wrote: I assume this is a size_t vs int type problem, but the diagnostic points to the function declaration, not to an actual binary expression, and

Re: h8300-elf build broken

2012-05-09 Thread Marc Glisse
On Wed, 9 May 2012, Richard Guenther wrote: On Wed, May 9, 2012 at 1:24 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Tue, May 8, 2012 at 5:14 PM, DJ Delorie d...@redhat.com wrote: I assume this is a size_t vs int type problem, but the diagnostic points to the function

Re: h8300-elf build broken

2012-05-09 Thread Gabriel Dos Reis
On Wed, May 9, 2012 at 3:27 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, May 9, 2012 at 1:24 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Tue, May 8, 2012 at 5:14 PM, DJ Delorie d...@redhat.com wrote: I assume this is a size_t vs int type problem, but the

Re: h8300-elf build broken

2012-05-09 Thread Gabriel Dos Reis
On Wed, May 9, 2012 at 3:41 AM, Marc Glisse marc.gli...@inria.fr wrote: On Wed, 9 May 2012, Richard Guenther wrote: On Wed, May 9, 2012 at 1:24 AM, Gabriel Dos Reis g...@integrable-solutions.net wrote: On Tue, May 8, 2012 at 5:14 PM, DJ Delorie d...@redhat.com wrote: I assume this is a

Re: h8300-elf build broken

2012-05-09 Thread Marc Glisse
On Wed, 9 May 2012, Gabriel Dos Reis wrote: On Wed, May 9, 2012 at 3:41 AM, Marc Glisse marc.gli...@inria.fr wrote: necessary because of platforms where size_t is unsigned short (I didn't know those existed...) Well, I suspect AVR might be such platform but I do not seem to have an ABI

h8300-elf build broken

2012-05-08 Thread DJ Delorie
I assume this is a size_t vs int type problem, but the diagnostic points to the function declaration, not to an actual binary expression, and I can't figure out what it's complaining about: /greed/dj/m32c/gcc/h8300-elf/./gcc/xgcc -shared-libgcc -B/greed/dj/m32c/gcc/h8300-elf/./gcc -nostdinc++

Re: h8300-elf build broken

2012-05-08 Thread Gabriel Dos Reis
On Tue, May 8, 2012 at 5:14 PM, DJ Delorie d...@redhat.com wrote: I assume this is a size_t vs int type problem, but the diagnostic points to the function declaration, not to an actual binary expression, and I can't figure out what it's complaining about: My mailer uses proportional fonts so