[MPlayer-dev-eng] [PATCH] add support alpry lossless codec

[Roberto Togni]

On 2004.03.24 20:04, melisha johnson wrote:
> Patch to add support for alpary soft lossless video
> codec
> 
> http://www.alparysoft.com/prod/compression/index.php
> 
> both dshow and vfw dlls supported
> 
> patch attached
Will apply it. I can't access ftp at the moment.


Is there a reason for these changes?
> @@ -3982,7 +4030,7 @@
>      return NULL;
>  }
> 
> -static int expwsprintfA(char* string, char* format, ...)
> +static int expwsprintfA(char* string, const char* format, ...)
Why changing it to const?
>  {
>      va_list va;
>      int result;
> @@ -3997,7 +4045,7 @@
>  {
>      va_list args;
>      int r;
> -    dbgprintf("sprintf(%s, %s)\n", str, format);
> +    dbgprintf("sprintf(0x%x, %s)\n", str, format);
>      va_start(args, format);
>      r = vsprintf(str, format, args);
>      va_end(args);
> @@ -4513,13 +4561,13 @@
>      return;
>  }
> 
> -int expRegisterClassA(const void/*WNDCLASSA*/ *wc)
> +static int WINAPI expRegisterClassA(const void/*WNDCLASSA*/ *wc)
>  {
>      dbgprintf("RegisterClassA(%p) => random id\n", wc);
>      return time(NULL); /* be precise ! */
>  }
> 
> -int expUnregisterClassA(const char *className, HINSTANCE hInstance)
> +static int WINAPI expUnregisterClassA(const char *className,  
> HINSTANCE hInstance)
>  {
>      dbgprintf("UnregisterClassA(%s, %p) => 0\n", className,  
> hInstance);
>      return 0;
Why changing the declarations of expRegisterClassA and  
expUnregisterClassA?

Are these changes needed for Alphary codecs? They look good, but i want  
to be sure they don't break other codecs (i can't thest them all).

Ciao,
 Roberto