[FFmpeg-devel] [PATCH] Unsharp filter

Daniel G. Taylor dan
Mon Apr 5 16:06:34 CEST 2010


Hey,

On 04/04/2010 11:29 PM, Bobby Bingham wrote:
> On Fri, 26 Mar 2010 11:21:39 -0400
> "Daniel G. Taylor"<dan at programmer-art.org>  wrote:
>> +static void init_filter_param(AVFilterContext *ctx, FilterParam *fp, const char *effect_type, int width)
>> +{
>> +    int z;
>> +    const char *effect;
>> +
>> +    effect = fp->amount == 0 ? "none" : fp->amount<  0 ? "blur" : "sharpen";
>> +
>> +    av_log(ctx, AV_LOG_INFO, "effect:%s type:%s msize_x:%d msize_y:%d amount:%0.2f\n", effect, effect_type, fp->msize_x, fp->msize_y, fp->amount / 65535.0);
>> +
>> +    memset(fp->sc, 0, sizeof(fp->sc));
>
> This memset doesn't look necessary.

Fixed this and fixed all nits from Stefano.

>> +    for (z = 0; z<  2 * fp->steps_y; z++)
>> +        fp->sc[z] = av_malloc(sizeof(*(fp->sc[z])) * (width + 2 * fp->steps_x));
>> +}
>

Attached is the latest patch. I also have permission to relicense from 
Remi under the LGPL as of yesterday, so this latest patch uses the LGPL 
and removes the change to configure to check for the GPL.

Take care,
-- 
Daniel G. Taylor
http://programmer-art.org/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: unsharp.diff
Type: text/x-diff
Size: 11644 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100405/c2f2638a/attachment.diff>



More information about the ffmpeg-devel mailing list