summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFlorent Le Coz <louiz@louiz.org>2015-12-03 21:09:03 +0100
committerFlorent Le Coz <louiz@louiz.org>2015-12-03 21:09:03 +0100
commit756b297045c5ec3ce051b48379a692d80398ef47 (patch)
tree321954d2eddcd85f26a7d59c5deff5e7d93bc53b
parent284af79128c4bae9fa76f2dd5916ebfd81ecf718 (diff)
downloadbiboumi-756b297045c5ec3ce051b48379a692d80398ef47.tar.gz
biboumi-756b297045c5ec3ce051b48379a692d80398ef47.tar.bz2
biboumi-756b297045c5ec3ce051b48379a692d80398ef47.tar.xz
biboumi-756b297045c5ec3ce051b48379a692d80398ef47.zip
Small cleanup
-rw-r--r--louloulibs/xmpp/adhoc_session.cpp4
-rw-r--r--louloulibs/xmpp/adhoc_session.hpp7
-rw-r--r--louloulibs/xmpp/jid.cpp2
3 files changed, 2 insertions, 11 deletions
diff --git a/louloulibs/xmpp/adhoc_session.cpp b/louloulibs/xmpp/adhoc_session.cpp
index bf8d292..dda4bea 100644
--- a/louloulibs/xmpp/adhoc_session.cpp
+++ b/louloulibs/xmpp/adhoc_session.cpp
@@ -13,10 +13,6 @@ AdhocSession::AdhocSession(const AdhocCommand& command, const std::string& owner
{
}
-AdhocSession::~AdhocSession()
-{
-}
-
const AdhocStep& AdhocSession::get_next_step()
{
assert(this->current_step < this->command.callbacks.size());
diff --git a/louloulibs/xmpp/adhoc_session.hpp b/louloulibs/xmpp/adhoc_session.hpp
index 05afc91..7f07dc1 100644
--- a/louloulibs/xmpp/adhoc_session.hpp
+++ b/louloulibs/xmpp/adhoc_session.hpp
@@ -16,17 +16,15 @@ class AdhocSession;
* A function executed as an ad-hoc command step. It takes a <command/>
* XmlNode and modifies it accordingly (inserting for example an <error/>
* node, or a data form…).
- * TODO fix this:
- * It also must call one of step_passed(), cancel() etc on the AdhocSession object.
*/
-typedef std::function<void(XmppComponent*, AdhocSession&, XmlNode&)> AdhocStep;
+using AdhocStep = std::function<void(XmppComponent*, AdhocSession&, XmlNode&)>;
class AdhocSession
{
public:
explicit AdhocSession(const AdhocCommand& command, const std::string& owner_jid,
const std::string& to_jid);
- ~AdhocSession();
+ ~AdhocSession() = default;
/**
* Return the function to be executed, found in our AdhocCommand, for the
* current_step. And increment the current_step.
@@ -52,7 +50,6 @@ public:
return this->owner_jid;
}
-
private:
/**
* A reference of the command concerned by this session. Used for example
diff --git a/louloulibs/xmpp/jid.cpp b/louloulibs/xmpp/jid.cpp
index e6fee45..dcd7012 100644
--- a/louloulibs/xmpp/jid.cpp
+++ b/louloulibs/xmpp/jid.cpp
@@ -30,8 +30,6 @@ Jid::Jid(const std::string& jid)
this->domain = jid.substr(at, slash - at);
}
-#include <iostream>
-
static constexpr size_t max_jid_part_len = 1023;
std::string jidprep(const std::string& original)