diff options
author | Noel Grandin <noel@peralex.com> | 2015-02-11 13:20:49 +0200 |
---|---|---|
committer | Noel Grandin <noel@peralex.com> | 2015-02-23 09:26:58 +0200 |
commit | ba233e87efddf0a6751b35784dca1c805364ff3b (patch) | |
tree | 9d7c8a4256e688c2d47cb6ecf580ac196c4da2c0 /idlc/source | |
parent | import the document properties before the document (diff) | |
download | core-ba233e87efddf0a6751b35784dca1c805364ff3b.tar.gz core-ba233e87efddf0a6751b35784dca1c805364ff3b.zip |
remove unnecessary parenthesis in return statements
found with
$ git grep -lP 'return\s*\(\s*\w+\s*\)\s*;'
Change-Id: Ic51606877a9edcadeb647c5bf17bc928b69ab60e
Diffstat (limited to 'idlc/source')
-rw-r--r-- | idlc/source/idlcmain.cxx | 6 | ||||
-rw-r--r-- | idlc/source/options.cxx | 14 |
2 files changed, 10 insertions, 10 deletions
diff --git a/idlc/source/idlcmain.cxx b/idlc/source/idlcmain.cxx index 9d672fd7bbf0..86a9755b1d08 100644 --- a/idlc/source/idlcmain.cxx +++ b/idlc/source/idlcmain.cxx @@ -29,7 +29,7 @@ SAL_IMPLEMENT_MAIN_WITH_ARGS(argc, argv) for (int i = 1; i < argc; i++) { if (!Options::checkArgument (args, argv[i], strlen(argv[i]))) - return (1); + return 1; } Options options(argv[0]); @@ -38,7 +38,7 @@ SAL_IMPLEMENT_MAIN_WITH_ARGS(argc, argv) try { if (!options.initOptions(args)) - return (0); + return 0; setIdlc(&options); @@ -156,7 +156,7 @@ SAL_IMPLEMENT_MAIN_WITH_ARGS(argc, argv) fprintf(stderr, "Illegal argument: %s\n%s", e.m_message.getStr(), options.prepareVersion().getStr()); - return (99); + return 99; } return nErrors; diff --git a/idlc/source/options.cxx b/idlc/source/options.cxx index dfdb9bd291a9..0db0e215d478 100644 --- a/idlc/source/options.cxx +++ b/idlc/source/options.cxx @@ -110,7 +110,7 @@ bool Options::checkArgument (std::vector< std::string > & rArgs, char const * ar break; } } - return (result); + return result; } // static @@ -120,7 +120,7 @@ bool Options::checkCommandFile (std::vector< std::string > & rArgs, char const * if (fp == 0) { fprintf(stderr, "ERROR: can't open command file \"%s\"\n", filename); - return (false); + return false; } std::string buffer; @@ -147,7 +147,7 @@ bool Options::checkCommandFile (std::vector< std::string > & rArgs, char const * if (!Options::checkArgument(rArgs, buffer.c_str(), buffer.size())) { (void) fclose(fp); - return (false); + return false; } buffer.clear(); } @@ -165,7 +165,7 @@ bool Options::checkCommandFile (std::vector< std::string > & rArgs, char const * if (!Options::checkArgument(rArgs, buffer.c_str(), buffer.size())) { (void) fclose(fp); - return (false); + return false; } buffer.clear(); } @@ -188,7 +188,7 @@ bool Options::setOption(char const * option, std::string const & rArg) bool result = (0 == strcmp(option, rArg.c_str())); if (result) m_options[rArg.c_str()] = OString(rArg.c_str(), rArg.size()); - return (result); + return result; } #ifdef WNT @@ -357,7 +357,7 @@ bool Options::initOptions(std::vector< std::string > & rArgs) throw(IllegalArgum } { (void) fprintf(stdout, "%s", prepareHelp().getStr()); - return (false); + return false; } // break; // Unreachable } @@ -374,7 +374,7 @@ bool Options::initOptions(std::vector< std::string > & rArgs) throw(IllegalArgum return badOption("unknown", option); } } - return (true); + return true; } OString Options::prepareHelp() |