[patch] Fix for PR gdb/9538 (loading of separate debuginfo and symlinks).
Jan Kratochvil
jan.kratochvil@redhat.com
Thu Jan 12 23:27:00 GMT 2012
On Fri, 13 Jan 2012 00:17:55 +0100, Doug Evans wrote:
> OOC, Jan, what discussion led rise to having braces here:
There was only Mark's agreement with me as I read now:
Re: Code formatting [Re: [patch] s390*: watchpoints regression [repost]]
http://sourceware.org/ml/gdb-patches/2011-12/msg00600.html
So still not sure what is the general consensus.
> and does that reasoning apply here:
>
> + if (debuglink == NULL)
> + /* There's no separate debug info, hence there's no way we could
> + load it => no warning. */
> + return NULL;
There exist already many cases in GDB not compliant with this rule.
Regards,
Jan
More information about the Gdb-patches
mailing list