[llvm-commits] [llvm] r75790 - /llvm/trunk/include/llvm/System/Path.h

[Chris Lattner]

On Jul 15, 2009, at 11:48 AM, Dan Gohman wrote:

>>
>> Is this really worthwhile?  This tightly couples sys::Path and
>> raw_ostream.  Why can't the client just use toString?  The API for
>> sys::Path is already significantly broken in a lot of ways, I'd like
>> for it to not expand in scope.
>
> Yes. It makes porting from std::ostream to raw_ostream easier, and
> it doesn't add any additional burden to the task of refactoring
> sys::Path.

Ok, I guess I was mostly opposed to adding the #include of raw_ostream  
to Path.h in the short-term.  Can we at least avoid that?

-Chris