[Lldb-commits] [PATCH] D56230: [gdb-remote] Use lldb's portable Host::GetEnvironment() instead of getenv

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jan 17 07:53:55 PST 2019


labath added a comment.

In D56230#1358356 <https://reviews.llvm.org/D56230#1358356>, @zturner wrote:

> I've always disliked this argument and hoped that someday someone would remove it entirely.  My recollection (which may be wrong) is that the only actual use of it is so that if someone types a command, and we later need to print the command back, we will print it with the same quote char.  It almost seems like we could just delete the argument and use a standardized quote char when flattening a command string.


+100

BTW, today I've tried to switch ProcessLauncherWindows to `flattenWindowsCommandLine` and this change alone was enough to fix `TestQuoting`, which has some tests XFAILed for windows due to quoting problems. I haven't sent out a patch yet because that has also broken `platform shell dir c:\` for some reason, and I haven't gotten around to investigating that.  If you (for any value of you) have some time, I'd encourage you to look into that. Otherwise, I'll get to that eventually.


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

https://reviews.llvm.org/D56230





More information about the lldb-commits mailing list