<div dir="ltr"><div class="gmail_quote">On Wed Feb 11 2015 at 11:13:22 AM Greg Clayton <<a href="mailto:gclayton@apple.com">gclayton@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> Operations:<br>
> strcpy, strncpy, etc         // Use member functions of SmallString / StringRef<br>
> sprintf, snprintf, etc       // Use llvm::raw_ostream<br>
<br>
We have StringStream, people should be using that. raw_ostream doesn't support all printf style formats. So all sprintf  and snprintf function calls should switch to using lldb_private::StreamString.<br></blockquote><div><br></div><div>What if llvm::SmallString<> had Format methods that delegated to snprintf etc?  Then everything would just work, and it would integrate nicely with the rest of my suggested points such as converting things to StringRef, SmallVectorImpl, etc.</div></div></div>