Am 30.10.2013 00:25, schrieb Peter Hutterer:
> Signed-off-by: Peter Hutterer <[email protected]>
> ---
> 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);
>
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
