This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 1/2]: Refactor relocate_path to also check if the relocated file/directory exists

From: Andreas Schwab <schwab at linux-m68k dot org>
To: Jan Kratochvil <jan dot kratochvil at redhat dot com>
Cc: Eli Zaretskii <eliz at gnu dot org>, khooyp at cs dot umd dot edu, gdb-patches at sourceware dot org
Date: Fri, 21 Sep 2012 21:09:00 +0200
Subject: Re: [PATCH 1/2]: Refactor relocate_path to also check if the relocated file/directory exists
References: <21ACC598-F6B4-4117-BA7B-B316414DE9E3@cs.umd.edu> <20120919130040.GA20442@host2.jankratochvil.net> <F768B2DD-1531-4911-9732-DAE6D346F159@cs.umd.edu> <20120921182637.GA8747@host2.jankratochvil.net> <83y5k3w7es.fsf@gnu.org> <20120921184623.GA12696@host2.jankratochvil.net>

Jan Kratochvil <jan.kratochvil@redhat.com> writes:

> It would have to be written as:
>   	  if (stat (path, &s) != 0 || !isdir != !S_ISDIR (s.st_mode))

A common idiom is to use !! when the other side is already boolean.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Follow-Ups:
- Re: [PATCH 1/2]: Refactor relocate_path to also check if the relocated file/directory exists
  - From: Khoo Yit Phang

References:
- [PATCH] Try to initialize data-directory by first searching for "data-directory" in the same directory as the gdb binary
  - From: Khoo Yit Phang
- Re: [PATCH] Try to initialize data-directory by first searching for "data-directory" in the same directory as the gdb binary
  - From: Jan Kratochvil
- [PATCH 1/2]: Refactor relocate_path to also check if the relocated file/directory exists
  - From: Khoo Yit Phang
- Re: [PATCH 1/2]: Refactor relocate_path to also check if the relocated file/directory exists
  - From: Jan Kratochvil
- Re: [PATCH 1/2]: Refactor relocate_path to also check if the relocated file/directory exists
  - From: Eli Zaretskii
- Re: [PATCH 1/2]: Refactor relocate_path to also check if the relocated file/directory exists
  - From: Jan Kratochvil

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]