diff options
author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2005-07-27 02:17:06 +0000 |
---|---|---|
committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2005-07-27 02:17:06 +0000 |
commit | fd04c2f861047cf52fcdb66e6aa48cc0a154a043 (patch) | |
tree | affbda3c786c373acd8ce06fceee939e63cee98b /channels | |
parent | 0052c91419da1601916bc765d16646a610a8b854 (diff) |
and now we know why #define-ing constants and trying to keep them in sequence is a bad idea...
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@6232 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels')
-rwxr-xr-x | channels/chan_sip.c | 47 |
1 files changed, 29 insertions, 18 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index bedf67537..fbf76cfb3 100755 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -123,31 +123,42 @@ static const char channeltype[] = "SIP"; static const char config[] = "sip.conf"; static const char notify_config[] = "sip_notify.conf"; -#define SIP_RESPONSE 1 -#define SIP_REGISTER 2 -#define SIP_OPTIONS 3 -#define SIP_NOTIFY 4 -#define SIP_INVITE 5 -#define SIP_ACK 6 -#define SIP_PRACK 7 -#define SIP_BYE 8 -#define SIP_REFER 9 -#define SIP_SUBSCRIBE 10 -#define SIP_MESSAGE 11 -#define SIP_UPDATE 12 -#define SIP_INFO 13 -#define SIP_CANCEL 14 -#define SIP_PUBLISH 15 - #define RTP 1 #define NO_RTP 0 +/* Do _NOT_ make any changes to this enum, or the array following it; + if you think you are doing the right thing, you are probably + not doing the right thing. If you think there are changes + needed, get someone else to review them first _before_ + submitting a patch. If these two lists do not match properly + bad things will happen. +*/ + +enum sipmethod { + SIP_UNKNOWN, + SIP_RESPONSE, + SIP_REGISTER, + SIP_OPTIONS, + SIP_NOTIFY, + SIP_INVITE, + SIP_ACK, + SIP_PRACK, + SIP_BYE, + SIP_REFER, + SIP_SUBSCRIBE, + SIP_MESSAGE, + SIP_UPDATE, + SIP_INFO, + SIP_CANCEL, + SIP_PUBLISH, +} sip_method_list; + static const struct cfsip_methods { - int id; + enum sipmethod id; int need_rtp; /* when this is the 'primary' use for a pvt structure, does it need RTP? */ char * const text; } sip_methods[] = { - { 0, RTP, "-UNKNOWN-" }, + { SIP_UNKNOWN, RTP, "-UNKNOWN-" }, { SIP_RESPONSE, NO_RTP, "SIP/2.0" }, { SIP_REGISTER, NO_RTP, "REGISTER" }, { SIP_OPTIONS, NO_RTP, "OPTIONS" }, |