On Fri, Aug 01, 2014 at 09:17:53PM +0200, Reimar Döffinger wrote: > On Fri, Aug 01, 2014 at 04:28:54PM +0200, Diego Biurrun wrote: > > ffmpeg | branch: master | Diego Biurrun <[email protected]> | Tue Jul 29 > > 08:43:40 2014 -0700| [5735552f1f17ea01dcbc99b08f54b5bf52176a8f] | > > committer: Diego Biurrun > > > > pngenc: Drop pointless pointer cast in png_write_row() > > > > > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=5735552f1f17ea01dcbc99b08f54b5bf52176a8f > > --- > > > > libavcodec/pngenc.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c > > index a349ee8..3d11f37 100644 > > --- a/libavcodec/pngenc.c > > +++ b/libavcodec/pngenc.c > > @@ -213,7 +213,7 @@ static int png_write_row(PNGEncContext *s, const > > uint8_t *data, int size) > > int ret; > > > > s->zstream.avail_in = size; > > - s->zstream.next_in = (uint8_t *)data; > > + s->zstream.next_in = data; > > Huh? Did they seriously fix their API??? > Either way, next_in used to be non-const, which was the reason for the > cast... > Though I believe we get a warning either way, just a different one, so I > guess you could discuss how much sense the cast had.
It seems they did in a tricky way: a0ce85ac7de098d3f9b53b51b77a09bad700a011 -- Clément B.
pgprnpigWW7Vl.pgp
Description: PGP signature
_______________________________________________ ffmpeg-cvslog mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog
