[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