[Ffmpeg-cvslog] r7585 - trunk/libavcodec/h264.c

[Michael Niedermayer]

Hi

On Fri, Jan 19, 2007 at 01:21:19PM +0100, Guillaume POIRIER wrote:
> Hi,
> 
> On 1/19/07, Michael Niedermayer <michaelni at gmx.at> wrote:
> >On Fri, Jan 19, 2007 at 10:37:08AM +0100, gpoirier wrote:
> >> --- trunk/libavcodec/h264.c   (original)
> >> +++ trunk/libavcodec/h264.c   Fri Jan 19 10:37:04 2007
> >[...]
> >> @@ -4521,8 +4532,8 @@
> >>          return -1;
> >>      }
> >>
> >> -    if(h->dequant_coeff_pps != pps_id){
> >> -        h->dequant_coeff_pps = pps_id;
> >> +    if(h->dequant_coeff_pps != (int)pps_id){
> >> +        h->dequant_coeff_pps = (int)pps_id;
> >
> >argh ...
> 
> Sorry, I had not understand that this hunk wasn't ok when I read your 
> answer.
> Should I revert that very hunk? I guess the answer is yes, but I
> prefer to ask before doing anything.

hmm as you ask ... well ... why not implement paff support? and we
forget about this little mistake ;))))

failing that ... yes iam happy too if you reverse this hunk

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070119/fe6a15c7/attachment.pgp>