On Wed, Aug 12, 2015 at 06:24:32PM +0530, Niklesh Lalwani wrote:
[...]
> +#define BOTTOM_LEFT     1
> +#define BOTTOM_CENTER   2
> +#define BOTTOM_RIGHT    3
> +#define MIDDLE_LEFT     4
> +#define MIDDLE_CENTER   5
> +#define MIDDLE_RIGHT    6
> +#define TOP_LEFT        7
> +#define TOP_CENTER      8
> +#define TOP_RIGHT       9
[...]
> -    tx3g_ptr += 2;
> +    h_align = *tx3g_ptr++;
> +    v_align = *tx3g_ptr++;

> +    if (h_align == 0) {
> +        if (v_align == 0)
> +            m->d.alignment = TOP_LEFT;
> +        if (v_align == 1)
> +            m->d.alignment = MIDDLE_LEFT;
> +        if (v_align == -1)
> +            m->d.alignment = BOTTOM_LEFT;
> +    }
> +    if (h_align == 1) {
> +        if (v_align == 0)
> +            m->d.alignment = TOP_CENTER;
> +        if (v_align == 1)
> +            m->d.alignment = MIDDLE_CENTER;
> +        if (v_align == -1)
> +            m->d.alignment = BOTTOM_CENTER;
> +    }
> +    if (h_align == -1) {
> +        if (v_align == 0)
> +            m->d.alignment = TOP_RIGHT;
> +        if (v_align == 1)
> +            m->d.alignment = MIDDLE_RIGHT;
> +        if (v_align == -1)
> +            m->d.alignment = BOTTOM_RIGHT;
> +    }

You might not like it, but you can simply do:

    m->d.alignment = (2 - (v_align+3) % 3) * 3 + ((h_align+3) % 3) + 1

and also drop the macro...

[...]

-- 
Clément B.

Attachment: pgpXGpV4xeQkb.pgp
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to