diff mbox

[FFmpeg-devel,01/12] bitpacked: Remove dead store

Message ID 20170706182842.7785-2-derek.buitenhuis@gmail.com
State Accepted
Commit b603ef0870f3c7f10cdea6d8b16cb0115b0775db
Headers show

Commit Message

Derek Buitenhuis July 6, 2017, 6:28 p.m. UTC 
Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
---
 libavcodec/bitpacked.c | 4 ----
 1 file changed, 4 deletions(-)

Comments

Rostislav Pehlivanov July 6, 2017, 8:58 p.m. UTC | #1 
On 6 July 2017 at 19:28, Derek Buitenhuis <derek.buitenhuis@gmail.com>
wrote:

> Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
> ---
>  libavcodec/bitpacked.c | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/libavcodec/bitpacked.c b/libavcodec/bitpacked.c
> index 12174d6007..f0b417d595 100644
> --- a/libavcodec/bitpacked.c
> +++ b/libavcodec/bitpacked.c
> @@ -68,10 +68,6 @@ static int bitpacked_decode_yuv422p10(AVCodecContext
> *avctx, AVFrame *frame,
>      if (ret < 0)
>          return ret;
>
> -    y = (uint16_t*)frame->data[0];
> -    u = (uint16_t*)frame->data[1];
> -    v = (uint16_t*)frame->data[2];
> -
>      if (frame_size > packet_size)
>          return AVERROR_INVALIDDATA;
>
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

LGTM
diff mbox

Patch

diff --git a/libavcodec/bitpacked.c b/libavcodec/bitpacked.c
index 12174d6007..f0b417d595 100644
--- a/libavcodec/bitpacked.c
+++ b/libavcodec/bitpacked.c
@@ -68,10 +68,6 @@  static int bitpacked_decode_yuv422p10(AVCodecContext *avctx, AVFrame *frame,
     if (ret < 0)
         return ret;
 
-    y = (uint16_t*)frame->data[0];
-    u = (uint16_t*)frame->data[1];
-    v = (uint16_t*)frame->data[2];
-
     if (frame_size > packet_size)
         return AVERROR_INVALIDDATA;