From 6d42675a3a10de1dcb20116faca396263786feef Mon Sep 17 00:00:00 2001 From: J-P Nurmi Date: Sun, 13 Sep 2015 02:07:14 +0200 Subject: [PATCH] CMessage::SetParams(): forget the colon 1a3e9ec made CMessage try to retain the colon if the original message contained one. We should not, however, remember that when the params are replaced entirely. Consider for example an extended-join message that is made suitable for a client that doesn't have extended-join capability: CMessage msg(":nick!ident@host JOIN #chan account :real name"); msg.SetParams({msg.GetParam(0)}); msg.ToString(); // ":nick!ident@host JOIN :#chan" --- src/Message.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/Message.cpp b/src/Message.cpp index e02d28ce..6d9c5ad2 100644 --- a/src/Message.cpp +++ b/src/Message.cpp @@ -65,6 +65,7 @@ CString CMessage::GetParams(unsigned int uIdx, unsigned int uLen) const void CMessage::SetParams(const VCString& vsParams) { m_vsParams = vsParams; + m_bColon = false; if (m_eType == Type::Text || m_eType == Type::Notice || m_eType == Type::Action || m_eType == Type::CTCP) { InitType();