From d18c3664b53b543a34da82e2229129cacf8a11ff Mon Sep 17 00:00:00 2001 From: "Robin H. Johnson" Date: Sat, 17 Nov 2012 23:31:17 +0000 Subject: [PATCH] rtp: rename modarg destination to destination_ip The module argument 'source' already has special meaning as the pa_source, however, the argument 'destination' expects an IP address. Prior to introducing a source IP modarg for the source IP address, rename the 'destination' argument to 'destination_ip'. Include compatibility support for old RTP users so they don't need to change their module usage immediately. (changes in v2: minor formatting fixes, s/dstip/destination_ip) Signed-off-by: Robin H. Johnson --- src/modules/rtp/module-rtp-send.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/modules/rtp/module-rtp-send.c b/src/modules/rtp/module-rtp-send.c index 95cb439cb..00cc6c919 100644 --- a/src/modules/rtp/module-rtp-send.c +++ b/src/modules/rtp/module-rtp-send.c @@ -63,7 +63,7 @@ PA_MODULE_USAGE( "format= " "channels= " "rate= " - "destination= " + "destination_ip= " "port= " "mtu= " "loop= " @@ -73,7 +73,7 @@ PA_MODULE_USAGE( #define DEFAULT_PORT 46000 #define DEFAULT_TTL 1 #define SAP_PORT 9875 -#define DEFAULT_DESTINATION "224.0.0.56" +#define DEFAULT_DESTINATION_IP "224.0.0.56" #define MEMBLOCKQ_MAXLENGTH (1024*170) #define DEFAULT_MTU 1280 #define SAP_INTERVAL (5*PA_USEC_PER_SEC) @@ -83,7 +83,8 @@ static const char* const valid_modargs[] = { "format", "channels", "rate", - "destination", + "destination", /* Compatbility */ + "destination_ip", "port", "mtu" , "loop", @@ -241,7 +242,9 @@ int pa__init(pa_module*m) { goto fail; } - dst_addr = pa_modargs_get_value(ma, "destination", DEFAULT_DESTINATION); + dst_addr = pa_modargs_get_value(ma, "destination", NULL); + if (dst_addr == NULL) + dst_addr = pa_modargs_get_value(ma, "destination_ip", DEFAULT_DESTINATION_IP); if (inet_pton(AF_INET, dst_addr, &dst_sa4.sin_addr) > 0) { dst_sa4.sin_family = af = AF_INET;