[PATCH 2/5] os: use a constant for backtrace array size
walter harms
wharms at bfs.de
Thu Oct 31 16:59:20 CET 2013
Am 30.10.2013 00:25, schrieb Peter Hutterer:
> Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
> os/backtrace.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/os/backtrace.c b/os/backtrace.c
> index 426f9b1..3d1195b 100644
> --- a/os/backtrace.c
> +++ b/os/backtrace.c
> @@ -114,14 +114,15 @@ xorg_backtrace(void)
> void
> xorg_backtrace(void)
> {
> - void *array[64];
> + const int BT_SIZE = 64;
> + void *array[BT_SIZE];
> const char *mod;
> int size, i;
> Dl_info info;
>
Your intentions are good but personaly i find this is confusing.
The std.way is to use a #define BT_SIZE 64
re,
wh
> ErrorFSigSafe("\n");
> ErrorFSigSafe("Backtrace:\n");
> - size = backtrace(array, 64);
> + size = backtrace(array, BT_SIZE);
> for (i = 0; i < size; i++) {
> int rc = dladdr(array[i], &info);
>
More information about the xorg-devel
mailing list