Author: muggenhor
Date: Wed Oct 15 19:48:35 2008
New Revision: 6166
URL: http://svn.gna.org/viewcvs/warzone?rev=6166&view=rev
Log:
* Pass the sequence text type to seq_AddTextForVideo and seq_AddTextFromFile
by using enum constants (from SEQ_TEXT_POSITIONING) instead of magic "numbers"
- (true and false are even worse than magic numbers when they're used to
represent non-boolean values, i.e. abused for the fact that they happen to be
defined to 1 and 0 respectively.
* This change *will* affect the way how FMV subtitles are displayed; this
could be considered a bugfix, or not...
- See the changes to seqdisp.c around line 680 and keep in mind that true =
1; and SEQ_TEXT_JUSTIFY = 2;
- Looking at the commented out code I think SEQ_TEXT_JUSTIFY is intented,
though due to true being 1 we've effectively been using SEQ_TEXT_FOLLOW_ON
instead since r1, see browser:trunk/game/src/[EMAIL PROTECTED]
Modified:
trunk/src/intelmap.c
trunk/src/seqdisp.c
_______________________________________________
Warzone-commits mailing list
[email protected]
https://mail.gna.org/listinfo/warzone-commits
