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]

Re: RFC: initial TLS patch


+ extern struct so_list *get_solib_by_objfile (struct objfile *OBJFILE);

Tipo OBJFILE -> objfile.

It was capitalized to match the comment above:

+ /* Return the so_list entry whose object file is OBJFILE. */
+ extern struct so_list *get_solib_by_objfile (struct objfile *OBJFILE);
+
But the rest of GDB doesn't do that, so I'll change this.
Just FYI, from the coding doco:

The comment on a function is much clearer if you use the argument names to speak about the argument values. The variable name itself should be lower case, but write it in upper case when you are speaking about the value rather than the variable itself. Thus, "the inode number NODE_NUM" rather than "an inode".
enjoy,
Andrew



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]