[FFmpeg-devel] lavfi noise generator
Vitor Sessak
vitor1001
Thu Jan 15 21:53:06 CET 2009
Stefano Sabatini wrote:
> On date Saturday 2009-01-03 18:09:49 +0100, Stefano Sabatini encoded:
>> On date Monday 2008-12-29 13:45:23 +0100, Michael Niedermayer encoded:
>>> On Mon, Dec 29, 2008 at 11:39:30AM +0100, Stefano Sabatini wrote:
>>>> On date Monday 2008-12-29 00:52:18 +0100, Michael Niedermayer encoded:
>>>>> On Mon, Dec 29, 2008 at 12:25:02AM +0100, Stefano Sabatini wrote:
>>>>>> Hi all,
>>>>>>
>>>>>> no Michael I'm not posting it for you or others to review it, I'm just
>>>>> no but i reject it anyway
>>>>> your code is crap, sorry
>>>>>
>>>>> People didnt want to port libmpcodecs when i suggested it, i dont mind
>>>>> this at all, libmpcodecs has its serious issues but this does not mean i
>>>>> will accept code that is inferrior than it.
>>>>>
>>>>> vf_noise.c should be MUCH faster than your code.
>>> [...]
>>>> As for the speed concerns, I see two major issues to be addressed, the
>>>> use of the av_random() for setting every single byte, ideally we
>>>> should set the whole buffer for example reading from /dev/urandom but
>>>> this would not be portable, and the av_picture_copy() can be avoided
>>>> using properly the permissions system and passing a picture reference.
>>> did you read vf_noise.c ?
>> What about a noise module in libavutil (based on your vf_noise.c)?
>>
>> For example it may be shared between a noise source and filter, also I
>> think those routines may be useful also in other places.
>
> In attachment my idea, yes it still needs some rework (mainly I need
> to understand better the code of noise.[ch]), I basically just did
> some monkey translation from your code.
Just one comment:
> static int query_formats(AVFilterContext *ctx)
> {
> /* all YUV planar formats */
> avfilter_set_common_formats(ctx,
> avfilter_make_format_list(10,
> PIX_FMT_YUV444P, PIX_FMT_YUV422P, PIX_FMT_YUV420P,
> PIX_FMT_YUV411P, PIX_FMT_YUV410P,
> PIX_FMT_YUVJ444P, PIX_FMT_YUVJ422P, PIX_FMT_YUVJ420P,
> PIX_FMT_YUV440P, PIX_FMT_YUVJ440P)
> );
Can't the filter be simply modified to also accept most RGB formats?
-Vitor
More information about the ffmpeg-devel
mailing list