diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-02-15 12:56:52 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-02-18 07:15:57 +0100 |
| commit | aa51774e6a309f277e71ca3a3b9d5d5b4b3dbf1a (patch) | |
| tree | c69ad9f8591f69749699ddd7c108238820532eb3 /idlc | |
| parent | 9712dd74bfb0c9b99cab37bd147fe267b48c6d7d (diff) | |
loplugin:simplifybool, check for !(!a op !b)
Change-Id: Ic3ee9c05705817580633506498f848aac3ab7ba6
Reviewed-on: https://gerrit.libreoffice.org/67866
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'idlc')
| -rw-r--r-- | idlc/source/options.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/idlc/source/options.cxx b/idlc/source/options.cxx index 04db926dad0d..4f49cd202de4 100644 --- a/idlc/source/options.cxx +++ b/idlc/source/options.cxx @@ -227,7 +227,7 @@ bool Options::initOptions(std::vector< std::string > & rArgs) { case 'O': { - if (!((++first != last) && ((*first)[0] != '-'))) + if ((++first == last) || ((*first)[0] == '-')) { return badOption("invalid", option); } @@ -237,7 +237,7 @@ bool Options::initOptions(std::vector< std::string > & rArgs) } case 'M': { - if (!((++first != last) && ((*first)[0] != '-'))) + if ((++first == last) || ((*first)[0] == '-')) { return badOption("invalid", option); } @@ -247,7 +247,7 @@ bool Options::initOptions(std::vector< std::string > & rArgs) } case 'I': { - if (!((++first != last) && ((*first)[0] != '-'))) + if ((++first == last) || ((*first)[0] == '-')) { return badOption("invalid", option); } @@ -283,7 +283,7 @@ bool Options::initOptions(std::vector< std::string > & rArgs) } case 'D': { - if (!((++first != last) && ((*first)[0] != '-'))) + if ((++first == last) || ((*first)[0] == '-')) { return badOption("invalid", option); } |