diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2014-05-18 15:14:13 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2014-05-18 15:14:13 +0000 |
commit | aad181f64485fbc75ce02b7dc993004d3aa168c2 (patch) | |
tree | 2ce79acebb20a147f1a0ce764a685dc96c378610 /lib/Option | |
parent | bf3e78f5728f116067f8a373a522c221ccdaa08f (diff) | |
download | llvm-aad181f64485fbc75ce02b7dc993004d3aa168c2.tar.gz llvm-aad181f64485fbc75ce02b7dc993004d3aa168c2.tar.bz2 llvm-aad181f64485fbc75ce02b7dc993004d3aa168c2.tar.xz |
Options: Use erase_if to remove Args from the list.
While there make getOption return a const reference so we don't have to put it
on the stack when calling methods on it. No functionality change.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@209088 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Option')
-rw-r--r-- | lib/Option/ArgList.cpp | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/lib/Option/ArgList.cpp b/lib/Option/ArgList.cpp index 1f16331e07..a5ab8d747d 100644 --- a/lib/Option/ArgList.cpp +++ b/lib/Option/ArgList.cpp @@ -41,14 +41,9 @@ void ArgList::append(Arg *A) { } void ArgList::eraseArg(OptSpecifier Id) { - for (iterator it = begin(), ie = end(); it != ie; ) { - if ((*it)->getOption().matches(Id)) { - it = Args.erase(it); - ie = end(); - } else { - ++it; - } - } + Args.erase(std::remove_if(begin(), end(), + [=](Arg *A) { return A->getOption().matches(Id); }), + end()); } Arg *ArgList::getLastArgNoClaim(OptSpecifier Id) const { |