Quoting Leo Izen (2024-02-04 08:20:55)
> 
> If we're changing this block of code anyway, could we possibly replace 
> this line:
> 
>              if ((ret = ff_nut_add_sp(nut, nut->last_syncpoint_pos, 0 
> /*unused*/, pkt->dts)) < 0)
>                  goto fail;
> 
> With something like this?
> 
> ret = ff_nut_add_sp(nut, nut->last_syncpoint_pos, 0 /*unused*/, pkt->dts);
> if (ret < 0)
>      goto fail;
> 
> It's a bit cleaner.

I much prefer not to mix whitespace and other changes, it breaks
color-moved, ignore-all-space and similar.

-- 
Anton Khirnov
_______________________________________________
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