From patchwork Tue Mar 26 06:58:34 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lauri Kasanen X-Patchwork-Id: 12449 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 35E0F447FCF for ; Tue, 26 Mar 2019 08:55:58 +0200 (EET) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 0DFCF68A0C1; Tue, 26 Mar 2019 08:55:58 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mout.gmx.net (mout.gmx.net [212.227.17.22]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 55221689DD1 for ; Tue, 26 Mar 2019 08:55:51 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1553583350; bh=Q8pm8Yn/rj3lmI8WyMNXSvOkevfph5ylXC4FACYxeac=; h=X-UI-Sender-Class:Date:From:To:Subject; b=PQT2rhVB8PZPt5QO1ODe5fH8OR2TzHrPoHEz14vcK+TNqabnN8MiF4PYqSB6munsC kIQdmjAoua3CPPdhzzMhJuFtQ0s+L6AbtXPrbcFhiObGGJ9rfCN4P+17W/PQp2Dtud M78YkkgJG4vVTUOWbO73s3K94kJh2v0tAp6NvdIQ= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Received: from Valinor ([84.250.81.169]) by mail.gmx.com (mrgmx103 [212.227.17.174]) with ESMTPSA (Nemesis) id 0MVedf-1hTp4T222B-00Yx8d for ; Tue, 26 Mar 2019 07:55:50 +0100 Date: Tue, 26 Mar 2019 08:58:34 +0200 From: Lauri Kasanen To: ffmpeg-devel@ffmpeg.org Message-Id: <20190326085834.f59d563f90d83535a8c59684@gmx.com> X-Mailer: Sylpheed 3.5.0 (GTK+ 2.18.6; x86_64-unknown-linux-gnu) Mime-Version: 1.0 X-Provags-ID: V03:K1:Y2tOj3Bddn3Hk4canLSotIHiqG3Eoguh70py0CybMeGzY/+Kduo 1tkEFeO58z49yz6qXf8AUKSeIQ/ApI9NzIB1PxDbipjEDvh3CaMNHwbQJHOupk9tJJSfTEy 9LX+SKJv2frsEcTRv4uBxsoxBesTV0inw+nqzJesi9NoaOQ4svrYtE/kkL7+fL9HVdnYNOM c4EQ4xZGz1atR0sJGH41g== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V03:K0:oM8Kk2Uuj7g=:EnIpt4uzZa+KyUgGY3ysbX HHbM7/nOnaYpg0O2x0iFGrjlN3GxIgBFN3cCGGvp9PNaxbM2JmrNwqWCo+cGql/g1G42VNE8z oUb00zDxd4TQ3POHzrnA2RK16Ct8S/EYbM/JH0p1MZwfyffkqlhL7h+tKBFRqqC4jrEJyHiNG XDOc2Fbm2zNSLZQ4YASRq4dc/rZumMs8ZAOIWoyBD0HcD5fAZH/qJKQfuzpHBtSjdMPc45Stk 9WW3PITXhcIh6mNA4q1srus1Oi6CLAevr90sOPYO9+CODBslNkuSRQSQM/ST8ao5dDN4ixjmK o5lH2DjbONRxD5hEVQzXITmtBWhmg6UumK6Fw9fdIKJNOUikgJySdEWM/Y8S9Zwp2KHlw2VBz nsmaxpPJu50ROpxZWLbmBgPwUMCCHmzwX0O5hnFuVbNlOyfWpLefe3bUKLLuCVYg1Y7xGwgTv QdO0B0S9jaWsnfSI86pJXNU91btCiSM9Ll8Tv89U4rUYAgPhnDgRqKeIBaFoP5cqQF56JCFkt utvLfwr1jg4Ir7faZJQJ3DRHM6HjeeoIcRBxNy9dXYdZaDCZrljn9MksfPvHE229on+JgeIBK h01TSU6HsscYAbSfT1d5K8lqAhF3kzJQqPy3WIkWSwosnYuAh48AFHqdI7TNCW9Npygmf8VHo SoXkRNToXZh2YO1WpZqkA5XB+VHLDuZMbgW2Fo0OIlQDu7n2Qp3UQv+i3S9LwMPElVDRfqU66 mGwCa6ylfA8Lo2Lf4wakfgb8OF0bQ+p+qpZUiigX1V1CzeQ3f5OncoyNvKrYO5QU/nch4jrex ThCHSEegQfFYfn099bd6HmJ9cuPFtMY9yhL2pM56rX+H8OHnGGcP9F5a/CLJtyqhozB4mRbnZ kURV5l3cps08MLi8QfKM+jqhDfCPvWJiDbD06Zt0gqD9TiYaeevnns3aYGMUwF Subject: [FFmpeg-devel] [PATCH v2] swscale: Remove duplicated code X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" In this function, the exact same clamping happens both in the if and unconditionally. Signed-off-by: Lauri Kasanen --- libswscale/output.c | 10 ---------- 1 file changed, 10 deletions(-) v2: Remove the unconditional instead of the if'd clipping. I'll leave changing the bit pattern to others, there's so many funcs using 0x100. -- 2.6.2 diff --git a/libswscale/output.c b/libswscale/output.c index d7c53e6..d3401f0 100644 --- a/libswscale/output.c +++ b/libswscale/output.c @@ -853,11 +853,6 @@ yuv2422_1_c_template(SwsContext *c, const int16_t *buf0, V = av_clip_uint8(V); } - Y1 = av_clip_uint8(Y1); - Y2 = av_clip_uint8(Y2); - U = av_clip_uint8(U); - V = av_clip_uint8(V); - output_pixels(i * 4, Y1, U, Y2, V); } } else { @@ -875,11 +870,6 @@ yuv2422_1_c_template(SwsContext *c, const int16_t *buf0, V = av_clip_uint8(V); } - Y1 = av_clip_uint8(Y1); - Y2 = av_clip_uint8(Y2); - U = av_clip_uint8(U); - V = av_clip_uint8(V); - output_pixels(i * 4, Y1, U, Y2, V); } }