[MPlayer-dev-eng] Re: [PATCH] vobsub positioning code clean-up & scaling & all

[Csillag Kristof]

2005-01-21, p keltezéssel 18.31-kor Csillag Kristof ezt írta:
> Hi!
> 
> I finished the clean-up & the scaling. [...]
> [...] the scaling patch.
> [...] Now it's done.
...well, it did have some inaccuracies, and the documentation was
missing. So, use these instead:

1. a0_positioning-fix-and-cleanup.patch

	No new features, just bugfixes and code clean-up

2. a1_scaling.patch (depends on the previous patch)

	Introduces the -spu-scale option
	(Man page modification is included in the patch)

3. a2_horizontal_alignment.patch (depends on the previous patch)

	Introduces the -spualign_h option,
	which makes it possible to 
	align the subtitles left, center, right,
	or leave it in place.
	(Man page modification is included in the patch)

  * * *

In the earlier version of my scaling patch, I introduced the
"-spu-center: option, but later I realized that

a) it is a separate feature, so shoud go into a separate patch 

b) if I add an option, I should support not just center, but also
	left/right alignment, too.

So "-spu-center" is now gone, welcome "-spualign_h"
Yes, I know, the name sucks. How should I call it?
("-spualign" is already there for vertical alignment..)

  * * *

My other pending patches (namely, b0_crop_horizontally_too.patch, 
b1_black_box_bg.patch, d_calculate_alpha_efficiently.patch)
are compatible with these.

-- 
Csillag Kristof <fenwick at freemail.hu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: a0_positioning-fix-and-cleanup.patch
Type: text/x-patch
Size: 10681 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050122/51a1c061/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: a1_scaling.patch
Type: text/x-patch
Size: 5609 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050122/51a1c061/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: a2_horizontal_alignment.patch
Type: text/x-patch
Size: 3284 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050122/51a1c061/attachment-0002.bin>