libwine: Make the debug functions obsolete.

Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Alexandre Julliard 2020-01-29 10:49:04 +01:00
parent 39138478fd
commit c1dadbcfb1
2 changed files with 35 additions and 33 deletions

View File

@ -20,6 +20,9 @@
#include "config.h" #include "config.h"
#include "wine/port.h" #include "wine/port.h"
#include "wine/asm.h"
#ifdef __ASM_OBSOLETE
#include <stdlib.h> #include <stdlib.h>
#include <stdio.h> #include <stdio.h>
@ -30,12 +33,6 @@
# include <sys/stat.h> # include <sys/stat.h>
#endif #endif
#define __wine_dbg_get_channel_flags __wine_dbg_get_channel_flags_inline
#define wine_dbg_sprintf wine_dbg_sprintf_inline
#define wine_dbg_printf wine_dbg_printf_inline
#define wine_dbg_log wine_dbg_log_inline
#define wine_dbgstr_an wine_dbgstr_an_inline
#define wine_dbgstr_wn wine_dbgstr_wn_inline
#include "wine/debug.h" #include "wine/debug.h"
#include "wine/library.h" #include "wine/library.h"
@ -70,8 +67,7 @@ static int cmp_name( const void *p1, const void *p2 )
} }
/* get the flags to use for a given channel, possibly setting them too in case of lazy init */ /* get the flags to use for a given channel, possibly setting them too in case of lazy init */
#undef __wine_dbg_get_channel_flags unsigned char __wine_dbg_get_channel_flags_obsolete( struct __wine_debug_channel *channel )
unsigned char __wine_dbg_get_channel_flags( struct __wine_debug_channel *channel )
{ {
if (nb_debug_options == -1) debug_init(); if (nb_debug_options == -1) debug_init();
@ -87,8 +83,8 @@ unsigned char __wine_dbg_get_channel_flags( struct __wine_debug_channel *channel
} }
/* set the flags to use for a given channel; return 0 if the channel is not available to set */ /* set the flags to use for a given channel; return 0 if the channel is not available to set */
int __wine_dbg_set_channel_flags( struct __wine_debug_channel *channel, int __wine_dbg_set_channel_flags_obsolete( struct __wine_debug_channel *channel,
unsigned char set, unsigned char clear ) unsigned char set, unsigned char clear )
{ {
if (nb_debug_options == -1) debug_init(); if (nb_debug_options == -1) debug_init();
@ -228,8 +224,7 @@ static void debug_init(void)
} }
/* varargs wrapper for funcs.dbg_vprintf */ /* varargs wrapper for funcs.dbg_vprintf */
#undef wine_dbg_printf int wine_dbg_printf_obsolete( const char *format, ... )
int wine_dbg_printf( const char *format, ... )
{ {
int ret; int ret;
va_list valist; va_list valist;
@ -241,8 +236,7 @@ int wine_dbg_printf( const char *format, ... )
} }
/* printf with temp buffer allocation */ /* printf with temp buffer allocation */
#undef wine_dbg_sprintf const char *wine_dbg_sprintf_obsolete( const char *format, ... )
const char *wine_dbg_sprintf( const char *format, ... )
{ {
static const int max_size = 200; static const int max_size = 200;
char *ret; char *ret;
@ -260,14 +254,13 @@ const char *wine_dbg_sprintf( const char *format, ... )
/* varargs wrapper for funcs.dbg_vlog */ /* varargs wrapper for funcs.dbg_vlog */
#undef wine_dbg_log int wine_dbg_log_obsolete( enum __wine_debug_class cls, struct __wine_debug_channel *channel,
int wine_dbg_log( enum __wine_debug_class cls, struct __wine_debug_channel *channel, const char *func, const char *format, ... )
const char *func, const char *format, ... )
{ {
int ret; int ret;
va_list valist; va_list valist;
if (!(__wine_dbg_get_channel_flags( channel ) & (1 << cls))) return -1; if (!(__wine_dbg_get_channel_flags_obsolete( channel ) & (1 << cls))) return -1;
va_start(valist, format); va_start(valist, format);
ret = funcs.dbg_vlog( cls, channel, func, format, valist ); ret = funcs.dbg_vlog( cls, channel, func, format, valist );
@ -424,7 +417,7 @@ static int default_dbg_vlog( enum __wine_debug_class cls, struct __wine_debug_ch
int ret = 0; int ret = 0;
if (cls < ARRAY_SIZE(debug_classes)) if (cls < ARRAY_SIZE(debug_classes))
ret += wine_dbg_printf( "%s:%s:%s ", debug_classes[cls], channel->name, func ); ret += wine_dbg_printf_obsolete( "%s:%s:%s ", debug_classes[cls], channel->name, func );
if (format) if (format)
ret += funcs.dbg_vprintf( format, args ); ret += funcs.dbg_vprintf( format, args );
return ret; return ret;
@ -432,20 +425,18 @@ static int default_dbg_vlog( enum __wine_debug_class cls, struct __wine_debug_ch
/* wrappers to use the function pointers */ /* wrappers to use the function pointers */
#undef wine_dbgstr_an const char *wine_dbgstr_an_obsolete( const char * s, int n )
const char *wine_dbgstr_an( const char * s, int n )
{ {
return funcs.dbgstr_an(s, n); return funcs.dbgstr_an(s, n);
} }
#undef wine_dbgstr_wn const char *wine_dbgstr_wn_obsolete( const WCHAR *s, int n )
const char *wine_dbgstr_wn( const WCHAR *s, int n )
{ {
return funcs.dbgstr_wn(s, n); return funcs.dbgstr_wn(s, n);
} }
void __wine_dbg_set_functions( const struct __wine_debug_functions *new_funcs, void __wine_dbg_set_functions_obsolete( const struct __wine_debug_functions *new_funcs,
struct __wine_debug_functions *old_funcs, size_t size ) struct __wine_debug_functions *old_funcs, size_t size )
{ {
if (old_funcs) memcpy( old_funcs, &funcs, min(sizeof(funcs),size) ); if (old_funcs) memcpy( old_funcs, &funcs, min(sizeof(funcs),size) );
if (new_funcs) memcpy( &funcs, new_funcs, min(sizeof(funcs),size) ); if (new_funcs) memcpy( &funcs, new_funcs, min(sizeof(funcs),size) );
@ -460,3 +451,14 @@ static struct __wine_debug_functions funcs =
default_dbg_vprintf, default_dbg_vprintf,
default_dbg_vlog default_dbg_vlog
}; };
__ASM_OBSOLETE(__wine_dbg_get_channel_flags);
__ASM_OBSOLETE(__wine_dbg_set_channel_flags);
__ASM_OBSOLETE(__wine_dbg_set_functions);
__ASM_OBSOLETE(wine_dbg_log);
__ASM_OBSOLETE(wine_dbg_printf);
__ASM_OBSOLETE(wine_dbg_sprintf);
__ASM_OBSOLETE(wine_dbgstr_an);
__ASM_OBSOLETE(wine_dbgstr_wn);
#endif /* __ASM_OBSOLETE */

View File

@ -1,9 +1,6 @@
WINE_1.0 WINE_1.0
{ {
global: global:
__wine_dbg_get_channel_flags;
__wine_dbg_set_channel_flags;
__wine_dbg_set_functions;
__wine_dll_register; __wine_dll_register;
__wine_main_argc; __wine_main_argc;
__wine_main_argv; __wine_main_argv;
@ -19,11 +16,6 @@ WINE_1.0
wine_cp_wcstombs; wine_cp_wcstombs;
wine_cpsymbol_mbstowcs; wine_cpsymbol_mbstowcs;
wine_cpsymbol_wcstombs; wine_cpsymbol_wcstombs;
wine_dbg_log;
wine_dbg_printf;
wine_dbg_sprintf;
wine_dbgstr_an;
wine_dbgstr_wn;
wine_dlclose; wine_dlclose;
wine_dll_enum_load_path; wine_dll_enum_load_path;
wine_dll_get_owner; wine_dll_get_owner;
@ -76,6 +68,9 @@ WINE_1.0
/* the following functions are obsolete and only exported for backwards compatibility */ /* the following functions are obsolete and only exported for backwards compatibility */
__wine_dbg_get_channel_flags;
__wine_dbg_set_channel_flags;
__wine_dbg_set_functions;
atoiW; atoiW;
atolW; atolW;
get_char_typeW; get_char_typeW;
@ -117,6 +112,11 @@ WINE_1.0
vsnprintfW; vsnprintfW;
vsprintfW; vsprintfW;
wine_call_on_stack; wine_call_on_stack;
wine_dbg_log;
wine_dbg_printf;
wine_dbg_sprintf;
wine_dbgstr_an;
wine_dbgstr_wn;
wine_pthread_get_functions; wine_pthread_get_functions;
wine_pthread_set_functions; wine_pthread_set_functions;
wine_switch_to_stack; wine_switch_to_stack;