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.
pgpXGpV4xeQkb.pgp
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel