Hi Tom and community members, >> static const char * >> -rs6000_aix_auto_wide_charset (void) >> +rs6000_aix_auto_wide_charset_32 (void) >Could you remove the 'void' here? >> +static const char * >> +rs6000_aix_auto_wide_charset_64 (void) >Here too? >'void' is only needed here for C, it's an anachronism in the gdb code, >so when touching one of these lines it's nice to remove it; and not to >add new instances. Thanks for the review, Tom. Pushed with this change. https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=6fd421ad78bcb259f371394498e2c1346380e5b7 Regards, Aditya