diff options
| author | Jeremy Lainé <jeremy.laine@m4x.org> | 2010-02-22 14:56:11 +0000 |
|---|---|---|
| committer | Jeremy Lainé <jeremy.laine@m4x.org> | 2010-02-22 14:56:11 +0000 |
| commit | eddc6bebeed612f448bb1b59dfba1f942032b046 (patch) | |
| tree | 63ef23319e19fcca46822a2bef1888e02f196df1 /source/QXmppMessage.cpp | |
| parent | be8f5fd37b142e8a6e76a29f6e5f15d5c8e2b7d5 (diff) | |
| download | qxmpp-eddc6bebeed612f448bb1b59dfba1f942032b046.tar.gz | |
* check whether body is empty before adding "body" element to a message
* remove some gratuitous accessor calls
Diffstat (limited to 'source/QXmppMessage.cpp')
| -rw-r--r-- | source/QXmppMessage.cpp | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/source/QXmppMessage.cpp b/source/QXmppMessage.cpp index 72ae50c2..580e8dc1 100644 --- a/source/QXmppMessage.cpp +++ b/source/QXmppMessage.cpp @@ -173,11 +173,12 @@ void QXmppMessage::toXml(QXmlStreamWriter *xmlWriter) const helperToXmlAddAttribute(xmlWriter, "to", getTo()); helperToXmlAddAttribute(xmlWriter, "from", getFrom()); helperToXmlAddAttribute(xmlWriter, "type", getTypeStr()); - if (!getSubject().isEmpty()) - helperToXmlAddTextElement(xmlWriter, "subject", getSubject()); - helperToXmlAddTextElement(xmlWriter,"body", getBody()); - if (!getThread().isEmpty()) - helperToXmlAddTextElement(xmlWriter,"thread", getThread()); + if (!m_subject.isEmpty()) + helperToXmlAddTextElement(xmlWriter, "subject", m_subject); + if (!m_body.isEmpty()) + helperToXmlAddTextElement(xmlWriter, "body", m_body); + if (!m_thread.isEmpty()) + helperToXmlAddTextElement(xmlWriter, "thread", m_thread); getError().toXml(xmlWriter); if (m_state > None && m_state <= Paused) |