This is the mail archive of the gdb-patches@sourceware.org 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]

New ARI warning Wed Dec 7 01:55:04 UTC 2011


448,451d447
< gdb/linespec.c:454: code: sprintf: Do not use sprintf, instead use xsnprintf or xstrprintf
gdb/linespec.c:454:      sprintf (canonical_name, ', filename, symname);
< gdb/linespec.c:459: code: sprintf: Do not use sprintf, instead use xsnprintf or xstrprintf
gdb/linespec.c:459:      sprintf (canonical_name, ', filename, sal->line);
< gdb/linespec.c:1528: code: OP eol: Do not use &&, or || at the end of a line
gdb/linespec.c:1528:  if (sym_class &&
< gdb/linespec.c:1531: code: if assignment: An IF statement's expression contains an assignment (the GNU coding standard discourages this)
gdb/linespec.c:1531:  if (sym_class &&      (t = check_typedef (SYMBOL_TYPE (sym_class)),       (TYPE_CODE (t) == TYPE_CODE_STRUCT	|| TYPE_CODE (t) == TYPE_CODE_UNION)))
847a844,854
> gdb/s390-tdep.c:429: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:429:  /* Linux-specific optional "registers".  */
> gdb/s390-tdep.c:457: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:457:  /* Linux-specific optional "registers".  */
> gdb/s390-tdep.c:481: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:481:  /* Linux-specific optional "registers".  */
> gdb/s390-tdep.c:505: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:505:  /* Linux-specific optional "registers".  */
> gdb/s390-tdep.c:527: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:527:  /* Linux-specific optional "registers".  */
> gdb/s390-tdep.c:549: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:549:  /* Linux-specific optional "registers".  */
> gdb/s390-tdep.c:571: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:571:  /* Linux-specific optional "registers".  */
> gdb/s390-tdep.c:3071: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:3071:      /* Optional Linux-specific "registers".  */
> gdb/s390-tdep.c:3148: deprecate: write_pc: Replace write_pc() with get_frame_base_address or get_frame_id; at present the inferior function call code still uses this when doing a DECR_PC_AFTER_BREAK
gdb/s390-tdep.c:3148:  set_gdbarch_write_pc (gdbarch, s390_write_pc);
gdb/s390-tdep.c:3148:  set_gdbarch_write_pc (gdbarch, s390_write_pc);
> gdb/s390-tdep.c:3293: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.c:3293:  /* Initialize the Linux target descriptions.  */
> gdb/s390-tdep.h:98: comment: GNU/Linux: Do not use 'Linux', instead use 'Linux kernel' or 'GNU/Linux system'; comments should clearly differentiate between the two (this test assumes that word 'Linux' appears on the same line as the word 'GNU' or 'kernel' or a kernel version
gdb/s390-tdep.h:98:/* Linux-specific optional registers.  */


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