[FFmpeg-devel] [PATCH] Use more appropriate types when decoding vorbisfloats
Måns Rullgård
mans
Mon Jul 26 11:48:34 CEST 2010
Rafa?l Carr? <rafael.carre at gmail.com> writes:
> On Sun, 25 Jul 2010 16:02:31 +0200
> Michael Niedermayer <michaelni at gmx.at> wrote:
>
>> On Sun, Jul 25, 2010 at 03:09:14PM +0200, Rafa?l Carr? wrote:
>
>> > diff --git a/tools/patcheck b/tools/patcheck
>> > index de20500..cae25ab 100755
>> > --- a/tools/patcheck
>> > +++ b/tools/patcheck
>> > @@ -97,6 +97,7 @@ hiegrep ':\+[^}#]*else' 'missing } prior to else'
>> > $* #FIXME this should print the previous statement maybe
>> > hiegrep ':\+ *{ *$' '{ should be on the same line as the related
>> > previous statement' $*
>> > +hiegrep 'int.*[^0-9a-zA-Z_][0-9]+[Uu]?[Ll]?[Ll][^0-9a-zA-Z_]'
>> > 'possibly mixing int variable and long constant' $*
>>
>> why int?
>
> i'm trying to match use of (unsigned) (long) long constants in
> expression involving int, such as:
>
> int i = 1L;
> int j = i * 345UL;
> random_type k = (int)var & 0x12345LL;
> long internal_buffer = 1L; // false positive
Use grep -E '\<int\>...' to avoid matching partial words. In fact, the
'hiegrep' name suggests it's already using EREs.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list