Move comments off of class access specifier lines

The clang-format tool may turn this:

   public: // comment about access specifier

     // unrelated comment indented with code
     ...

Into:

   public: // comment about access specifier

           // unrelated comment indented with code
     ...

Avoid this by moving comments off of access specifier lines.
This commit is contained in:
Brad King 2016-05-06 14:49:44 -04:00
parent 64b5520346
commit 85425a3e6c
5 changed files with 28 additions and 13 deletions

View File

@ -61,7 +61,8 @@ public:
*/ */
bool IsVersionEqual(const char *version); bool IsVersionEqual(const char *version);
protected: // cmCPackGenerator reimplementation protected:
// cmCPackGenerator reimplementation
/** /**
* @brief Initialize generator * @brief Initialize generator
@ -110,7 +111,8 @@ protected: // cmCPackGenerator reimplementation
virtual bool SupportsAbsoluteDestination() const; virtual bool SupportsAbsoluteDestination() const;
virtual bool SupportsComponentInstallation() const; virtual bool SupportsComponentInstallation() const;
protected: // Methods protected:
// Methods
bool IsOnePackage() const; bool IsOnePackage() const;
@ -124,7 +126,8 @@ protected: // Methods
void WriteGeneratedByToStrim(cmXMLWriter& xout); void WriteGeneratedByToStrim(cmXMLWriter& xout);
protected: // Data protected:
// Data
friend class cmCPackIFWPackage; friend class cmCPackIFWPackage;
friend class cmCPackIFWInstaller; friend class cmCPackIFWInstaller;

View File

@ -24,7 +24,8 @@ class cmXMLWriter;
*/ */
class cmCPackIFWInstaller class cmCPackIFWInstaller
{ {
public: // Types public:
// Types
typedef std::map<std::string, cmCPackIFWPackage*> PackagesMap; typedef std::map<std::string, cmCPackIFWPackage*> PackagesMap;
@ -37,14 +38,16 @@ public: // Types
std::string DisplayName; std::string DisplayName;
}; };
public: // Constructor public:
// Constructor
/** /**
* Construct installer * Construct installer
*/ */
cmCPackIFWInstaller(); cmCPackIFWInstaller();
public: // Configuration public:
// Configuration
/// Name of the product being installed /// Name of the product being installed
std::string Name; std::string Name;
@ -94,7 +97,8 @@ public: // Configuration
/// Filename for a custom installer control script /// Filename for a custom installer control script
std::string ControlScript; std::string ControlScript;
public: // Internal implementation public:
// Internal implementation
const char* GetOption(const std::string& op) const; const char* GetOption(const std::string& op) const;
bool IsOn(const std::string& op) const; bool IsOn(const std::string& op) const;

View File

@ -26,7 +26,9 @@ class cmXMLWriter;
*/ */
class cmCPackIFWPackage class cmCPackIFWPackage
{ {
public: // Types public:
// Types
enum CompareTypes enum CompareTypes
{ {
CompareNone = 0x0, CompareNone = 0x0,
@ -61,14 +63,16 @@ public: // Types
} }
}; };
public: // [Con|De]structor public:
// [Con|De]structor
/** /**
* Construct package * Construct package
*/ */
cmCPackIFWPackage(); cmCPackIFWPackage();
public: // Configuration public:
// Configuration
/// Human-readable name of the component /// Human-readable name of the component
std::string DisplayName; std::string DisplayName;
@ -103,7 +107,8 @@ public: // Configuration
/// Determines that the package must always be installed /// Determines that the package must always be installed
std::string ForcedInstallation; std::string ForcedInstallation;
public: // Internal implementation public:
// Internal implementation
const char* GetOption(const std::string& op) const; const char* GetOption(const std::string& op) const;
bool IsOn(const std::string& op) const; bool IsOn(const std::string& op) const;

View File

@ -44,7 +44,9 @@ private:
void LoadRevisions(); void LoadRevisions();
void LoadModifications(); void LoadModifications();
public: // needed by older Sun compilers // "public" needed by older Sun compilers
public:
// Parsing helper classes. // Parsing helper classes.
class OneLineParser; class OneLineParser;
class DiffParser; class DiffParser;

View File

@ -69,7 +69,8 @@ protected:
virtual bool WriteXMLUpdates(cmXMLWriter& xml); virtual bool WriteXMLUpdates(cmXMLWriter& xml);
#if defined(__SUNPRO_CC) && __SUNPRO_CC <= 0x510 #if defined(__SUNPRO_CC) && __SUNPRO_CC <= 0x510
public: // Sun CC 5.1 needs help to allow cmCTestSVN::Revision to see this // Sun CC 5.1 needs help to allow cmCTestSVN::Revision to see this
public:
#endif #endif
/** Basic information about one revision of a tree or file. */ /** Basic information about one revision of a tree or file. */
struct Revision struct Revision