[FFmpeg-devel] Fwd: [PATCH] lavfi: add xbr filter

arwa arif arwaarif1994 at gmail.com
Sun Nov 9 18:36:17 CET 2014


On Sun, Nov 9, 2014 at 3:36 PM, arwa arif <arwaarif1994 at gmail.com> wrote:

>
>
> On Sun, Nov 9, 2014 at 7:07 AM, Michael Niedermayer <michaelni at gmx.at>
> wrote:
>
>> On Sun, Nov 09, 2014 at 01:13:18AM +0100, Stefano Sabatini wrote:
>> > On date Sunday 2014-11-09 00:59:36 +0530, arwa arif encoded:
>> [...]
>> >
>> > > +#define pg_lbmask        0x00FEFEFE
>> > > +#define pg_red_blue_mask 0x00FF00FF
>> > > +#define pg_green_mask    0x0000FF00
>> >
>> > pg stands for? Also usually macros and defines are UPCASED.
>>
>> the names where taken from the reference code, so i suspect none of
>> us can know what pg stands for
>>
>>
>> [...]
>> > > +static uint32_t df(uint32_t x, uint32_t y, const uint32_t *r2y)
>> > > +{
>> > > +
>> > > +#define YMASK 0xff0000
>> > > +#define UMASK 0x00ff00
>> > > +#define VMASK 0x0000ff
>> > > +
>> > > +    uint32_t yuv1 = r2y[x & 0xffffff];
>> > > +    uint32_t yuv2 = r2y[y & 0xffffff];
>> >
>> > Note: is the mask needed?
>>
>> this depends on if AV_PIX_FMT_0RGB32 is guranteed to have the 0 byte
>> really 0. I would not bet on that, even if it works i suggest to leave
>> this masking in there somewhere
>>
>> [...]
>> --
>> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>>
>> The real ebay dictionary, page 1
>> "Used only once"    - "Some unspecified defect prevented a second use"
>> "In good condition" - "Can be repaird by experienced expert"
>> "As is" - "You wouldnt want it even if you were payed for it, if you knew
>> ..."
>>
>>
> I tried the running the filter without YUV masks, it is giving
> segmentation fault.
>
>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>>
>
I have implemented 2x,3x and 4x.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavfi-add-xbr-filter.patch
Type: text/x-patch
Size: 28550 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141109/51a8a0f4/attachment.bin>


More information about the ffmpeg-devel mailing list