From patchwork Mon May 27 06:22:12 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lance Wang X-Patchwork-Id: 13305 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 C7DD6448F42 for ; Mon, 27 May 2019 09:22:46 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id B3D6468A60E; Mon, 27 May 2019 09:22:46 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-pf1-f194.google.com (mail-pf1-f194.google.com [209.85.210.194]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 9FE566882CB for ; Mon, 27 May 2019 09:22:39 +0300 (EEST) Received: by mail-pf1-f194.google.com with SMTP id a23so8980823pff.4 for ; Sun, 26 May 2019 23:22:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=cHykf7syzFjWl3464DbBr93MmlAlsTnL8OipteTruEQ=; b=VF8QdGWR/fNlmgEBz1EzOCvXS+XnWAxOHh7cC6398DxY9hcC4stF5OnvsBf8fF5jmS Xs2cMgutS1v9i2wM5m6ywPRizqogtiommaloL6cqxH7K3BvLVmtdlJLy98FIxODg9iu8 3cWWe7padIPcXL9AoDvBLMq0h0sFitk7WmMRU3btfLeNCauMeWwOoYUbzoB4RdZx8aM7 pH16Tf+n2eQ2hKWsnRNunwTKl0pYry48/92tyBZdCpVKmzvDA4zyU591gSqXbBuLxGD4 NRvJkS88xu5o77+opoPq6SxuYZ62EbMa/ifj4yLl0VDFegHSXtx5WqijmSXrdebiGPvA LCSw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=cHykf7syzFjWl3464DbBr93MmlAlsTnL8OipteTruEQ=; b=lJ40fDrDmtD1Kwf97V0xhkapZiMPbgmhFh1vtaL0ddt0nhqeiRodOkSIk83EAwJk7l zfi/Qi0EiPvfB7z4zmLJOX/XwCVwIOS4R0Bf5nQ9bcAXqdMTEbz1pYU4tR53xKh0FmQI Ob0g98qLRsLOf146mYFwd9FTQ86BLkFdF8s/jcpUQnOP/ka7V59aaJ0jhTOnbldNG/L4 BS1ROpbOV1TtsOTDWWHCtJ+Nl3ylsyvtl451f2dK4kALvay3jp48I0hhJRDfv6l6jQ2b 7yAIzdx/eIwJCC42b1Os1gyjkUTtVL8CpKpeQETGrbaCULg5Qlgi+U/JRjNV7hrJTNAj 4Qjg== X-Gm-Message-State: APjAAAVHflp6PYz+iArL0/Z1ELqqBfJrQjnpTdZ+/1P1w+lcDj2hOxPf P962GLzy0fV0/hnDvLl6qPMnyn2C X-Google-Smtp-Source: APXvYqyfSInlS/Rr46IjMkUmq7c+Fd8lW2BIcj1Xq7qH0POOW6akf0MzJjMWdLxovdHiLrdGRs3Nmg== X-Received: by 2002:a62:6d41:: with SMTP id i62mr94193579pfc.227.1558938157140; Sun, 26 May 2019 23:22:37 -0700 (PDT) Received: from localhost.localdomain ([47.90.99.151]) by smtp.gmail.com with ESMTPSA id p16sm13750252pfq.153.2019.05.26.23.22.34 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Sun, 26 May 2019 23:22:36 -0700 (PDT) From: lance.lmwang@gmail.com To: ffmpeg-devel@ffmpeg.org Date: Mon, 27 May 2019 14:22:12 +0800 Message-Id: <20190527062212.96201-2-lance.lmwang@gmail.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190527062212.96201-1-lance.lmwang@gmail.com> References: <20190527062212.96201-1-lance.lmwang@gmail.com> MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 2/2] libavfilter/vf_blend.c: remove duplicate code with macro-defined function 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 Cc: onemda@gmail.com, Limin Wang Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" From: Limin Wang Signed-off-by: Limin Wang --- libavfilter/vf_blend.c | 188 +++++++++++------------------------------ 1 file changed, 48 insertions(+), 140 deletions(-) diff --git a/libavfilter/vf_blend.c b/libavfilter/vf_blend.c index 46e0518d26..cf71200773 100644 --- a/libavfilter/vf_blend.c +++ b/libavfilter/vf_blend.c @@ -555,156 +555,64 @@ static av_cold void uninit(AVFilterContext *ctx) av_expr_free(s->params[i].e); } +#define DEFINE_INIT_BLEND_FUNC(depth, nbits) \ +static av_cold void init_blend_func_##depth##_##nbits##bit(FilterParams *param) \ +{ \ + switch (param->mode) { \ + case BLEND_ADDITION: param->blend = blend_addition_##depth##bit; break; \ + case BLEND_GRAINMERGE: param->blend = blend_grainmerge_##depth##bit; break; \ + case BLEND_AND: param->blend = blend_and_##depth##bit; break; \ + case BLEND_AVERAGE: param->blend = blend_average_##depth##bit; break; \ + case BLEND_BURN: param->blend = blend_burn_##depth##bit; break; \ + case BLEND_DARKEN: param->blend = blend_darken_##depth##bit; break; \ + case BLEND_DIFFERENCE: param->blend = blend_difference_##depth##bit; break; \ + case BLEND_GRAINEXTRACT: param->blend = blend_grainextract_##depth##bit; break; \ + case BLEND_DIVIDE: param->blend = blend_divide_##depth##bit; break; \ + case BLEND_DODGE: param->blend = blend_dodge_##depth##bit; break; \ + case BLEND_EXCLUSION: param->blend = blend_exclusion_##depth##bit; break; \ + case BLEND_EXTREMITY: param->blend = blend_extremity_##depth##bit; break; \ + case BLEND_FREEZE: param->blend = blend_freeze_##depth##bit; break; \ + case BLEND_GLOW: param->blend = blend_glow_##depth##bit; break; \ + case BLEND_HARDLIGHT: param->blend = blend_hardlight_##depth##bit; break; \ + case BLEND_HARDMIX: param->blend = blend_hardmix_##depth##bit; break; \ + case BLEND_HEAT: param->blend = blend_heat_##depth##bit; break; \ + case BLEND_LIGHTEN: param->blend = blend_lighten_##depth##bit; break; \ + case BLEND_LINEARLIGHT:param->blend = blend_linearlight_##depth##bit;break; \ + case BLEND_MULTIPLY: param->blend = blend_multiply_##depth##bit; break; \ + case BLEND_MULTIPLY128:param->blend = blend_multiply128_##depth##bit;break; \ + case BLEND_NEGATION: param->blend = blend_negation_##depth##bit; break; \ + case BLEND_NORMAL: param->blend = blend_normal_##nbits##bit; break; \ + case BLEND_OR: param->blend = blend_or_##depth##bit; break; \ + case BLEND_OVERLAY: param->blend = blend_overlay_##depth##bit; break; \ + case BLEND_PHOENIX: param->blend = blend_phoenix_##depth##bit; break; \ + case BLEND_PINLIGHT: param->blend = blend_pinlight_##depth##bit; break; \ + case BLEND_REFLECT: param->blend = blend_reflect_##depth##bit; break; \ + case BLEND_SCREEN: param->blend = blend_screen_##depth##bit; break; \ + case BLEND_SOFTLIGHT: param->blend = blend_softlight_##depth##bit; break; \ + case BLEND_SUBTRACT: param->blend = blend_subtract_##depth##bit; break; \ + case BLEND_VIVIDLIGHT: param->blend = blend_vividlight_##depth##bit; break; \ + case BLEND_XOR: param->blend = blend_xor_##depth##bit; break; \ + } \ +} +DEFINE_INIT_BLEND_FUNC(8, 8); +DEFINE_INIT_BLEND_FUNC(10, 16); +DEFINE_INIT_BLEND_FUNC(12, 16); +DEFINE_INIT_BLEND_FUNC(16, 16); + void ff_blend_init(FilterParams *param, int depth) { switch (depth) { case 8: - switch (param->mode) { - case BLEND_ADDITION: param->blend = blend_addition_8bit; break; - case BLEND_GRAINMERGE: param->blend = blend_grainmerge_8bit; break; - case BLEND_AND: param->blend = blend_and_8bit; break; - case BLEND_AVERAGE: param->blend = blend_average_8bit; break; - case BLEND_BURN: param->blend = blend_burn_8bit; break; - case BLEND_DARKEN: param->blend = blend_darken_8bit; break; - case BLEND_DIFFERENCE: param->blend = blend_difference_8bit; break; - case BLEND_GRAINEXTRACT: param->blend = blend_grainextract_8bit; break; - case BLEND_DIVIDE: param->blend = blend_divide_8bit; break; - case BLEND_DODGE: param->blend = blend_dodge_8bit; break; - case BLEND_EXCLUSION: param->blend = blend_exclusion_8bit; break; - case BLEND_EXTREMITY: param->blend = blend_extremity_8bit; break; - case BLEND_FREEZE: param->blend = blend_freeze_8bit; break; - case BLEND_GLOW: param->blend = blend_glow_8bit; break; - case BLEND_HARDLIGHT: param->blend = blend_hardlight_8bit; break; - case BLEND_HARDMIX: param->blend = blend_hardmix_8bit; break; - case BLEND_HEAT: param->blend = blend_heat_8bit; break; - case BLEND_LIGHTEN: param->blend = blend_lighten_8bit; break; - case BLEND_LINEARLIGHT:param->blend = blend_linearlight_8bit;break; - case BLEND_MULTIPLY: param->blend = blend_multiply_8bit; break; - case BLEND_MULTIPLY128:param->blend = blend_multiply128_8bit;break; - case BLEND_NEGATION: param->blend = blend_negation_8bit; break; - case BLEND_NORMAL: param->blend = blend_normal_8bit; break; - case BLEND_OR: param->blend = blend_or_8bit; break; - case BLEND_OVERLAY: param->blend = blend_overlay_8bit; break; - case BLEND_PHOENIX: param->blend = blend_phoenix_8bit; break; - case BLEND_PINLIGHT: param->blend = blend_pinlight_8bit; break; - case BLEND_REFLECT: param->blend = blend_reflect_8bit; break; - case BLEND_SCREEN: param->blend = blend_screen_8bit; break; - case BLEND_SOFTLIGHT: param->blend = blend_softlight_8bit; break; - case BLEND_SUBTRACT: param->blend = blend_subtract_8bit; break; - case BLEND_VIVIDLIGHT: param->blend = blend_vividlight_8bit; break; - case BLEND_XOR: param->blend = blend_xor_8bit; break; - } + init_blend_func_8_8bit(param); break; case 10: - switch (param->mode) { - case BLEND_ADDITION: param->blend = blend_addition_10bit; break; - case BLEND_GRAINMERGE: param->blend = blend_grainmerge_10bit; break; - case BLEND_AND: param->blend = blend_and_10bit; break; - case BLEND_AVERAGE: param->blend = blend_average_10bit; break; - case BLEND_BURN: param->blend = blend_burn_10bit; break; - case BLEND_DARKEN: param->blend = blend_darken_10bit; break; - case BLEND_DIFFERENCE: param->blend = blend_difference_10bit; break; - case BLEND_GRAINEXTRACT: param->blend = blend_grainextract_10bit; break; - case BLEND_DIVIDE: param->blend = blend_divide_10bit; break; - case BLEND_DODGE: param->blend = blend_dodge_10bit; break; - case BLEND_EXCLUSION: param->blend = blend_exclusion_10bit; break; - case BLEND_EXTREMITY: param->blend = blend_extremity_10bit; break; - case BLEND_FREEZE: param->blend = blend_freeze_10bit; break; - case BLEND_GLOW: param->blend = blend_glow_10bit; break; - case BLEND_HARDLIGHT: param->blend = blend_hardlight_10bit; break; - case BLEND_HARDMIX: param->blend = blend_hardmix_10bit; break; - case BLEND_HEAT: param->blend = blend_heat_10bit; break; - case BLEND_LIGHTEN: param->blend = blend_lighten_10bit; break; - case BLEND_LINEARLIGHT:param->blend = blend_linearlight_10bit;break; - case BLEND_MULTIPLY: param->blend = blend_multiply_10bit; break; - case BLEND_MULTIPLY128:param->blend = blend_multiply128_10bit;break; - case BLEND_NEGATION: param->blend = blend_negation_10bit; break; - case BLEND_NORMAL: param->blend = blend_normal_16bit; break; - case BLEND_OR: param->blend = blend_or_10bit; break; - case BLEND_OVERLAY: param->blend = blend_overlay_10bit; break; - case BLEND_PHOENIX: param->blend = blend_phoenix_10bit; break; - case BLEND_PINLIGHT: param->blend = blend_pinlight_10bit; break; - case BLEND_REFLECT: param->blend = blend_reflect_10bit; break; - case BLEND_SCREEN: param->blend = blend_screen_10bit; break; - case BLEND_SOFTLIGHT: param->blend = blend_softlight_10bit; break; - case BLEND_SUBTRACT: param->blend = blend_subtract_10bit; break; - case BLEND_VIVIDLIGHT: param->blend = blend_vividlight_10bit;break; - case BLEND_XOR: param->blend = blend_xor_10bit; break; - } + init_blend_func_10_16bit(param); break; case 12: - switch (param->mode) { - case BLEND_ADDITION: param->blend = blend_addition_12bit; break; - case BLEND_GRAINMERGE: param->blend = blend_grainmerge_12bit; break; - case BLEND_AND: param->blend = blend_and_12bit; break; - case BLEND_AVERAGE: param->blend = blend_average_12bit; break; - case BLEND_BURN: param->blend = blend_burn_12bit; break; - case BLEND_DARKEN: param->blend = blend_darken_12bit; break; - case BLEND_DIFFERENCE: param->blend = blend_difference_12bit; break; - case BLEND_GRAINEXTRACT: param->blend = blend_grainextract_12bit; break; - case BLEND_DIVIDE: param->blend = blend_divide_12bit; break; - case BLEND_DODGE: param->blend = blend_dodge_12bit; break; - case BLEND_EXCLUSION: param->blend = blend_exclusion_12bit; break; - case BLEND_EXTREMITY: param->blend = blend_extremity_12bit; break; - case BLEND_FREEZE: param->blend = blend_freeze_12bit; break; - case BLEND_GLOW: param->blend = blend_glow_12bit; break; - case BLEND_HARDLIGHT: param->blend = blend_hardlight_12bit; break; - case BLEND_HARDMIX: param->blend = blend_hardmix_12bit; break; - case BLEND_HEAT: param->blend = blend_heat_12bit; break; - case BLEND_LIGHTEN: param->blend = blend_lighten_12bit; break; - case BLEND_LINEARLIGHT:param->blend = blend_linearlight_12bit;break; - case BLEND_MULTIPLY: param->blend = blend_multiply_12bit; break; - case BLEND_MULTIPLY128:param->blend = blend_multiply128_12bit;break; - case BLEND_NEGATION: param->blend = blend_negation_12bit; break; - case BLEND_NORMAL: param->blend = blend_normal_16bit; break; - case BLEND_OR: param->blend = blend_or_12bit; break; - case BLEND_OVERLAY: param->blend = blend_overlay_12bit; break; - case BLEND_PHOENIX: param->blend = blend_phoenix_12bit; break; - case BLEND_PINLIGHT: param->blend = blend_pinlight_12bit; break; - case BLEND_REFLECT: param->blend = blend_reflect_12bit; break; - case BLEND_SCREEN: param->blend = blend_screen_12bit; break; - case BLEND_SOFTLIGHT: param->blend = blend_softlight_12bit; break; - case BLEND_SUBTRACT: param->blend = blend_subtract_12bit; break; - case BLEND_VIVIDLIGHT: param->blend = blend_vividlight_12bit;break; - case BLEND_XOR: param->blend = blend_xor_12bit; break; - } + init_blend_func_12_16bit(param); break; case 16: - switch (param->mode) { - case BLEND_ADDITION: param->blend = blend_addition_16bit; break; - case BLEND_GRAINMERGE: param->blend = blend_grainmerge_16bit; break; - case BLEND_AND: param->blend = blend_and_16bit; break; - case BLEND_AVERAGE: param->blend = blend_average_16bit; break; - case BLEND_BURN: param->blend = blend_burn_16bit; break; - case BLEND_DARKEN: param->blend = blend_darken_16bit; break; - case BLEND_DIFFERENCE: param->blend = blend_difference_16bit; break; - case BLEND_GRAINEXTRACT: param->blend = blend_grainextract_16bit; break; - case BLEND_DIVIDE: param->blend = blend_divide_16bit; break; - case BLEND_DODGE: param->blend = blend_dodge_16bit; break; - case BLEND_EXCLUSION: param->blend = blend_exclusion_16bit; break; - case BLEND_EXTREMITY: param->blend = blend_extremity_16bit; break; - case BLEND_FREEZE: param->blend = blend_freeze_16bit; break; - case BLEND_GLOW: param->blend = blend_glow_16bit; break; - case BLEND_HARDLIGHT: param->blend = blend_hardlight_16bit; break; - case BLEND_HARDMIX: param->blend = blend_hardmix_16bit; break; - case BLEND_HEAT: param->blend = blend_heat_16bit; break; - case BLEND_LIGHTEN: param->blend = blend_lighten_16bit; break; - case BLEND_LINEARLIGHT:param->blend = blend_linearlight_16bit;break; - case BLEND_MULTIPLY: param->blend = blend_multiply_16bit; break; - case BLEND_MULTIPLY128:param->blend = blend_multiply128_16bit;break; - case BLEND_NEGATION: param->blend = blend_negation_16bit; break; - case BLEND_NORMAL: param->blend = blend_normal_16bit; break; - case BLEND_OR: param->blend = blend_or_16bit; break; - case BLEND_OVERLAY: param->blend = blend_overlay_16bit; break; - case BLEND_PHOENIX: param->blend = blend_phoenix_16bit; break; - case BLEND_PINLIGHT: param->blend = blend_pinlight_16bit; break; - case BLEND_REFLECT: param->blend = blend_reflect_16bit; break; - case BLEND_SCREEN: param->blend = blend_screen_16bit; break; - case BLEND_SOFTLIGHT: param->blend = blend_softlight_16bit; break; - case BLEND_SUBTRACT: param->blend = blend_subtract_16bit; break; - case BLEND_VIVIDLIGHT: param->blend = blend_vividlight_16bit;break; - case BLEND_XOR: param->blend = blend_xor_16bit; break; - } + init_blend_func_16_16bit(param); break; }