[cfe-commits] r145439 - /cfe/trunk/lib/Lex/ModuleMap.cpp
Chris Lattner
clattner at apple.com
Tue Nov 29 14:57:24 PST 2011
On Nov 29, 2011, at 2:27 PM, Douglas Gregor wrote:
> URL: http://llvm.org/viewvc/llvm-project?rev=145439&view=rev
> Log:
> Escape strings when printing module maps, for silly operating systems
> that use \ as a separator.
>
> +static void printEscapedString(llvm::raw_ostream &OS, StringRef String) {
This looks familiar, should it become a method on StringRef or something?
-Chris
> + for (StringRef::iterator I = String.begin(), E = String.end(); I != E; ++I) {
> + unsigned char Char = *I;
> +
> + switch (Char) {
> + default:
> + if (isprint(Char))
> + OS << (char)Char;
> + else // Output anything hard as an octal escape.
> + OS << '\\'
> + << (char)('0'+ ((Char >> 6) & 7))
> + << (char)('0'+ ((Char >> 3) & 7))
> + << (char)('0'+ ((Char >> 0) & 7));
> + break;
> + // Handle some common non-printable cases to make dumps prettier.
> + case '\\': OS << "\\\\"; break;
> + case '"': OS << "\\\""; break;
> + case '\n': OS << "\\n"; break;
> + case '\t': OS << "\\t"; break;
> + case '\a': OS << "\\a"; break;
> + case '\b': OS << "\\b"; break;
> + }
> + }
> +}
> +
> void ModuleMap::Module::print(llvm::raw_ostream &OS, unsigned Indent) const {
> indent(OS, Indent);
> if (IsFramework)
> @@ -82,12 +107,16 @@
>
> if (UmbrellaHeader) {
> indent(OS, Indent + 2);
> - OS << "umbrella \"" << UmbrellaHeader->getName() << "\"\n";
> + OS << "umbrella \"";
> + printEscapedString(OS, UmbrellaHeader->getName());
> + OS << "\"\n";
> }
>
> for (unsigned I = 0, N = Headers.size(); I != N; ++I) {
> indent(OS, Indent + 2);
> - OS << "header \"" << Headers[I]->getName() << "\"\n";
> + OS << "header \"";
> + printEscapedString(OS, Headers[I]->getName());
> + OS << "\"\n";
> }
>
> for (llvm::StringMap<Module *>::const_iterator MI = SubModules.begin(),
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
More information about the cfe-commits
mailing list