Re: [PATCH libXau 0/3] remove redundant null check on calling free()

2017-10-28 Thread Daniel Martin
Am 28.10.2017 19:16 schrieb "walter harms" : After the last patch for libXau i checked the code with smatch and started to remove remove redundant One redundant "remove" here. ;-) All 3 patches are Reviewed-by: Daniel Martin ___ xorg-devel@lists.x.or

[PATCH libXau 3/3] AuRead.c: remove redundant null check on calling free()

2017-10-28 Thread walter harms
this removes simply unneeded code from XauReadAuth Signed-off-by: Walter Harms --- AuRead.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/AuRead.c b/AuRead.c index 5d41f03..d48906b 100644 --- a/AuRead.c +++ b/AuRead.c @@ -77,25 +77,25 @@ XauReadAuth (FIL

[PATCH libXau 2/3]Au FileName.c: remove redundant null check on calling free()

2017-10-28 Thread walter harms
remove redundant null check on calling free() Signed-off-by: Walter Harms --- AuFileName.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/AuFileName.c b/AuFileName.c index 2946c80..4ccda9d 100644 --- a/AuFileName.c +++ b/AuFileName.c @@ -68,8 +68,7 @@ XauFileName (void)

[PATCH libXau 1/3] AuDispose.c:remove redundant null check on calling free()

2017-10-28 Thread walter harms
redundant null check on auth->address calling free() redundant null check on auth->number calling free() redundant null check on auth->name calling free() Signed-off-by: Walter Harms --- AuDispose.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/AuDispose.c b/AuDispos

[PATCH libXau 0/3] remove redundant null check on calling free()

2017-10-28 Thread walter harms
After the last patch for libXau i checked the code with smatch and started to remove remove redundant null checks. just some cleanup. re, wh ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lis