Re: [PATCH 0/3] [gdb] Remove doxygen support

[Tom de Vries <tdevries@suse.de>]

On 5/30/26 1:39 AM, Kevin Buettner wrote:
> Hi Tom,
> 
> Thanks for doing this cleanup.
> 

Hi Kevin,

Thanks for the reviews.

> Claude tells me that '/**<' is also a doxygen marker.  There are 6
> occurrences in gdb/gdbtypes.h and 17 lines with that marker in
> gdb/type-codes.def.
> 
> I think it'd be a good idea to clean up those markers too.
> 

Ack, in a v2 I've added a patch for that ( 
https://sourceware.org/pipermail/gdb-patches/2026-June/227724.html ).

> In part 2, I noted a nit.  Parts 1 and 3 LGTM.  That said, I only gave
> a "Reviewed-by" semi-approval on those since Eli is the documentation
> maintainer for GDB.  After reconsidering, I think I should have given
> an "Approved-by" for part 3 - it's a .h file.  So, for part 2 (after
> the nit is fixed) and part 3:
> 
> Approved-By: Kevin Buettner <kevinb@redhat.com>

I'm guessing that the documentation maintainer role focuses on 
user-visible documentation, so I'm fine with committing without a review 
by Eli.

Thanks,
- Tom