[FFmpeg-devel] dshow patches

Roger Pack rogerdpack2 at gmail.com
Tue Mar 25 17:30:35 CET 2014


And one more, adding a log message (and continue) if unable to set
audio buffer on a pin.

On 3/21/14, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Fri, Mar 21, 2014 at 06:01:35PM -0600, Roger Pack wrote:
>> These are pretty trivial, but I think useful.
>> The first causes it to *not* drop audio packets as frequently (which
>> I've seen people complain about before).  The second is to have better
>> default latency for mic's, as per msdn's suggested default.
>> Thank you.
>> -roger-
>
>>  dshow.c |   18 +++++++++---------
>>  1 file changed, 9 insertions(+), 9 deletions(-)
>> 62f3a9f206df4d98e9986c14ff0aade65f0919f6
>> 0001-dshow-use-distinct-buffers-per-stream.-Basically-som.patch
>> From e6ff24d390f085e5daee79e10a074c1ef0878636 Mon Sep 17 00:00:00 2001
>> From: rogerdpack <rogerpack2005 at gmail.com>
>> Date: Fri, 21 Mar 2014 16:27:01 -0600
>> Subject: [PATCH 1/4] dshow: use distinct buffers per stream.  Basically
>>  sometimes if you start dropping video packets, you also
>>  drop audio packets.  Now they each have separate
>>  buffers to avoid this unexpected behavior.
>
> applied
>
> [...]
>>
>> diff --git a/libavdevice/dshow.c b/libavdevice/dshow.c
>> index 51693be..5d600c1 100644
>> --- a/libavdevice/dshow.c
>> +++ b/libavdevice/dshow.c
>> @@ -1073,7 +1073,7 @@ static const AVOption options[] = {
>>      { "false", "", 0, AV_OPT_TYPE_CONST, {.i64=0}, 0, 0, DEC,
>> "list_options" },
>>      { "video_device_number", "set video device number for devices with
>> same name (starts at 0)", OFFSET(video_device_number), AV_OPT_TYPE_INT,
>> {.i64 = 0}, 0, INT_MAX, DEC },
>>      { "audio_device_number", "set audio device number for devices with
>> same name (starts at 0)", OFFSET(audio_device_number), AV_OPT_TYPE_INT,
>> {.i64 = 0}, 0, INT_MAX, DEC },
>> -    { "audio_buffer_size", "set audio device buffer latency size in
>> milliseconds (default is the device's default)",
>> OFFSET(audio_buffer_size), AV_OPT_TYPE_INT, {.i64 = 0}, 0, INT_MAX, DEC
>> },
>> +    { "audio_buffer_size", "set audio device buffer latency size in
>> milliseconds (default is 80ms)", OFFSET(audio_buffer_size),
>> AV_OPT_TYPE_INT, {.i64 = 0}, 80, INT_MAX, DEC },
>
> this looks like it chnages the minimum and not the default
>
> [...]
>
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> If you think the mosad wants you dead since a long time then you are either
> wrong or dead since a long time.
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-dshow-log-error-and-continue-if-unable-to-set-audio-.patch
Type: application/octet-stream
Size: 1075 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140325/14462df7/attachment.obj>


More information about the ffmpeg-devel mailing list