attila said:
On Sat, May 08, 2004 at 07:52:27PM +0200, Jindrich Makovicka CVS wrote:
CVS change done by Jindrich Makovicka CVS
Update of /cvsroot/mplayer/main/libmpdemux In directory mail:/var2/tmp/cvs-serv31330/libmpdemux
Modified Files: demux_mkv.c demux_ogg.c Log Message: ENCA support (http://trific.ath.cx/software/enca/)
Index: demux_mkv.c =================================================================== RCS file: /cvsroot/mplayer/main/libmpdemux/demux_mkv.c,v retrieving revision 1.14 retrieving revision 1.15 diff -u -r1.14 -r1.15 --- demux_mkv.c 26 Apr 2004 09:22:17 -0000 1.14 +++ demux_mkv.c 8 May 2004 17:52:25 -0000 1.15 @@ -1832,7 +1832,7 @@ char *str;
#ifdef USE_ICONV - subcp_open(); + subcp_open_noenca(); #endif
Hmm ? This looks very bogous. I can currently not chekc what ENCA is (writting this mail offline) but it's definitly not iconv. Please either state why ENCA is iconv or reverse this change.
Attila Kinali Enca is a library that is trying to guess the codepage by knowing the country and having a sample from the text. subcp_open_noenca() simply parse the subcp command line parameter and pass it to subcp_open(char *), that now takes an codepage as parameter. Yeh i agree that it looks like cosmetic change, but I guess it is too late already:(
Wish You Best Ivan Kalvachev iive