[asterisk-dev] [asterisk-commits] kpfleming: trunk r135302 - /trunk/channels/chan_skinny.c

[Michiel van Baak]

On 13:21, Sat 02 Aug 08, SVN commits to the Asterisk project wrote:
> Author: kpfleming
> Date: Sat Aug  2 08:21:16 2008
> New Revision: 135302
> 
> URL: http://svn.digium.com/view/asterisk?view=rev&rev=135302
> Log:
> --enable-dev-mode is your friend :-)

I did use './configure --enable-dev-mode'

I just doublechecked and I dont get a warning here.
I do see the warning on buildbot's output though.

Can this be because of gcc versions ?

% gcc -v
Reading specs from /usr/lib/gcc-lib/amd64-unknown-openbsd4.4/3.3.5/specs
Configured with: 
Thread model: single
gcc version 3.3.5 (propolice)

% gmake -v
GNU Make 3.81
Copyright (C) 2006  Free Software Foundation, Inc.
This is free software; see the source for copying conditions.
There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A
PARTICULAR PURPOSE.

This program built for x86_64-unknown-openbsd4.4

> 
> Modified:
>     trunk/channels/chan_skinny.c
> 
> Modified: trunk/channels/chan_skinny.c
> URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_skinny.c?view=diff&rev=135302&r1=135301&r2=135302
> ==============================================================================
> --- trunk/channels/chan_skinny.c (original)
> +++ trunk/channels/chan_skinny.c Sat Aug  2 08:21:16 2008
> @@ -4166,7 +4166,7 @@
>  
>  static int handle_register_message(struct skinny_req *req, struct skinnysession *s)
>  {
> -	struct skinny_device *d;
> +	struct skinny_device *d = s->device;
>  	char name[16];
>  	int res;
>  
> @@ -4185,8 +4185,6 @@
>  	}
>  	ast_verb(3, "Device '%s' successfully registered\n", name);
>  	
> -	d = s->device;
> -
>  	if (!(req = req_alloc(sizeof(struct register_ack_message), REGISTER_ACK_MESSAGE)))
>  		return -1;
>  
> 
> 
> _______________________________________________
> --Bandwidth and Colocation Provided by http://www.api-digital.com--
> 
> asterisk-commits mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-commits

-- 

Michiel van Baak
michiel at vanbaak.eu
http://michiel.vanbaak.eu
GnuPG key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x71C946BD

"Why is it drug addicts and computer aficionados are both called users?"