diff options
author | attilamolnar <attilamolnar@hush.com> | 2013-05-16 15:45:05 +0200 |
---|---|---|
committer | attilamolnar <attilamolnar@hush.com> | 2013-05-18 21:08:25 +0200 |
commit | 21eba8e86b3d7361bc9334b4e15e968ba095db76 (patch) | |
tree | cc13a1daebf1bae4d86602451f55aa29295c1a72 /src | |
parent | 780757cbc172daa4d9973e8e3b87fd42cfac5541 (diff) |
irc::stringjoiner cleanup
- Get rid of unused constructors
- signed -> unsigned
- return const ref from GetJoined()
Diffstat (limited to 'src')
-rw-r--r-- | src/hashcomp.cpp | 26 |
1 files changed, 3 insertions, 23 deletions
diff --git a/src/hashcomp.cpp b/src/hashcomp.cpp index a2f99b70c..9fded8b4b 100644 --- a/src/hashcomp.cpp +++ b/src/hashcomp.cpp @@ -430,37 +430,17 @@ int irc::modestacker::GetStackedLine(std::vector<std::string> &result, int max_l return n; } -irc::stringjoiner::stringjoiner(const std::string &seperator, const std::vector<std::string> &sequence, int begin, int end) +irc::stringjoiner::stringjoiner(const std::string& seperator, const std::vector<std::string>& sequence, unsigned int begin, unsigned int end) { if (end < begin) return; // nothing to do here - for (int v = begin; v < end; v++) + for (unsigned int v = begin; v < end; v++) joined.append(sequence[v]).append(seperator); joined.append(sequence[end]); } -irc::stringjoiner::stringjoiner(const std::string &seperator, const std::deque<std::string> &sequence, int begin, int end) -{ - if (end < begin) - return; // nothing to do here - - for (int v = begin; v < end; v++) - joined.append(sequence[v]).append(seperator); - joined.append(sequence[end]); -} - -irc::stringjoiner::stringjoiner(const std::string &seperator, const char* const* sequence, int begin, int end) -{ - if (end < begin) - return; // nothing to do here - - for (int v = begin; v < end; v++) - joined.append(sequence[v]).append(seperator); - joined.append(sequence[end]); -} - -std::string& irc::stringjoiner::GetJoined() +const std::string& irc::stringjoiner::GetJoined() const { return joined; } |