On Fri, Mar 23, 2012 at 12:23 PM, Vitor Sessak <[email protected]> wrote:
> On 03/23/2012 07:17 AM, Kostya Shishkov wrote:
>>
>> On Thu, Mar 22, 2012 at 01:46:13PM -0700, Ronald S. Bultje wrote:
>>>
>>> Hi,
>>>
>>> On Thu, Mar 22, 2012 at 1:08 PM, Kostya Shishkov
>>> <[email protected]>  wrote:
>>>>
>>>> On Thu, Mar 22, 2012 at 12:55:38PM -0700, Ronald S. Bultje wrote:
>>>>>
>>>>> From: "Ronald S. Bultje"<[email protected]>
>>>>>
>>>>> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
>>>>> ---
>>>>>  libavcodec/ralfdata.h |   12 ++++++------
>>>>>  1 file changed, 6 insertions(+), 6 deletions(-)
>>>>
>>>>
>>>> It's senseless IMO - especially with current bitstream reader.
>>>
>>>
>>> The current bitstream reader does overread, but by a guaranteed amount
>>> of less than FF_INPUT_BUFFER_PADDING_BYTES bytes.
>>
>>
>> OK, but don't forget a proper #include for FF_INPUT_BUFFER_PADDING_BYTES,
>> or it will fail make checkheaders and make Diego unhappier.
>
>
> Wouldn't this make the huge tables even bigger? Another solution would be
> not to use the bitstream reader...
>

It looks like these tables are read one half byte at a time so I agree
with Vitor.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to