[asterisk-dev] [Code Review] dial by name in chan_dahdi
Tzafrir Cohen
tzafrir.cohen at xorcom.com
Wed Mar 3 05:20:47 CST 2010
Hi,
Thanks for the review. Looking into your connects right now.
Some more general nits:
On Tue, Mar 02, 2010 at 07:24:05PM -0000, rmudgett at digium.com wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/535/#review1621
> -----------------------------------------------------------
> /trunk/channels/chan_dahdi.c
> <https://reviewboard.asterisk.org/r/535/#comment3627>
>
> Please add a doxygen function header comment. You can use the dahdi_new_pri_nobch_channel() comment header as a template.
* Do I really have to tell doxygen that a static function is internal?
* Do I need to specify version (\since 1.8) for such an internal
function?
--
Tzafrir Cohen
icq#16849755 jabber:tzafrir.cohen at xorcom.com
+972-50-7952406 mailto:tzafrir.cohen at xorcom.com
http://www.xorcom.com iax:guest at local.xorcom.com/tzafrir
More information about the asterisk-dev
mailing list