Re: [v2 4/7] msvcirt: Implement ostream::operator<< for floats

[Piotr Caban]

On 06/22/16 14:23, Iván Matellanes wrote:
> On 22/06/16 12:25, Piotr Caban wrote:
> > On 06/22/16 11:53, Iván Matellanes wrote:
> > > +        str_length += _scprintf(sprintf_fmt, prec, d); /* number
> > > representation */
> > > +        if (str_length > 24) {
> > > +            /* too long to output */
> > > +            ostream_writepad(ostr, "", "");
> > > Could you please add a test for it? Also a comment saying that this is
> > > expected behavior would be useful.
> > > Thanks for the feedback.
> > > This is the expected behavior on Vista+. XP/2k3 crashes though, so I
> > > ended up removing the related tests. What's the proper way to handle this?
> > > I think that you can just change the comment to make clear this is not 
> > > incomplete implementation.

Thanks,
Piotr