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/CPack/IFW | |
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/CPack/IFW')
-rw-r--r-- | Source/CPack/IFW/cmCPackIFWGenerator.cxx | 22 | ||||
-rw-r--r-- | Source/CPack/IFW/cmCPackIFWInstaller.cxx | 9 | ||||
-rw-r--r-- | Source/CPack/IFW/cmCPackIFWPackage.cxx | 14 |
3 files changed, 0 insertions, 45 deletions
diff --git a/Source/CPack/IFW/cmCPackIFWGenerator.cxx b/Source/CPack/IFW/cmCPackIFWGenerator.cxx index c10803c..cf8587c 100644 --- a/Source/CPack/IFW/cmCPackIFWGenerator.cxx +++ b/Source/CPack/IFW/cmCPackIFWGenerator.cxx @@ -31,38 +31,32 @@ #include <cmVersionConfig.h> #include <cmXMLWriter.h> -//---------------------------------------------------------------------------- cmCPackIFWGenerator::cmCPackIFWGenerator() { } -//---------------------------------------------------------------------------- cmCPackIFWGenerator::~cmCPackIFWGenerator() { } -//---------------------------------------------------------------------------- bool cmCPackIFWGenerator::IsVersionLess(const char *version) { return cmSystemTools::VersionCompare(cmSystemTools::OP_LESS, FrameworkVersion.data(), version); } -//---------------------------------------------------------------------------- bool cmCPackIFWGenerator::IsVersionGreater(const char *version) { return cmSystemTools::VersionCompare(cmSystemTools::OP_GREATER, FrameworkVersion.data(), version); } -//---------------------------------------------------------------------------- bool cmCPackIFWGenerator::IsVersionEqual(const char *version) { return cmSystemTools::VersionCompare(cmSystemTools::OP_EQUAL, FrameworkVersion.data(), version); } -//---------------------------------------------------------------------------- int cmCPackIFWGenerator::PackageFiles() { cmCPackLogger(cmCPackLog::LOG_OUTPUT, "- Configuration" << std::endl); @@ -223,7 +217,6 @@ int cmCPackIFWGenerator::PackageFiles() return 1; } -//---------------------------------------------------------------------------- const char *cmCPackIFWGenerator::GetPackagingInstallPrefix() { const char *defPrefix = cmCPackGenerator::GetPackagingInstallPrefix(); @@ -240,13 +233,11 @@ const char *cmCPackIFWGenerator::GetPackagingInstallPrefix() return this->GetOption("CPACK_IFW_PACKAGING_INSTALL_PREFIX"); } -//---------------------------------------------------------------------------- const char *cmCPackIFWGenerator::GetOutputExtension() { return ExecutableSuffix.c_str(); } -//---------------------------------------------------------------------------- int cmCPackIFWGenerator::InitializeInternal() { // Search Qt Installer Framework tools @@ -367,7 +358,6 @@ int cmCPackIFWGenerator::InitializeInternal() return this->Superclass::InitializeInternal(); } -//---------------------------------------------------------------------------- std::string cmCPackIFWGenerator::GetComponentInstallDirNameSuffix( const std::string& componentName) @@ -384,7 +374,6 @@ cmCPackIFWGenerator::GetComponentInstallDirNameSuffix( + suffix; } -//---------------------------------------------------------------------------- cmCPackComponent* cmCPackIFWGenerator::GetComponent(const std::string &projectName, const std::string &componentName) @@ -433,7 +422,6 @@ cmCPackIFWGenerator::GetComponent(const std::string &projectName, return component; } -//---------------------------------------------------------------------------- cmCPackComponentGroup* cmCPackIFWGenerator::GetComponentGroup(const std::string &projectName, const std::string &groupName) @@ -471,32 +459,27 @@ cmCPackIFWGenerator::GetComponentGroup(const std::string &projectName, return group; } -//---------------------------------------------------------------------------- enum cmCPackGenerator::CPackSetDestdirSupport cmCPackIFWGenerator::SupportsSetDestdir() const { return cmCPackGenerator::SETDESTDIR_SHOULD_NOT_BE_USED; } -//---------------------------------------------------------------------------- bool cmCPackIFWGenerator::SupportsAbsoluteDestination() const { return false; } -//---------------------------------------------------------------------------- bool cmCPackIFWGenerator::SupportsComponentInstallation() const { return true; } -//---------------------------------------------------------------------------- bool cmCPackIFWGenerator::IsOnePackage() const { return componentPackageMethod == ONE_PACKAGE; } -//---------------------------------------------------------------------------- std::string cmCPackIFWGenerator::GetRootPackageName() { // Default value @@ -525,7 +508,6 @@ std::string cmCPackIFWGenerator::GetRootPackageName() return name; } -//---------------------------------------------------------------------------- std::string cmCPackIFWGenerator::GetGroupPackageName(cmCPackComponentGroup *group) const { @@ -556,7 +538,6 @@ cmCPackIFWGenerator::GetGroupPackageName(cmCPackComponentGroup *group) const return name; } -//---------------------------------------------------------------------------- std::string cmCPackIFWGenerator::GetComponentPackageName( cmCPackComponent *component) const { @@ -592,7 +573,6 @@ std::string cmCPackIFWGenerator::GetComponentPackageName( return name; } -//---------------------------------------------------------------------------- cmCPackIFWPackage* cmCPackIFWGenerator::GetGroupPackage( cmCPackComponentGroup *group) const { @@ -601,7 +581,6 @@ cmCPackIFWPackage* cmCPackIFWGenerator::GetGroupPackage( return pit != GroupPackages.end() ? pit->second : 0; } -//---------------------------------------------------------------------------- cmCPackIFWPackage* cmCPackIFWGenerator::GetComponentPackage( cmCPackComponent *component) const { @@ -610,7 +589,6 @@ cmCPackIFWPackage* cmCPackIFWGenerator::GetComponentPackage( return pit != ComponentPackages.end() ? pit->second : 0; } -//---------------------------------------------------------------------------- void cmCPackIFWGenerator::WriteGeneratedByToStrim(cmXMLWriter &xout) { std::stringstream comment; diff --git a/Source/CPack/IFW/cmCPackIFWInstaller.cxx b/Source/CPack/IFW/cmCPackIFWInstaller.cxx index 224d606..dfb336c 100644 --- a/Source/CPack/IFW/cmCPackIFWInstaller.cxx +++ b/Source/CPack/IFW/cmCPackIFWInstaller.cxx @@ -32,44 +32,37 @@ } \ } while ( 0 ) -//---------------------------------------------------------------------------- cmCPackIFWInstaller::cmCPackIFWInstaller() : Generator(0) { } -//---------------------------------------------------------------------------- const char *cmCPackIFWInstaller::GetOption(const std::string &op) const { return Generator ? Generator->GetOption(op) : 0; } -//---------------------------------------------------------------------------- bool cmCPackIFWInstaller::IsOn(const std::string &op) const { return Generator ? Generator->IsOn(op) : false; } -//---------------------------------------------------------------------------- bool cmCPackIFWInstaller::IsVersionLess(const char *version) { return Generator ? Generator->IsVersionLess(version) : false; } -//---------------------------------------------------------------------------- bool cmCPackIFWInstaller::IsVersionGreater(const char *version) { return Generator ? Generator->IsVersionGreater(version) : false; } -//---------------------------------------------------------------------------- bool cmCPackIFWInstaller::IsVersionEqual(const char *version) { return Generator ? Generator->IsVersionEqual(version) : false; } -//---------------------------------------------------------------------------- void cmCPackIFWInstaller::ConfigureFromOptions() { // Name; @@ -324,7 +317,6 @@ void cmCPackIFWInstaller::ConfigureFromOptions() } } -//---------------------------------------------------------------------------- void cmCPackIFWInstaller::GenerateInstallerFile() { // Lazy directory initialization @@ -486,7 +478,6 @@ void cmCPackIFWInstaller::GenerateInstallerFile() xout.EndDocument(); } -//---------------------------------------------------------------------------- void cmCPackIFWInstaller::GeneratePackageFiles() { if (Packages.empty() || Generator->IsOnePackage()) diff --git a/Source/CPack/IFW/cmCPackIFWPackage.cxx b/Source/CPack/IFW/cmCPackIFWPackage.cxx index b71c7e3..a22c0b9 100644 --- a/Source/CPack/IFW/cmCPackIFWPackage.cxx +++ b/Source/CPack/IFW/cmCPackIFWPackage.cxx @@ -45,7 +45,6 @@ cmCPackIFWPackage::DependenceStruct::DependenceStruct() { } -//---------------------------------------------------------------------------- cmCPackIFWPackage::DependenceStruct::DependenceStruct( const std::string &dependence) { @@ -79,7 +78,6 @@ cmCPackIFWPackage::DependenceStruct::DependenceStruct( Name = pos == std::string::npos ? dependence : dependence.substr(0, pos); } -//---------------------------------------------------------------------------- std::string cmCPackIFWPackage::DependenceStruct::NameWithCompare() const { if (Compare.Type == CompareNone) return Name; @@ -119,38 +117,32 @@ cmCPackIFWPackage::cmCPackIFWPackage() : { } -//---------------------------------------------------------------------------- const char *cmCPackIFWPackage::GetOption(const std::string &op) const { const char *option = Generator ? Generator->GetOption(op) : 0; return option && *option ? option : 0; } -//---------------------------------------------------------------------------- bool cmCPackIFWPackage::IsOn(const std::string &op) const { return Generator ? Generator->IsOn(op) : false; } -//---------------------------------------------------------------------------- bool cmCPackIFWPackage::IsVersionLess(const char *version) { return Generator ? Generator->IsVersionLess(version) : false; } -//---------------------------------------------------------------------------- bool cmCPackIFWPackage::IsVersionGreater(const char *version) { return Generator ? Generator->IsVersionGreater(version) : false; } -//---------------------------------------------------------------------------- bool cmCPackIFWPackage::IsVersionEqual(const char *version) { return Generator ? Generator->IsVersionEqual(version) : false; } -//---------------------------------------------------------------------------- std::string cmCPackIFWPackage::GetComponentName(cmCPackComponent *component) { if (!component) return ""; @@ -161,7 +153,6 @@ std::string cmCPackIFWPackage::GetComponentName(cmCPackComponent *component) return option ? option : component->Name; } -//---------------------------------------------------------------------------- void cmCPackIFWPackage::DefaultConfiguration() { DisplayName = ""; @@ -176,7 +167,6 @@ void cmCPackIFWPackage::DefaultConfiguration() ForcedInstallation = ""; } -//---------------------------------------------------------------------------- // Defaul configuration (all in one package) int cmCPackIFWPackage::ConfigureFromOptions() { @@ -222,7 +212,6 @@ int cmCPackIFWPackage::ConfigureFromOptions() return 1; } -//---------------------------------------------------------------------------- int cmCPackIFWPackage::ConfigureFromComponent(cmCPackComponent *component) { if(!component) return 0; @@ -329,7 +318,6 @@ int cmCPackIFWPackage::ConfigureFromComponent(cmCPackComponent *component) return 1; } -//---------------------------------------------------------------------------- int cmCPackIFWPackage::ConfigureFromGroup(cmCPackComponentGroup *group) { @@ -389,7 +377,6 @@ cmCPackIFWPackage::ConfigureFromGroup(cmCPackComponentGroup *group) return 1; } -//---------------------------------------------------------------------------- int cmCPackIFWPackage::ConfigureFromGroup(const std::string &groupName) { // Group configuration @@ -431,7 +418,6 @@ int cmCPackIFWPackage::ConfigureFromGroup(const std::string &groupName) return ConfigureFromGroup(&group); } -//---------------------------------------------------------------------------- void cmCPackIFWPackage::GeneratePackageFile() { // Lazy directory initialization |