[llvm-commits] [PATCH] Fix lit.py for gtest "typed tests"
jyasskin at google.com
Sat Oct 17 18:37:14 PDT 2009
On Sat, Oct 17, 2009 at 4:18 PM, Daniel Dunbar <daniel at zuster.org> wrote:
> On Sat, Oct 17, 2009 at 11:31 AM, Jeffrey Yasskin <jyasskin at google.com> wrote:
>> GTest has a kind of test whose names look like
>> This broke lit.py, which os.path.join()ed the test name onto the
>> test's path and assumed that it could recover the name as the second
>> result of os.path.split. This patch shifts path components from the
>> testPath to the testName until the testPath exists.
> LGTM, although I can't tell if its possible to have multiple slashes
> in the test name from my cursory reading of the doc. If it isn't,
> maybe better to just have an if instead of a while?
I don't think it's possible to have multiple slashes. I was just
future-proofing. I'll change it to an if. Thanks!
More information about the llvm-commits