[Lldb-commits] [PATCH] D122025: [lldb] Remove lldbassert from CommandInterpreter::PrintCommandOutput

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

I think it makes sense to require that the CommandResultObject's command output is UTF-8 string with no embedded '\0'-s.  We wouldn't do smart things with rendering command output with nulls in it, and I don't see any reason to support such a thing.

OTOH, this seems like the wrong place to check that, since it's pretty far removed from whoever would have added the null's.  If we wanted to do this right, we would sanitize when you write to the CommandResultObject.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122025/new/

https://reviews.llvm.org/D122025