[MPlayer-dev-eng] unrarlib still broken on 64bit
Attila Kinali
attila at kinali.ch
Tue Jul 3 18:12:29 CEST 2007
On Sun, 01 Jul 2007 18:56:26 -0300
Peter Cordes <peter at cordes.ca> wrote:
> --- unrarlib.h.upstream 2007-06-20 01:15:09.000000000 -0300
> +++ unrarlib.h 2007-07-01 17:33:55.192251911 -0300
> @@ -94,17 +94,11 @@
> #endif
> #endif
>
> -#ifdef _WIN_32
> -typedef unsigned char UBYTE; /* WIN32 definitions */
> -typedef unsigned short UWORD;
> -typedef unsigned long UDWORD;
> -#endif
> +#include <stdint.h>
> +typedef uint8_t UBYTE;
> +typedef uint16_t UWORD;
> +typedef uint32_t UDWORD;
>
> -#ifdef _UNIX /* LINUX/UNIX definitions */
> -typedef unsigned char UBYTE;
> -typedef unsigned short UWORD;
> -typedef unsigned long UDWORD;
> -#endif
The patch like this ins not acceptable, as IIRC
uint8_t & co are not necessarily defined on cygwin or mingw
(one of those, don't remember which one uses a horribly
outdated gcc). But at least the unix defines can be replaced,
no have to be replaced (they are wrong this way if specific
sizes are implied in the code).
> BTW, I noticed this because mplayer doesn't work with vobsubs in rar files
> on my 64bit machine, while the same subtitles still work on my x86 machine
> (both with current SVN mplayer). My patch fixes rar vobsubs on my 64bit
> machine.
>
> It's a pain creating old rar files to be compatible with unrarlib, but
> I don't like wasting several MB on uncompressed subs. (I know it's not
> much, but it bugs me.)
>
> 7-zip would appear to be an ideal archive format for vobsubs: it
> compresses very well (better than old compatible rar), it's fast, and is
> Free (LGPL), and there's a Free library for accessing 7z files.
> http://www.7-zip.org/sdk.html
> http://en.wikipedia.org/wiki/7z
>
AFAIK currently only rar is supported for compression.
(Am I right? i couldnt find anything in the manpage right now
and i'm to lazy to look at the code)
IMHO this should be replaced by a plugable system that is able
to support various compression schemes, foremost gzip, pkzip
and rar (if possible also rar3 and newer).
> But I wish AVI would just die and people would start using mkv. (it
> compresses subs.)
I wish MKV would just die and people would start using nut. ;-)
Attila Kinali
--
Praised are the Fountains of Shelieth, the silver harp of the waters,
But blest in my name forever this stream that stanched my thirst!
-- Deed of Morred
More information about the MPlayer-dev-eng
mailing list