Hi Jeevaka,

On 11/26/2010 08:57 AM, Jeevaka Badrappan wrote:
> ---
>  src/call-volume.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/src/call-volume.c b/src/call-volume.c
> index dd1be92..0accdf2 100644
> --- a/src/call-volume.c
> +++ b/src/call-volume.c
> @@ -114,7 +114,6 @@ static DBusMessage *cv_get_properties(DBusConnection *conn,
>  	dbus_bool_t muted;
>  
>  	reply = dbus_message_new_method_return(msg);
> -
>  	if (!reply)
>  		return NULL;
>  

Style wise this is actually OK, but if you really want to be
style-pedantic :) then this should be:

reply = ...
if (reply == NULL)
	return NULL

Regards,
-Denis