[pulseaudio-discuss] [PATCH] dbus: Remove a silly return statement.

Colin Guthrie gmane at colin.guthr.ie
Fri Mar 23 02:47:00 PDT 2012


'Twas brillig, and Tanu Kaskinen at 23/03/12 07:49 did gyre and gimble:
> On Fri, 2012-03-23 at 09:46 +0200, Tanu Kaskinen wrote:
>> ---
>>  src/modules/dbus/iface-device.c |    1 -
>>  1 files changed, 0 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/modules/dbus/iface-device.c b/src/modules/dbus/iface-device.c
>> index df64d36..97ccd91 100644
>> --- a/src/modules/dbus/iface-device.c
>> +++ b/src/modules/dbus/iface-device.c
>> @@ -604,7 +604,6 @@ static void handle_get_latency(DBusConnection *conn, DBusMessage *msg, void *use
>>      else if (d->type == PA_DEVICE_TYPE_SOURCE && !(d->source->flags & PA_SOURCE_LATENCY))
>>          pa_dbus_send_error(conn, msg, PA_DBUS_ERROR_NO_SUCH_PROPERTY,
>>                             "Source %s doesn't support latency querying.", d->source->name);
>> -    return;
>>  
>>      latency = (d->type == PA_DEVICE_TYPE_SINK) ? pa_sink_get_latency(d->sink) : pa_source_get_latency(d->source);
> 
> Sorry, I'm clearly not thinking what I'm doing. Please ignore this
> patch.
> 

Perhaps the remainder of the code should be commented out or at lease a
"header" comment after the return?

It certainly looks in the code like it's not meant to be there....

(either that or each if statement should be inside {} with it's own
individual return statements?)

Col



-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited http://www.tribalogic.net/
Open Source:
  Mageia Contributor http://www.mageia.org/
  PulseAudio Hacker http://www.pulseaudio.org/
  Trac Hacker http://trac.edgewall.org/



More information about the pulseaudio-discuss mailing list