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: [RFA-v2] Avoid invalid parameter warnings in C runtime function for mingw built GDB


On 08/14/2013 12:38 PM, Pierre Muller wrote:

> Is this OK to commit?
> Maybe some comments on the ChangeLog entry?

> 	* common/filestuff.c (gdb_fopen_cloexec): Do not try to use "e"
> 	mode if operating system doesn't know O_CLOEXEC, this allows to
> 	avoid getting a output debug string warning for mingw hosted
> 	GDB executables.

This comment should really be in the sources instead.  That here you'd
have:

	* common/filestuff.c (gdb_fopen_cloexec): Do not try to use "e"
	mode if operating system doesn't know O_CLOEXEC.

and in the source, where you have:

> +  /* If O_CLOEXEC is zero, the operating system doesn't
> +     know about close on exec mode "e", so don't even try to use it.  */
> +  static int fopen_e_ever_failed = O_CLOEXEC == 0;

I suggest:

  /* Probe for "e" support once.  But, if we can tell the operating
     system doesn't know about close on exec mode "e" without probing,
     skip it.  E.g., the Windows runtime issues an "Invalid parameter
     passed to C runtime function" OutputDebugString warning for
     unknown modes.  Assume that if O_CLOEXEC is zero, then "e" isn't
     supported.  */
  static int fopen_e_ever_failed;

-- 
Pedro Alves


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