[Home]

Summary:ASTERISK-10437: [patch] Does not like , as seperator (still using |)
Reporter:Niklas Larsson (pnlarsson)Labels:
Date Opened:2007-10-03 13:28:07Date Closed:2007-10-04 11:35:37
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Resources/res_jabber
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20071003__bug10876.diff.txt
( 1) res_jabber.diff.txt
Description:JabberSend(asterisk,nick@tese.se,Hello) does not work becouse it expects | instead of ,
Comments:By: Niklas Larsson (pnlarsson) 2007-10-03 13:28:50

Patch added
By: Dmitry Andrianov (dimas) 2007-10-03 13:55:36

Hmmm... Isn't it a better way to use special asterisk macros for that
       AST_DECLARE_APP_ARGS(args,
               AST_APP_ARG(sender);
               AST_APP_ARG(recipient);
               AST_APP_ARG(message);
       );
?
By: Tilghman Lesher (tilghman) 2007-10-03 14:59:59

I was actually working on that, specifically.
By: Niklas Larsson (pnlarsson) 2007-10-04 08:19:49

Tried your patch (20071003__bug10876.diff.txt) and it's doing fine.
By: Digium Subversion (svnbot) 2007-10-04 11:35:37

Repository: asterisk
Revision: 84671

U   trunk/res/res_jabber.c

------------------------------------------------------------------------
r84671 | tilghman | 2007-10-04 11:35:36 -0500 (Thu, 04 Oct 2007) | 2 lines

Update to current coding standards, also changing the argument delimiter to ',' (Closes issue ASTERISK-10437)

------------------------------------------------------------------------