This is an archive of the discontinued LLVM Phabricator instance.

dotest.py - fixed a bug displaying usage
ClosedPublic

Authored by vharron on May 22 2015, 7:54 AM.

Details

Reviewers
zturner
clayborg

Diff Detail

Event Timeline

vharron updated this revision to Diff 26324.May 22 2015, 7:54 AM
vharron retitled this revision from to dotest.py - fixed a bug displaying usage.
vharron updated this object.
vharron edited the test plan for this revision. (Show Details)
vharron added reviewers: clayborg, zturner.
vharron added a subscriber: Unknown Object (MLST).
zturner edited edge metadata.May 22 2015, 10:34 AM

Looks fine, in the future it seems like a good cleanup would be to move all
the argument validation into dotest_args, but this seems fine for now.

clayborg accepted this revision.May 26 2015, 3:14 PM
clayborg edited edge metadata.

I believe this is already fixed?

This revision is now accepted and ready to land.May 26 2015, 3:14 PM

Vince submitted a fix for this in r238043.

vharron closed this revision.May 26 2015, 9:47 PM