Re: _AC_CHECK_TYPE_REPLACEMENT_TYPE_P

[Andreas Schwab]

Akim Demaille <address@hidden> writes:

|> >>>>> "Andreas" == Andreas Schwab <address@hidden> writes:
|> 
|> Andreas> IMHO, _AC_CHECK_TYPE_REPLACEMENT_TYPE_P should treat words of
|> Andreas> the form [a-z]*_t as types so that AC_CHECK_TYPE(__u8,
|> Andreas> u_int8_t) does the right thing.  Incidentally,
|> Andreas> AC_CHECK_TYPE(__s8, int8_t) is already treated as an old
|> Andreas> form, because int8_t starts with "int".
|> 
|> This was settled with Paul.
|> 
|> Paul, the code is
|> 
|> # _AC_CHECK_TYPE_REPLACEMENT_TYPE_P(STRING)
|> # -----------------------------------------
|> # Return `1' if STRING seems to be a builtin C/C++ type, i.e., if it
|> # starts with `_Bool', `bool', `char', `double', `float', `int',
|> # `long', `short', `signed', or `unsigned' followed by characters
|> # that are defining types.
|> # Because many people have used `off_t' and `size_t' too, they are added
|> # for better common-useward backward compatibility.
|> m4_define([_AC_CHECK_TYPE_REPLACEMENT_TYPE_P],
|> [m4_if(m4_regexp([$1],
|>                  
[^\(_Bool\|bool\|char\|double\|float\|int\|long\|short\|\(un\)?signed\|size_t\|off_t\)\([_a-zA-Z0-9()
 *]\|\[\|\]\)*$]),
|>        0, 1, 0)dnl
|> ])# _AC_CHECK_TYPE_REPLACEMENT_TYPE_P
|> 
|> 
|> But Andreas, I guess you do have a warning, don't you?  What patch
|> would you suggest?

Yes, I get a warning.  I'd like to have the regex changed to

^\(_Bool\|bool\|char\|double\|float\|int\|long\|short\|\(un\)?signed\|[_a-z0-9][_a-z0-9]*_t\)\([_a-zA-Z0-9()
 *]\|\[\|\]\)*$

Andreas.

-- 
Andreas Schwab                                  "And now for something
SuSE Labs                                        completely different."
address@hidden
SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5