1 files changed, 4 insertions, 3 deletions
diff --git a/Swiften/Serializer/PayloadSerializers/FormSerializer.cpp b/Swiften/Serializer/PayloadSerializers/FormSerializer.cpp
index 15c4f32..ba658e3 100644
--- a/Swiften/Serializer/PayloadSerializers/FormSerializer.cpp
+++ b/Swiften/Serializer/PayloadSerializers/FormSerializer.cpp
@@ -1,44 +1,45 @@
 /*
  * Copyright (c) 2010 Kevin Smith
  * Licensed under the GNU General Public License v3.
  * See Documentation/Licenses/GPLv3.txt for more information.
  */
 
 #include <Swiften/Serializer/PayloadSerializers/FormSerializer.h>
 
 #include <boost/shared_ptr.hpp>
+#include <boost/smart_ptr/make_shared.hpp>
 #include <iostream>
 #include <string>
 
 #include <Swiften/Base/String.h>
 #include <Swiften/Base/Algorithm.h>
 #include <Swiften/Base/foreach.h>
 #include <Swiften/Serializer/XML/XMLTextNode.h>
 #include <Swiften/Serializer/XML/XMLRawTextNode.h>
 
 using namespace Swift;
 
 namespace {
 	template<typename T> void serializeValueAsString(boost::shared_ptr<FormField> field, boost::shared_ptr<XMLElement> parent) {
 		std::string value = boost::dynamic_pointer_cast<T>(field)->getValue();
 		if (!value.empty()) {
 			boost::shared_ptr<XMLElement> valueElement(new XMLElement("value"));
 			valueElement->addNode(XMLTextNode::create(value));
 			parent->addNode(valueElement);
 		}
 	}
 }
 
 
 namespace Swift {
 
 FormSerializer::FormSerializer() : GenericPayloadSerializer<Form>() {
 }
 
 std::string FormSerializer::serializePayload(boost::shared_ptr<Form> form)  const {
 	boost::shared_ptr<XMLElement> formElement(new XMLElement("x", "jabber:x:data"));
 	std::string type;
 	switch (form->getType()) {
 		case Form::FormType: type = "form"; break;
 		case Form::SubmitType: type = "submit"; break;
 		case Form::CancelType: type = "cancel"; break;
@@ -49,75 +50,75 @@ std::string FormSerializer::serializePayload(boost::shared_ptr<Form> form)  cons
 		multiLineify(form->getTitle(), "title", formElement);
 	}
 	if (!form->getInstructions().empty()) {
 		multiLineify(form->getInstructions(), "instructions", formElement);
 	}
 	foreach(boost::shared_ptr<FormField> field, form->getFields()) {
 		formElement->addNode(fieldToXML(field, true));
 	}
 	if (!form->getReportedFields().empty()) {
 		boost::shared_ptr<XMLElement> reportedElement(new XMLElement("reported"));
 		foreach(FormField::ref field, form->getReportedFields()) {
 			reportedElement->addNode(fieldToXML(field, true));
 		}
 		formElement->addNode(reportedElement);
 	}
 	foreach(Form::FormItem item, form->getItems()) {
 		boost::shared_ptr<XMLElement> itemElement(new XMLElement("item"));
 		foreach(FormField::ref field, item) {
 			itemElement->addNode(fieldToXML(field, false));
 		}
 		formElement->addNode(itemElement);
 	}
 
 	return formElement->serialize();
 }
 
 boost::shared_ptr<XMLElement> FormSerializer::fieldToXML(boost::shared_ptr<FormField> field, bool withTypeAttribute) const {
 	boost::shared_ptr<XMLElement> fieldElement(new XMLElement("field"));
 	if (!field->getName().empty()) {
 		fieldElement->setAttribute("var", field->getName());
 	}
 	if (!field->getLabel().empty()) {
 		fieldElement->setAttribute("label", field->getLabel());
 	}
 	if (field->getRequired()) {
-		fieldElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("required")));
+		fieldElement->addNode(boost::make_shared<XMLElement>("required"));
 	}
 	if (!field->getDescription().empty()) {
 		boost::shared_ptr<XMLElement> descriptionElement(new XMLElement("desc"));
-		descriptionElement->addNode(boost::shared_ptr<XMLTextNode>(new XMLTextNode(field->getDescription())));
+		descriptionElement->addNode(boost::make_shared<XMLTextNode>(field->getDescription()));
 		fieldElement->addNode(descriptionElement);
 	}
 
 	// Set the value and type
 	std::string fieldType;
 	if (boost::dynamic_pointer_cast<BooleanFormField>(field)) {
 		fieldType = "boolean";
 		boost::shared_ptr<XMLElement> valueElement(new XMLElement("value"));
 		valueElement->addNode(XMLTextNode::create(boost::dynamic_pointer_cast<BooleanFormField>(field)->getValue() ? "1" : "0"));
 		fieldElement->addNode(valueElement);
 	}
 	else if (boost::dynamic_pointer_cast<FixedFormField>(field)) {
 		fieldType = "fixed";
 		serializeValueAsString<FixedFormField>(field, fieldElement);
 	}
 	else if (boost::dynamic_pointer_cast<HiddenFormField>(field)) {
 		fieldType = "hidden";
 		serializeValueAsString<HiddenFormField>(field, fieldElement);
 	}
 	else if (boost::dynamic_pointer_cast<ListSingleFormField>(field)) {
 		fieldType = "list-single";
 		serializeValueAsString<ListSingleFormField>(field, fieldElement);
 	}
 	else if (boost::dynamic_pointer_cast<TextPrivateFormField>(field)) {
 		fieldType = "text-private";
 		serializeValueAsString<TextPrivateFormField>(field, fieldElement);
 	}
 	else if (boost::dynamic_pointer_cast<TextSingleFormField>(field)) {
 		fieldType = "text-single";
 		serializeValueAsString<TextSingleFormField>(field, fieldElement);
 	}
 	else if (boost::dynamic_pointer_cast<JIDMultiFormField>(field)) {
 		fieldType = "jid-multi";
 		std::vector<JID> jids = boost::dynamic_pointer_cast<JIDMultiFormField>(field)->getValue();
 		foreach(const JID& jid, jids) {
@@ -142,41 +143,41 @@ boost::shared_ptr<XMLElement> FormSerializer::fieldToXML(boost::shared_ptr<FormF
 		}
 	}
 	else if (boost::dynamic_pointer_cast<TextMultiFormField>(field)) {
 		fieldType = "text-multi";
 		multiLineify(boost::dynamic_pointer_cast<TextMultiFormField>(field)->getValue(), "value", fieldElement);
 	}
 	else {
 		assert(false);
 	}
 	if (!fieldType.empty() && withTypeAttribute) {
 		fieldElement->setAttribute("type", fieldType);
 	}
 
 	foreach (const FormField::Option& option, field->getOptions()) {
 		boost::shared_ptr<XMLElement> optionElement(new XMLElement("option"));
 		if (!option.label.empty()) {
 			optionElement->setAttribute("label", option.label);
 		}
 
 		boost::shared_ptr<XMLElement> valueElement(new XMLElement("value"));
 		valueElement->addNode(XMLTextNode::create(option.value));
 		optionElement->addNode(valueElement);
 
 		fieldElement->addNode(optionElement);
 	}
 
 	return fieldElement;
 }
 
 void FormSerializer::multiLineify(const std::string& text, const std::string& elementName, boost::shared_ptr<XMLElement> element) const {
 	std::string unRdText(text);
 	erase(unRdText, '\r');
 	std::vector<std::string> lines = String::split(unRdText, '\n');
 	foreach (std::string line, lines) {
 		boost::shared_ptr<XMLElement> lineElement(new XMLElement(elementName));
-		lineElement->addNode(boost::shared_ptr<XMLTextNode>(new XMLTextNode(line)));
+		lineElement->addNode(boost::make_shared<XMLTextNode>(line));
 		element->addNode(lineElement);
 	}
 }
 
 }