diff --git a/src/modules/preservefiles/permissions.cpp b/src/modules/preservefiles/permissions.cpp index e85386292..a3f8ac136 100644 --- a/src/modules/preservefiles/permissions.cpp +++ b/src/modules/preservefiles/permissions.cpp @@ -21,17 +21,21 @@ #include #include "permissions.h" -Permissions::Permissions(QString p) : +Permissions::Permissions() : m_username(), m_group(), - m_valid(true), + m_valid(false), m_value(0) +{ +} + +Permissions::Permissions(QString p) : Permissions() { parsePermissions(p); } -const void Permissions::parsePermissions(const QString& p) { +void Permissions::parsePermissions(const QString& p) { QStringList segments = p.split(":"); diff --git a/src/modules/preservefiles/permissions.h b/src/modules/preservefiles/permissions.h index 5bbb17fba..4cb70a2c2 100644 --- a/src/modules/preservefiles/permissions.h +++ b/src/modules/preservefiles/permissions.h @@ -32,22 +32,25 @@ class Permissions public: -/** Constructor - * Splits the string @p at the colon (":") into separate elements for - * , , and (permissions), where is returned as - * an **octal** integer. - */ + /** @brief Constructor + * + * Splits the string @p at the colon (":") into separate elements for + * , , and (permissions), where is returned as + * an **octal** integer. + */ Permissions(QString p); + + /** @brief Default constructor of an invalid Permissions. */ + Permissions(); bool isValid() const { return m_valid; } QString username() const { return m_username; } QString group() const { return m_group; } int value() const { return m_value; } - + QString octal() const { return QString::number( m_value, 8 ); } private: - - const void parsePermissions(QString const &p); + void parsePermissions(QString const &p); QString m_username; QString m_group;