<div dir="ltr">Hi!<br>I've sent a patch what replaces std::ostream with llvm::raw_ostream.  What's the status of its review ? I know it is not an important patch but i would like to finish porting because i'd like to finish existing distcc clang code to be ready for merge sooner then later.<br>
<br>Cheers,<br>Csaba<br><br><div class="gmail_quote">2008/8/23 Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Aug 23, 2008, at 11:35 AM, Chris Lattner wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

I've found these problems:<br>
- raw_stream is currently not supports pointer printing. << (void*)X<br>
</blockquote>
<br>
This should be easy to add, just add a method to raw_stream?<br>
</blockquote>
<br></div>
I didn't add this, can you add it?<div class="Ih2E3d"><br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

- not supports field width like: << std::setw(3) << ... (this is in<br>
file: CFG.cpp line 1485)<br>
- not supports hexa printing like: <<std::hex << value (this is in<br>
file: StmtPrinter.cpp line 550)<br>
</blockquote>
<br>
Adding a full set of formatting is a non-trival job, and we don't want<br>
to follow the lead of iostreams.  I'll see if I can hack something in<br>
quickly.<br>
</blockquote>
<br></div>
Okay, you can now do stuff like this:<br>
<br>
OS << "whatever" << format("%4.3f some hex thing 0x%X", mydouble, 255) << " blah";<br>
<br>
This is efficient (no temporary strings etc), and you can use any (portable) printf formatting without penalty.<br><font color="#888888">
<br>
-Chris<br>
</font></blockquote></div><br></div>