This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] windows-nat: Decode system error numbers
- From: Pedro Alves <pedro at codesourcery dot com>
- To: gdb-patches at sourceware dot org, Eli Zaretskii <eliz at gnu dot org>
- Cc: "Maciej W. Rozycki" <macro at codesourcery dot com>
- Date: Wed, 9 Nov 2011 16:58:00 +0000
- Subject: Re: [PATCH] windows-nat: Decode system error numbers
- References: <alpine.DEB.1.10.1111091050490.5099@tp.orcam.me.uk> <83obwlgsc5.fsf@gnu.org>
On Wednesday 09 November 2011 16:42:50, Eli Zaretskii wrote:
> > Date: Wed, 9 Nov 2011 12:02:40 +0000
> > From: "Maciej W. Rozycki" <macro@codesourcery.com>
> >
> > + const char *msg = "Unspecified error.";
> > + unsigned long err;
> > + char buf[1025];
> > + size_t size;
> > +
> > + if (ok)
> > + return;
> > +
> > + err = GetLastError();
> > + size = FormatMessage (FORMAT_MESSAGE_FROM_SYSTEM
> > + | FORMAT_MESSAGE_IGNORE_INSERTS,
> > + NULL,
> > + err,
> > + MAKELANGID (LANG_NEUTRAL, SUBLANG_DEFAULT),
> > + buf, (sizeof (buf) - 1) / sizeof (TCHAR), NULL);
>
> Will this DTRT with Cygwin, which AFAIK wants the wide versions of the
> APIs? Is, for example, "char buf[1025];" appropriate in that case?
Yeah, if UNICODE is defined, FormatMessage maps to FormatMessageW and
buf will be filled with a wide string, though I'm not sure
if __USEWIDE implies UNICODE.
gdbserver/win32-low.c has strwinerror for this. It'd be nice if
windows-nat.c also abstracted this equally or similarly.
--
Pedro Alves