[Lldb-commits] [PATCH] D28858: Replace getcwd with the llvm equivalent

2017-01-23 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL292795: Replace getcwd with the llvm equivalent (authored by labath). Changed prior to commit: https://reviews.llvm.org/D28858?vs=84828&id=85399#toc Repository: rL LLVM https://reviews.llvm.org/D288

Re: [Lldb-commits] [PATCH] D28858: Replace getcwd with the llvm equivalent

2017-01-23 Thread Zachary Turner via lldb-commits
Sorry forgot about this, looks good On Mon, Jan 23, 2017 at 7:55 AM Pavel Labath via Phabricator < revi...@reviews.llvm.org> wrote: > labath added a comment. > > Any objections to this, Zach? > > > https://reviews.llvm.org/D28858 > > > > ___ lldb-commits

[Lldb-commits] [PATCH] D28858: Replace getcwd with the llvm equivalent

2017-01-23 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. Any objections to this, Zach? https://reviews.llvm.org/D28858 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D28858: Replace getcwd with the llvm equivalent

2017-01-18 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. getcwd() is not available (well.. um.. deprecated?) on windows, and the way PosixApi.h is providing it causes strange compile errors when it's included in the wrong order. The best way to avoid that is to just not use chdir. This replaces all uses of getcwd in generi