cameron314 added a comment. Thanks for all the feedback! I'll look into it in detail tomorrow.
Perhaps it would make more sense if I clarified where I'm coming from with this patch -- we use LLDB on Windows (with a custom backend) where I work, and wanted to be able to run a program with a non-ASCII path (and similarly place breakpoints in such files, etc.). So, I tried to make //minimally invasive// changes across the codebase in order to accomplish this, using the existing LLVM support functions as much as possible. This is why the changes are not entirely consistent as a whole -- I preferred consistency with the surrounding code (e.g. error handling and buffer allocation) over global consistency. Repository: rL LLVM http://reviews.llvm.org/D17107 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
