[FFmpeg-devel] [PATCH 02/11] avdevice/pulse_audio_common: add av_warn_unused_result

Ganesh Ajjanagadde gajjanag at mit.edu
Sat Oct 17 18:45:14 CEST 2015


On Sat, Oct 17, 2015 at 12:36 PM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
> On Thu, Oct 15, 2015 at 10:22:16PM -0400, Ganesh Ajjanagadde wrote:
>> This does not trigger any warnings, but adds robustness.
>>
>> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
>> ---
>>  libavdevice/pulse_audio_common.h | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/libavdevice/pulse_audio_common.h b/libavdevice/pulse_audio_common.h
>> index 02534f7..902795e 100644
>> --- a/libavdevice/pulse_audio_common.h
>> +++ b/libavdevice/pulse_audio_common.h
>> @@ -28,8 +28,10 @@
>>
>>  pa_sample_format_t ff_codec_id_to_pulse_format(enum AVCodecID codec_id);
>>
>> +av_warn_unused_result
>>  int ff_pulse_audio_get_devices(AVDeviceInfoList *devices, const char *server, int output);
>>
>> +av_warn_unused_result
>>  int ff_pulse_audio_connect_context(pa_mainloop **pa_ml, pa_context **pa_ctx,
>>                                     const char *server, const char *description);
>
> iam not the maintainer of this but the changes look reasonable

will wait for 48 hours, then push if no one responds.

>
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Democracy is the form of government in which you can choose your dictator
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list