:: Re: [Frei0r] "softglow" plugin
Page principale
Supprimer ce message
Répondre à ce message
Auteur: Dan Dennedy
Date:  
À: Minimalistic plugin API for video effects
Sujet: Re: [Frei0r] "softglow" plugin
On Mon, Oct 15, 2012 at 12:32 AM, Janne Liljeblad
<janne.liljeblad@???> wrote:
>> +++ b/src/filter/softglow/blur.h
>> @@ -0,0 +1,291 @@
>> +/* squareblur.c
>> + * Copyright (C) 2004--2005 Mathieu Guindon
>> + *                          Julien Keable
>> + *                          Jean-Sebastien Senecal (js@???)

>>
>> I see you copied this from src/filter/mask0mate, and it is also used
>> in src/filter/glow/. Should we perpetuate this madness or put blur.h
>> into include/ or refactor all of these to use a common blur.o?
>
> It is starting to get a bit silly, yes. I think putting blur.h into
> include/ would be appropriate here. If we agree on this, I'll can send
> you patch that does that.


I moved blur.h into include and pushed all of your patches.

--
+-DRD-+