diff mbox

[FFmpeg-devel,1/3] avformat/apc: Remove unnecessary resetting of flags

Message ID 20191210014145.3297-1-andreas.rheinhardt@gmail.com
State Accepted
Commit 91f775e0c533374f1c94e1a094d19d269b70e1e7
Headers show

Commit Message

Andreas Rheinhardt Dec. 10, 2019, 1:41 a.m. UTC
The packet a demuxer receives is freshly initialized, hence it is
unnecessary to reset any flags on them (as none are set), yet apc did
this.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
 libavformat/apc.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Michael Niedermayer Dec. 11, 2019, 3:53 p.m. UTC | #1
On Tue, Dec 10, 2019 at 02:41:43AM +0100, Andreas Rheinhardt wrote:
> The packet a demuxer receives is freshly initialized, hence it is
> unnecessary to reset any flags on them (as none are set), yet apc did
> this.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
> ---
>  libavformat/apc.c | 1 -
>  1 file changed, 1 deletion(-)

will apply patchset

thanks

[...]
diff mbox

Patch

diff --git a/libavformat/apc.c b/libavformat/apc.c
index 835d1b0f6e..571726affb 100644
--- a/libavformat/apc.c
+++ b/libavformat/apc.c
@@ -78,7 +78,6 @@  static int apc_read_packet(AVFormatContext *s, AVPacket *pkt)
 {
     if (av_get_packet(s->pb, pkt, MAX_READ_SIZE) <= 0)
         return AVERROR(EIO);
-    pkt->flags &= ~AV_PKT_FLAG_CORRUPT;
     pkt->stream_index = 0;
     return 0;
 }