[MPlayer-dev-eng] [patch] 64-bit fixes

Alex Beregszaszi alex at naxine.org
Thu May 23 15:53:28 CEST 2002


Hi,

-    i[3-9]86|x86*) return 0 ;;
+    i[3-9]86) return 0 ;;

why?
QNX uses the string "x86"
Yes, i saw you're checking for x86_64, also we should exclude it by the
above code.

And please avoid using stdint.h, use inttypes.h instead.
Third, do not patch TOOLS/, it's unneeded, also don't include it in the patch.

Could yo make a new patch according these restrictions?

+++ libmpcodecs/vf_palette.c
@@ -19,14 +19,14 @@
     IMGFMT_BGR24,
     IMGFMT_BGR16,
     IMGFMT_BGR15,
-    NULL
+    0
 };

Why is this needed?		

-
alex



More information about the MPlayer-dev-eng mailing list