Andreas Rheinhardt (12020-04-16):
> lcm is now unused, so should be removed with this commit.
> (I'm not commenting on the actual merits of the patch.)

Indeed. I updated the patch, it now contains an extra hunk:

@@ -117,7 +117,6 @@ static void framesync_sync_level_update(FFFrameSync *fs)
 int ff_framesync_configure(FFFrameSync *fs)
 {
     unsigned i;
-    int64_t gcd, lcm;
 
     if (!fs->opt_repeatlast || fs->opt_eof_action == EOF_ACTION_PASS) {
         fs->opt_repeatlast = 0;

And I think I can dispense with re-sending it.

> PS: Thanks to Andriy for adding the functionality to patchwork to check
> whether a patch introduces new compiler warnings. This has been found
> this way.

It is useful indeed. But I should have spotted it. Thanks.

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to