From patchwork Tue Mar 20 12:16:08 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gagandeep Singh X-Patchwork-Id: 8057 Delivered-To: ffmpegpatchwork@gmail.com Received: by 10.2.1.70 with SMTP id c67csp3812487jad; Tue, 20 Mar 2018 05:16:20 -0700 (PDT) X-Google-Smtp-Source: AG47ELv2SUW66Ia0xttbUuEc3oO8tP15yR7+AQhG93zDao462NpKyCG1JrEpslCtTOe49SibQAcV X-Received: by 10.223.167.5 with SMTP id c5mr11770485wrd.188.1521548180148; Tue, 20 Mar 2018 05:16:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521548180; cv=none; d=google.com; s=arc-20160816; b=jUdn+MGZbHSk5wLnVgM0C1imqYx3r3Z/ZGY4zLKiZwJfpE/MPLSMEtXW1bikFaM4Kt 5Zj4xe5ztLfUxRABsp6COLDwWafeY6JeDzcM+dIlop8MCI1qe1YE4/kiNL7TBQb415Mx ZyhHX4G8V186m8vR8wJ3gj6d3I5hm3gJovGWYLKKpJ2iNj4Rtjh46mq7nxipEFUsORKJ hB2NGeFzMS3Fv+5jQaSSfYLKe9MDPIv3kEjxv6RY+Vkyc0HtVPZW8eshvgjaOZrZz9GO +NVtje+fwoLO5IdpKGjST1qqygSi9dBGK1dGx4AA6CPQTGNgjGAMmXfsb7I8jwW+gQSY 4QtQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:reply-to:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:subject:to:message-id:date:from:references:in-reply-to :mime-version:dkim-signature:delivered-to:arc-authentication-results; bh=bJB0dA3hhyDi4KqAb+06QGSrP7/7RC98UqFn3+LFeUU=; b=fmu41WuRQGM0h9U5MR1e7QL8kCTcjrEtiRSM86cVH6Rl9xN1dS+b8KUUwY0Z4sVrOj 1GvQZNKJ8jbNRQW66Mjx+0TIwO1FOj/pkvYlYQufebonwj6c4Zpsj2np/h9xsHW/z61E CxKqkX1QM9BYQacyMRAU5kFbKpf79m9TwT4n3qZDD4pN+jDDN5gryiolvowx7L4Fw2zy iGg4Kmb+r50u5gepapcC/7UVJrJKDq528ygwdtqEkyT0CRBUl2j/QG6r3GxVXPU8WO66 XN4d4Tm5TPKleBHGuZy7jO19QF/ifbK15PwupIEzLBVUZOxRXxArCFupFEZZSXVkZ+CR ByIg== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20161025 header.b=JoZ5w+ql; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id f17si1323308wri.194.2018.03.20.05.16.18; Tue, 20 Mar 2018 05:16:20 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20161025 header.b=JoZ5w+ql; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id E795968A2FA; Tue, 20 Mar 2018 14:16:02 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-it0-f48.google.com (mail-it0-f48.google.com [209.85.214.48]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 94F0F680AA5 for ; Tue, 20 Mar 2018 14:15:56 +0200 (EET) Received: by mail-it0-f48.google.com with SMTP id d13-v6so2153116itf.0 for ; Tue, 20 Mar 2018 05:16:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to; bh=z7gRkllW3g+aQXuFAJQjVTZsQcrKW3e6caPmdeQ+2i8=; b=JoZ5w+qldKbcKIQTz9TMporOQMoadGJZky7pTgysmdK0TUFeSv8RWitg6poAJfuHVc i9zpsF0rmhanPjvwW+kFB8cPOT5R7IDilz4KtExBslJUAnlhYnAg5WeKf431rmQHovYh CF2C7HqklnGJtpbvmqScpfACX1qXkKVxKht3ZrQeknWM7jkQmqjpBPLt2hzD1gbWNsBF GWQuMsYQi3fBebncodKl+WF+QwaQv0BjSrqMe8FlIjfigiu7zNuLozz1Ue+SabfnFJCM BVOjxQuhp+URzHZytxNTzcq+L9f3hbtp2Ag9o/IiZoYJeH+UdvLbG4npNPHuObWSQpvd Z6Xg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to; bh=z7gRkllW3g+aQXuFAJQjVTZsQcrKW3e6caPmdeQ+2i8=; b=SfVO2j1z2HBOhwTQiHxXj6y89WSYQAqKeKtxthsCZEeVxGPpjCptx9NaOSqYp8kpul 70xyWtu8G30NBYV+NhrsHCmUeNHTttqaAUaLuJPlZkcJijbRvqr9UiFBnYy8yoIMQNJR a0mgqGO1Y1do5zGbOsacY8SwALqqqJ6VKUaASQIJ0/bM5R6Ez+tBcNTHQ94PcCR6BxGf um9TmJoHKeWG45meAMyEWkOlH/g7MDnm2riUD6P545/fOb/kKLa60CBsJozdK74d/nZR d7mPJVoTLM7Jmkjxxw5RyxEBN9OyJwu4sGc6KvYTeSLmAK3lrU133Broc1i3X+4qERrZ /bOw== X-Gm-Message-State: AElRT7F2+5LZNLOGLrOqSg2L/6c4TalQALpMwT8Ah/Z6XY0RgDgrGUvH XeNKiRJsW/cyhU89UIrHb424+J94b0FHSZD2CeW6eA== X-Received: by 2002:a24:a54b:: with SMTP id w11-v6mr2531463iti.136.1521548168825; Tue, 20 Mar 2018 05:16:08 -0700 (PDT) MIME-Version: 1.0 Received: by 10.107.8.170 with HTTP; Tue, 20 Mar 2018 05:16:08 -0700 (PDT) In-Reply-To: References: <20180320071346.21636-1-deepgagan231197@gmail.com> From: Gagandeep Singh Date: Tue, 20 Mar 2018 17:46:08 +0530 Message-ID: To: FFmpeg development discussions and patches X-Content-Filtered-By: Mailman/MimeDel 2.1.20 Subject: Re: [FFmpeg-devel] [PATCH] lavc/cfhd:fixed alpha channel decoding in gbrap12 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" On Tue, 20 Mar 2018, 16:31 Kieran Kunhya, wrote: > > > > ticket #6265, oops forgot this > > > > Can you confirm this makes the png in the ticket look as expected? > I was expecting the companding curve to also change for alpha. > > Kieran > preserving the limits (max and min) --- libavcodec/cfhd.c | 2 ++ 1 file changed, 2 insertions(+) } diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index d64acab8d4..a950dc0f28 100644 --- a/libavcodec/cfhd.c +++ b/libavcodec/cfhd.c @@ -103,6 +103,8 @@ static inline void process_alpha(int16_t *alpha, int width) channel <<= 8; channel -= 128; channel /= 223; + channel = FFMIN(channel, (1 << 12) - 1); + channel = FFMAX(channel, 0); alpha[i] = channel; }