Re: [PATCH] Install fileio.h to share File I/O protocol type definitions

[Julio Guerra <julio at farjump dot io>]

Le 25/07/2018 à 21:37, Tom Tromey a écrit :
>>>>>> "Julio" == Julio Guerra <julio@farjump.io> writes:
> Your message had ^M characters at the end of each line - no big deal but
> please make sure this isn't what would appear an eventual push.

I am not sure to understand where did you saw ^M chars? I used git
format-patch + send-email as usually, I don't understand where they
could come from.

> Julio> Prevent remote programs from redefining File I/O protocol types by installing
> Julio> include/gdb/fileio.h to share its type definitions.
>
> This seems reasonable enough, but what is it you plan to use it for?
> (I could guess I suppose but it seemed better to ask.)

When you implement gdb stubs supporting the File I/O protocol, the
remote programs then using File I/O services will require those types.
Right now, we have to define them ourselves, but by installing this
file, we could firstly stop redefining them, and secondly, have some
kind of single source of truth as it's easy to do mistakes when
redefining them (actually, our struct stat redefinition was wrong for
years, but no one was using it until now).

> It seems like it would be a good idea to have some documentation for
> this, if it's something we intend to support.

The documentation already talks about these types here
https://sourceware.org/gdb/onlinedocs/gdb/Protocol_002dspecific-Representation-of-Datatypes.html#Protocol_002dspecific-Representation-of-Datatypes
But do you want me to add some documentation about this file being
installed before the list?

-- 
Julio Guerra