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: [v3] Enable four 27_io tests


Ulrich Weigand wrote:

>   long l = -1;
> [...]
>   o << oct << l << ' ' << hex << l;
>   VERIFY( strbuf.str() == "37777777777 ffffffff" );
>   strbuf.str(str_blank);
>
> Shouldn't the test take the size of the long type into account here?

You are correct.

Indeed, the whole test03 of ostream_inserter_arith.cc blindly assumes way too
much (16 bit shorts, 32 bit ints and longs).

I'm not sure however if at this point is really worth properly fixing the test,
making it platform independent, or we can simply disable it again...

Benjamin?

Thanks for your feeback,
Paolo.



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