Re: [PATCH v4] Add gdb.Value.format_string ()

Tom Tromey
2019-03-29T15:03:12Z

Marco> I’ve attached a patch which is identical to the previous one except for:

Thanks. Do you have a copyright assignment? I see you aren't in
MAINTAINERS, so I presume you don't have write access. Contact me off
list and we can set that up.

Marco> I’ve done this and indented the braces by two spaces and the array’s
Marco> content by another two spaces.
Marco> This seems to match my understanding of the GNU/GDB indentation style, but
Marco> I’m not 100% sure it’s the correct one as, in other places in the code
Marco> base, this kind of brace is not indented.

Yeah, I think that's the rule for braces, but it wasn't consistently
followed for array initializers in the past.

Tom