[Mplayer-cvslog] CVS: main/libmpdemux demux_mkv.c, 1.14, 1.15 demux_ogg.c, 1.51, 1.52

[Ivan Kalvachev]

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