This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On Thu, Jan 20, 2005 at 11:10:31PM +0200, Eli Zaretskii wrote: > > Is the following declaration near the end of ax-gdb.c unnecessary? > > > > void _initialize_ax_gdb (void); > > No, it isn't unnecessary (under certain command-line options, GCC will > whine if it sees a function definition without a prototype anywhere in > scope). Do you mean -Wmissing-prototypes? If the goal is to detect API functions not declared in headers, then I'm not sure whether this human work can be done by the compiler. Apparently, for some reason this function doesn't need to be declared in the header. What is the point using this option, then? > Please change "Illegal" into "Invalid" (here and everywhere else in > your patches). Oops, I've forgotten about that, sorry. Here's the new version. > Should we have an i18n comment here explaining what's a pcb? (You did > that elsewhere in the patch.) Hmm, I thought one time would suffice. But ok, let's create a luxurious messages file this time :) . With kind regards, Baurzhan. 2005-01-29 Baurzhan Ismagulov <ibr@radix50.net> * ax-gdb.c, ax-general.c, bcache.c, bfd-target.c, bsd-kvm.c, buildsym.c, c-lang.c, c-typeprint.c, c-valprint.c, charset.c, coff-pe-read.c, coffread.c, complaints.c, copying.c: I18n markup.
Attachment:
gdb-ibr-i18n_03-20050129-1038.diff
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |