diff options
author | Brad King <brad.king@kitware.com> | 2016-05-06 18:30:06 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2016-05-09 13:41:43 (GMT) |
commit | 0ac18d40c8c29a17f1acfcaca506f41a26185901 (patch) | |
tree | 88b5d3c4b8c587d7979231d0e6ce9b03b0685a74 /Source/cmIDEOptions.cxx | |
parent | d95fbdb70944a8f9a7e6ac11bc51f410a99aafcd (diff) | |
download | CMake-0ac18d40c8c29a17f1acfcaca506f41a26185901.zip CMake-0ac18d40c8c29a17f1acfcaca506f41a26185901.tar.gz CMake-0ac18d40c8c29a17f1acfcaca506f41a26185901.tar.bz2 |
Remove `//------...` horizontal separator comments
Modern editors provide plenty of ways to visually separate functions.
Drop the explicit comments that previously served this purpose.
Use the following command to automate the change:
$ git ls-files -z -- \
"*.c" "*.cc" "*.cpp" "*.cxx" "*.h" "*.hh" "*.hpp" "*.hxx" |
egrep -z -v "^Source/cmCommandArgumentLexer\." |
egrep -z -v "^Source/cmCommandArgumentParser(\.y|\.cxx|Tokens\.h)" |
egrep -z -v "^Source/cmDependsJavaLexer\." |
egrep -z -v "^Source/cmDependsJavaParser(\.y|\.cxx|Tokens\.h)" |
egrep -z -v "^Source/cmExprLexer\." |
egrep -z -v "^Source/cmExprParser(\.y|\.cxx|Tokens\.h)" |
egrep -z -v "^Source/cmFortranLexer\." |
egrep -z -v "^Source/cmFortranParser(\.y|\.cxx|Tokens\.h)" |
egrep -z -v "^Source/cmListFileLexer\." |
egrep -z -v "^Source/cm_sha2" |
egrep -z -v "^Source/(kwsys|CursesDialog/form)/" |
egrep -z -v "^Utilities/(KW|cm).*/" |
xargs -0 sed -i '/^\(\/\/---*\|\/\*---*\*\/\)$/ {d;}'
This avoids modifying third-party sources and generated sources.
Diffstat (limited to 'Source/cmIDEOptions.cxx')
-rw-r--r-- | Source/cmIDEOptions.cxx | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/Source/cmIDEOptions.cxx b/Source/cmIDEOptions.cxx index 509602f..1c52519 100644 --- a/Source/cmIDEOptions.cxx +++ b/Source/cmIDEOptions.cxx @@ -15,7 +15,6 @@ #include <cmsys/String.h> -//---------------------------------------------------------------------------- cmIDEOptions::cmIDEOptions() { this->DoingDefine = false; @@ -28,12 +27,10 @@ cmIDEOptions::cmIDEOptions() } } -//---------------------------------------------------------------------------- cmIDEOptions::~cmIDEOptions() { } -//---------------------------------------------------------------------------- void cmIDEOptions::HandleFlag(const char* flag) { // If the last option was -D then this option is the definition. @@ -92,7 +89,6 @@ void cmIDEOptions::HandleFlag(const char* flag) this->StoreUnknownFlag(flag); } -//---------------------------------------------------------------------------- bool cmIDEOptions::CheckFlagTable(cmIDEFlagTable const* table, const char* flag, bool& flag_handled) { @@ -147,7 +143,6 @@ bool cmIDEOptions::CheckFlagTable(cmIDEFlagTable const* table, return false; } -//---------------------------------------------------------------------------- void cmIDEOptions::FlagMapUpdate(cmIDEFlagTable const* entry, const char* new_value) { @@ -167,13 +162,11 @@ void cmIDEOptions::FlagMapUpdate(cmIDEFlagTable const* entry, } } -//---------------------------------------------------------------------------- void cmIDEOptions::AddDefine(const std::string& def) { this->Defines.push_back(def); } -//---------------------------------------------------------------------------- void cmIDEOptions::AddDefines(const char* defines) { if(defines) @@ -182,33 +175,28 @@ void cmIDEOptions::AddDefines(const char* defines) cmSystemTools::ExpandListArgument(defines, this->Defines); } } -//---------------------------------------------------------------------------- void cmIDEOptions::AddDefines(const std::vector<std::string> &defines) { this->Defines.insert(this->Defines.end(), defines.begin(), defines.end()); } -//---------------------------------------------------------------------------- void cmIDEOptions::AddFlag(const char* flag, const char* value) { this->FlagMap[flag] = value; } -//---------------------------------------------------------------------------- void cmIDEOptions::AddFlag(const char* flag, std::vector<std::string> const& value) { this->FlagMap[flag] = value; } -//---------------------------------------------------------------------------- void cmIDEOptions::AppendFlag(std::string const& flag, std::string const& value) { this->FlagMap[flag].push_back(value); } -//---------------------------------------------------------------------------- void cmIDEOptions::AppendFlag(std::string const& flag, std::vector<std::string> const& value) { @@ -216,19 +204,16 @@ void cmIDEOptions::AppendFlag(std::string const& flag, std::copy(value.begin(), value.end(), std::back_inserter(fv)); } -//---------------------------------------------------------------------------- void cmIDEOptions::RemoveFlag(const char* flag) { this->FlagMap.erase(flag); } -//---------------------------------------------------------------------------- bool cmIDEOptions::HasFlag(std::string const& flag) const { return this->FlagMap.find(flag) != this->FlagMap.end(); } -//---------------------------------------------------------------------------- const char* cmIDEOptions::GetFlag(const char* flag) { // This method works only for single-valued flags! |