[MPlayer-dev-eng] [PATCH] fix numerous const issues in libmpcodecs
Reimar Döffinger
Reimar.Doeffinger at stud.uni-karlsruhe.de
Fri Oct 19 10:48:07 CEST 2007
Hello,
On Thu, Oct 18, 2007 at 03:58:34PM -0400, Rich Felker wrote:
> @@ -43,7 +43,7 @@
> static unsigned int find_best(struct vf_instance_s* vf){
> unsigned int best=0;
> int ret;
> - unsigned int* p=bgr_list;
> + unsigned const int* p=bgr_list;
const in front for consistency would be nicer.
> Index: libmpcodecs/vf.c
> ===================================================================
> --- libmpcodecs/vf.c (revision 24811)
> +++ libmpcodecs/vf.c (working copy)
> @@ -24,84 +24,84 @@
> #include "libvo/fastmemcpy.h"
>
> extern vf_info_t vf_info_vo;
> -extern vf_info_t vf_info_rectangle;
Did you forget to change vf_info_vo or did you just not get that far?
> Index: libmpcodecs/vf_2xsai.c
> ===================================================================
> --- libmpcodecs/vf_2xsai.c (revision 24811)
> +++ libmpcodecs/vf_2xsai.c (working copy)
> @@ -16,6 +16,8 @@
> #define uint16 unsigned short
> #define uint8 unsigned char
>
> +/* FIXME: these all belong in the context, not as globals! */
> +
> static uint32 colorMask = 0xF7DEF7DE;
> static uint32 lowPixelMask = 0x08210821;
> static uint32 qcolorMask = 0xE79CE79C;
Just put that in an extra commit ;-)
As long as it compiles, I think it can't hurt to apply right away.
Greetings,
Reimar Döffinger
More information about the MPlayer-dev-eng
mailing list