[asterisk-commits] oej: trunk r89557 - /trunk/channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: oej
Date: Sun Nov 25 06:18:35 2007
New Revision: 89557

URL: http://svn.digium.com/view/asterisk?view=rev&rev=89557
Log:
More doxygen changes

Modified:
    trunk/channels/chan_sip.c

Modified: trunk/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_sip.c?view=diff&rev=89557&r1=89556&r2=89557
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Sun Nov 25 06:18:35 2007
@@ -261,6 +261,11 @@
 	MWI_NOTIFICATION
 };
 
+/*! \brief Subscription types that we support. We support
+   - dialoginfo updates (really device status, not dialog info as was the original intent of the standard)
+   - SIMPLE presence used for device status
+   - Voicemail notification subscriptions
+*/
 static const struct cfsubscription_types {
 	enum subscriptiontype type;
 	const char * const event;
@@ -495,7 +500,10 @@
 };
 
 
-/*! \brief SIP Methods we support */
+/*! \brief SIP Methods we support 
+	\todo This string should be set dynamically. We only support REFER and SUBSCRIBE is we have
+	allowsubscribe and allowrefer on in sip.conf.
+*/
 #define ALLOWED_METHODS "INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY"
 
 /*! \brief SIP Extensions we support */
@@ -510,8 +518,9 @@
  * we do not support this feature at the moment.
  */
 
-/* Default values, set and reset in reload_config before reading configuration */
-/* These are default values in the source. There are other recommended values in the
+/*! \brief Default values, set and reset in reload_config before reading configuration 
+
+   These are default values in the source. There are other recommended values in the
    sip.conf.sample for new installations. These may differ to keep backwards compatibility,
    yet encouraging new behaviour on new installations 
  */