# HG changeset patch
# User Kim Alvefur <zash@zash.se>
# Date 1373663910 -7200
# Node ID f1f0c87cffdd16d3c233349c8aed58720ba451e2
# Parent  1dc07833355eeec02c723bba3a25f7f731e4fa88
mod_mam: Neater reply building

diff -r 1dc07833355e -r f1f0c87cffdd mod_mam/mod_mam.lua
--- a/mod_mam/mod_mam.lua	Fri Jul 12 22:54:41 2013 +0200
+++ b/mod_mam/mod_mam.lua	Fri Jul 12 23:18:30 2013 +0200
@@ -213,13 +213,9 @@
 	-- That's all folks!
 	module:log("debug", "Archive query %s completed", tostring(qid));
 
-	local reply = st.reply(stanza);
-	if last then
-		-- This is a bit redundant, isn't it?
-		reply:query(xmlns_mam):add_child(rsm.generate{first = (reverse and last or first), last = (reverse and first or last), count = #data});
-	end
-	origin.send(reply);
-	return true
+	return origin.send(st.reply(stanza)
+		:query(xmlns_mam):add_child(rsm.generate {
+			first = (reverse and last or first), last = (reverse and first or last), count = #data}));
 end);
 
 local function has_in_roster(user, who)