Commit cb6a0840 authored by jimw@rama.(none)'s avatar jimw@rama.(none)
Browse files

Bug #2717: include/my_global.h mis-defines __attribute__ 

  Fix when __attribute__() is stubbed out, add ATTRIBUTE_FORMAT() for specifying
  __attribute__((format(...))) safely, make more use of the format attribute,
  and fix some of the warnings that this turns up (plus a bonus unrelated one).
parent 34c83fba

include/m_ctype.h

+3 −2
Original line number Diff line number Diff line
@@ -175,7 +175,7 @@ typedef struct my_charset_handler_st 
  

  /* Charset dependant snprintf() */

  int  (*snprintf)(struct charset_info_st *, char *to, uint n, const char *fmt,

		   ...);

		   ...) ATTRIBUTE_FORMAT(printf, 4, 5);

  int  (*long10_to_str)(struct charset_info_st *, char *to, uint n, int radix,

			long int val);

  int (*longlong10_to_str)(struct charset_info_st *, char *to, uint n,
@@ -300,7 +300,8 @@ int my_wc_mb_8bit(CHARSET_INFO *cs,my_wc_t wc, uchar *s, uchar *e); 
ulong my_scan_8bit(CHARSET_INFO *cs, const char *b, const char *e, int sq);



int my_snprintf_8bit(struct charset_info_st *, char *to, uint n,

		     const char *fmt, ...);

		     const char *fmt, ...)

  ATTRIBUTE_FORMAT(printf, 4, 5);



long        my_strntol_8bit(CHARSET_INFO *, const char *s, uint l, int base,

			    char **e, int *err);

include/m_string.h

+2 −1
Original line number Diff line number Diff line
@@ -247,7 +247,8 @@ extern ulonglong strtoull(const char *str, char **ptr, int base); 


extern int my_vsnprintf( char *str, size_t n,

                                const char *format, va_list ap );

extern int my_snprintf(char* to, size_t n, const char* fmt, ...);

extern int my_snprintf(char *to, size_t n, const char *fmt, ...)

  ATTRIBUTE_FORMAT(printf, 3, 4);



#if defined(__cplusplus) && !defined(OS2)

}

include/my_global.h

+21 −1
Original line number Diff line number Diff line
@@ -414,14 +414,34 @@ typedef unsigned short ushort; 
#define function_volatile	volatile

#define my_reinterpret_cast(A) reinterpret_cast<A>

#define my_const_cast(A) const_cast<A>

# ifndef GCC_VERSION

#  define GCC_VERSION (__GNUC__ * 1000 + __GNUC_MINOR__)

# endif

#elif !defined(my_reinterpret_cast)

#define my_reinterpret_cast(A) (A)

#define my_const_cast(A) (A)

#endif

#if !defined(__attribute__) && (defined(__cplusplus) || !defined(__GNUC__)  || __GNUC__ == 2 && __GNUC_MINOR__ < 8)



/*

  Disable __attribute__() on GCC < 2.7 and non-GCC compilers

*/

#if !defined(__attribute__) && (!defined(__GNUC__) || GCC_VERSION < 2007)

#define __attribute__(A)

#endif



/*

  __attribute__((format(...))) is only supported in gcc >= 2.8 and g++ >= 3.4

*/

#ifndef ATTRIBUTE_FORMAT

# if defined(__GNUC__) && \

     ((!defined(__cplusplus__) && GCC_VERSION >= 2008) || \

      GCC_VERSION >= 3004)

#  define ATTRIBUTE_FORMAT(style, m, n) __attribute__((format(style, m, n)))

# else

#  define ATTRIBUTE_FORMAT(style, m, n)

# endif

#endif



/* From old s-system.h */



/*

include/my_sys.h

+2 −2
Original line number Diff line number Diff line
@@ -588,8 +588,8 @@ extern int my_chsize(File fd,my_off_t newlength, int filler, myf MyFlags); 
extern int my_sync(File fd, myf my_flags);

extern int my_error _VARARGS((int nr,myf MyFlags, ...));

extern int my_printf_error _VARARGS((uint my_err, const char *format,

				     myf MyFlags, ...)

				    __attribute__ ((format (printf, 2, 4))));

				     myf MyFlags, ...))

                                    ATTRIBUTE_FORMAT(printf, 2, 4);

extern int my_message(uint my_err, const char *str,myf MyFlags);

extern int my_message_no_curses(uint my_err, const char *str,myf MyFlags);

extern int my_message_curses(uint my_err, const char *str,myf MyFlags);

sql/item_subselect.cc

+1 −1
Original line number Diff line number Diff line
@@ -545,7 +545,7 @@ Item_allany_subselect::Item_allany_subselect(Item * left_exp, 
                                             chooser_compare_func_creator fc,

					     st_select_lex *select_lex,

					     bool all_arg)

  :Item_in_subselect(), all(all_arg), func_creator(fc)

  :Item_in_subselect(), func_creator(fc), all(all_arg)

{

  DBUG_ENTER("Item_in_subselect::Item_in_subselect");

  left_expr= left_exp;