[asterisk-bugs] [JIRA] (ASTERISK-20599) A Module to Submit CDR via Curl to a webservice end point

Matt Jordan (JIRA) noreply at issues.asterisk.org
Tue Oct 30 20:59:18 CDT 2012


     [ https://issues.asterisk.org/jira/browse/ASTERISK-20599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Jordan updated ASTERISK-20599:
-----------------------------------

    Assignee: Rusty Newton  (was: Alex)
      Status: Open  (was: Waiting for Feedback)

So, before readying the patch for Review Board, you should make sure that the patch conforms to the coding guidelines on the Asterisk wiki:


https://wiki.asterisk.org/wiki/display/AST/Coding+Guidelines

Two things I noticed in the patch, but there may be others:

{noformat}
if (foo)
{ 
  /* ... */
}
else
{
  /* ... */
}

{noformat}

should be:

{noformat}

if (foo) {
 /* ... */
} else {
 /* ... */
}

{noformat}

And:

{noformat}

if (foo)
  bar;

{noformat}

should be:

{noformat}

if (foo) {
  bar;
}

{noformat}
                
> A Module to Submit CDR via Curl to a webservice end point
> ---------------------------------------------------------
>
>                 Key: ASTERISK-20599
>                 URL: https://issues.asterisk.org/jira/browse/ASTERISK-20599
>             Project: Asterisk
>          Issue Type: Improvement
>      Security Level: None
>          Components: Functions/func_curl
>    Affects Versions: 10.2.1
>            Reporter: Alex
>            Assignee: Rusty Newton
>            Severity: Minor
>         Attachments: cdr_curl.c, cdr_curl.conf
>
>
> A Module to Submit CDR via Curl to a webservice end point.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira



More information about the asterisk-bugs mailing list