diff options
author | Peter Powell <petpow@saberuk.com> | 2013-08-27 07:54:16 +0100 |
---|---|---|
committer | Peter Powell <petpow@saberuk.com> | 2013-08-27 12:20:03 +0100 |
commit | eaf658de3d1ef984c9a0b4273a9cfbd3029f8b5b (patch) | |
tree | 6d20eef5fd9b98fecd93df2caf197ad08c168562 /include/modules | |
parent | 0e7f74a7c8e804a0223b4d88bf637649838f0412 (diff) |
Fix various small issues.
- Add CXX11_OVERRIDE to *Regex::Matches and *RegexFactory::Create.
- Fix documentation comment on regex_string.
- Fix various code duplication/layout issues.
Diffstat (limited to 'include/modules')
-rw-r--r-- | include/modules/regex.h | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/include/modules/regex.h b/include/modules/regex.h index e278df502..0bced4e2b 100644 --- a/include/modules/regex.h +++ b/include/modules/regex.h @@ -25,18 +25,15 @@ class Regex : public classbase { protected: - std::string regex_string; // The raw uncompiled regex string. + /** The uncompiled regex string. */ + std::string regex_string; // Constructor may as well be protected, as this class is abstract. - Regex(const std::string& rx) : regex_string(rx) - { - } + Regex(const std::string& rx) : regex_string(rx) { } public: - virtual ~Regex() - { - } + virtual ~Regex() { } virtual bool Matches(const std::string& text) = 0; @@ -49,7 +46,7 @@ public: class RegexFactory : public DataProvider { public: - RegexFactory(Module* Creator, const std::string& Name) : DataProvider(Creator, Name) {} + RegexFactory(Module* creator, const std::string& name) : DataProvider(creator, name) { } virtual Regex* Create(const std::string& expr) = 0; }; |