----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3812/#review12830 -----------------------------------------------------------
Ship it! Couple minor comment issues, otherwise looks good to go! trunk/include/asterisk/xmldoc.h <https://reviewboard.asterisk.org/r/3812/#comment23170> Missing '*' at the beginning. trunk/include/asterisk/xmldoc.h <https://reviewboard.asterisk.org/r/3812/#comment23171> Let's loose the '?' trunk/include/asterisk/xmldoc.h <https://reviewboard.asterisk.org/r/3812/#comment23172> Same. - Corey Farrell On July 22, 2014, 10:42 p.m., opticron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3812/ > ----------------------------------------------------------- > > (Updated July 22, 2014, 10:42 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > Allow for responses to AMI actions/commands to be documented properly in XML > and displayed via the CLI. Response events are documented exactly as standard > AMI events are documented. > > > Diffs > ----- > > trunk/main/xmldoc.c 419221 > trunk/main/manager_bridges.c 419221 > trunk/main/manager.c 419221 > trunk/main/config_options.c 419221 > trunk/include/asterisk/xmldoc.h 419221 > trunk/include/asterisk/manager.h 419221 > trunk/doc/appdocsxml.dtd 419221 > > Diff: https://reviewboard.asterisk.org/r/3812/diff/ > > > Testing > ------- > > The BridgeInfo AMI command was documented and tested as an example. > > > Thanks, > > opticron > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
