Critical
Things previously eliminated but returned. This should always be empty.
| BUG | Total | Description |
|---|---|---|
| PTR | 6 | Do not use PTR, ISO C 90 implies `void *' |
| IN SOLIB DYNSYM RESOLVE CODE | 5 | Replace IN_SOLIB_DYNSYM_RESOLVE_CODE with nothing, not needed? |
| TARGET HAS HARDWARE WATCHPOINTS | 5 | Replace TARGET_HAS_HARDWARE_WATCHPOINTS with nothing, not needed |
| SOLIB LOADED LIBRARY PATHNAME | 4 | Replace SOLIB_LOADED_LIBRARY_PATHNAME with nothing, not needed? |
| ADD SHARED SYMBOL FILES | 3 | Replace ADD_SHARED_SYMBOL_FILES with nothing, not needed? |
| FUNCTION EPILOGUE SIZE | 2 | Replace FUNCTION_EPILOGUE_SIZE with nothing, not needed? |
| var boolean | 2 | Replace var_boolean with add_setshow_boolean_cmd |
| asprintf | 1 | Do not use asprintf(), instead use xstrprintf() |
| dirent.h | 1 | Do not include dirent.h, instead include gdb_dirent.h |
| d namelen | 1 | Do not use dirent.d_namelen, instead use NAMELEN |
| gdb/utils.c missing abort | 1 | Expecting 4 occurances of bug abort in file gdb/utils.c, only found 3 |
| hash | 1 | Do not use ` #...', instead use `#...'(some compilers only correctly parse a C preprocessor directive when `#' is the first character on the line) |
| strlen d name | 1 | Do not use strlen dirent.d_name, instead use NAMELEN |
| 13 | 33 |
Code
Coding standard problems, portability problems, readability problems.
| BUG | Total | Description |
|---|---|---|
| OP eol | 314 | Do not use &&, or || at the end of a line |
| sprintf | 145 | Do not use sprintf, instead use xstrprintf |
| if assignment | 103 | An IF statement's expression contains an assignment |
| long long | 50 | Do not use `long long', instead use LONGEST |
| %ll | 24 | Do not use printf("%ll"), instead use printf("%s",phex()) to dump a `long long' value |
| %p | 18 | Do not use printf("%p"), instead use printf("%s",paddr()) to dump a target address |
| UCASE function | 17 | Function name is uppercase. |
| editCase function | 12 | Function name starts lower case but has uppercased letters. |
| inline | 9 | Do not use the inline attribute; since the compiler generally ignores this, better algorithm selection is needed to improved performance |
| if clause more than 50 lines | 0 | An IF statement's expression expands over 50 lines |
| 10 | 692 |
Comments
Problems concerning comments in source files.
| BUG | Total | Description |
|---|---|---|
| GNU/Linux | 0 | 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 |
| 1 | 0 |
DOS 8.3 File Names
File names with problems on 8.3 file systems.
| BUG | Total | Description |
|---|---|---|
| POWERPC-.C DOS 8.3 dup | 15 | DOSCHK - the names POWERPC-.C and gdb/features/rs6000/powerpc-32.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| POWERPC-.XML DOS 8.3 dup | 13 | DOSCHK - the names POWERPC-.XML and gdb/features/rs6000/powerpc-32.xml resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECIMAL1.H DOS 8.3 dup | 6 | DOSCHK - the names DECIMAL1.H and libdecnumber/bid/decimal128.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| CHANGELO DOS 8.3 dup | 5 | DOSCHK - the names CHANGELO and bfd/ChangeLog resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECIMAL3.H DOS 8.3 dup | 4 | DOSCHK - the names DECIMAL3.H and libdecnumber/bid/decimal32.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECIMAL6.H DOS 8.3 dup | 4 | DOSCHK - the names DECIMAL6.H and libdecnumber/bid/decimal64.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECNUMBE.H DOS 8.3 dup | 3 | DOSCHK - the names DECNUMBE.H and libdecnumber/decNumber.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| EFI-BSDR.C DOS 8.3 dup | 3 | DOSCHK - the names EFI-BSDR.C and bfd/efi-bsdrv-ia32.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| EFI-RTDR.C DOS 8.3 dup | 3 | DOSCHK - the names EFI-RTDR.C and bfd/efi-rtdrv-ia32.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| HOST-IEE.C DOS 8.3 dup | 3 | DOSCHK - the names HOST-IEE.C and libdecnumber/bid/host-ieee128.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| POWERPC-.DAT DOS 8.3 dup | 3 | DOSCHK - the names POWERPC-.DAT and gdb/regformats/rs6000/powerpc-32.dat resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| WATCHPOI.C DOS 8.3 dup | 3 | DOSCHK - the names WATCHPOI.C and gdb/testsuite/gdb.base/watchpoint-solib-shr.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| too many dots | 3 | DOSCHK - too many dots |
| CPU-CR16.C DOS 8.3 dup | 2 | DOSCHK - the names CPU-CR16.C and bfd/cpu-cr16.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECCONTE.H DOS 8.3 dup | 2 | DOSCHK - the names DECCONTE.H and libdecnumber/decContext.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECDOUBL.H DOS 8.3 dup | 2 | DOSCHK - the names DECDOUBL.H and libdecnumber/decDouble.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECPACKE.H DOS 8.3 dup | 2 | DOSCHK - the names DECPACKE.H and libdecnumber/decPacked.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| DECSINGL.H DOS 8.3 dup | 2 | DOSCHK - the names DECSINGL.H and libdecnumber/decSingle.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| ELF32-SH.H DOS 8.3 dup | 2 | DOSCHK - the names ELF32-SH.H and bfd/elf32-sh-relocs.h resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| ELF32-SP.C DOS 8.3 dup | 2 | DOSCHK - the names ELF32-SP.C and bfd/elf32-sparc.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| HPPANBSD.C DOS 8.3 dup | 2 | DOSCHK - the names HPPANBSD.C and gdb/hppanbsd-nat.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| I386-SIZ.C DOS 8.3 dup | 2 | DOSCHK - the names I386-SIZ.C and gdb/testsuite/gdb.arch/i386-size-overlap.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| I386-SIZ.EXP DOS 8.3 dup | 2 | DOSCHK - the names I386-SIZ.EXP and gdb/testsuite/gdb.arch/i386-size-overlap.exp resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| MB-INLIN.CC DOS 8.3 dup | 2 | DOSCHK - the names MB-INLIN.CC and gdb/testsuite/gdb.cp/mb-inline1.cc resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| MIPS32-D.S DOS 8.3 dup | 2 | DOSCHK - the names MIPS32-D.S and sim/testsuite/sim/mips/mips32-dsp.s resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| MIPS64-C.XML DOS 8.3 dup | 2 | DOSCHK - the names MIPS64-C.XML and gdb/features/mips64-cp0.xml resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| MI-VAR-C.EXP DOS 8.3 dup | 2 | DOSCHK - the names MI-VAR-C.EXP and gdb/testsuite/gdb.mi/mi-var-child-f.exp resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| MPROTECT.C DOS 8.3 dup | 2 | DOSCHK - the names MPROTECT.C and sim/testsuite/sim/cris/c/mprotect1.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| POWERPC-.EXP DOS 8.3 dup | 2 | DOSCHK - the names POWERPC-.EXP and gdb/testsuite/gdb.arch/powerpc-d128-regs.exp resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| REG-X86-.DAT DOS 8.3 dup | 2 | DOSCHK - the names REG-X86-.DAT and gdb/regformats/reg-x86-64-linux.dat resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| RTSIGPRO.C DOS 8.3 dup | 2 | DOSCHK - the names RTSIGPRO.C and sim/testsuite/sim/cris/c/rtsigprocmask1.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| RTSIGSUS.C DOS 8.3 dup | 2 | DOSCHK - the names RTSIGSUS.C and sim/testsuite/sim/cris/c/rtsigsuspend1.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| SIGRETUR.C DOS 8.3 dup | 2 | DOSCHK - the names SIGRETUR.C and sim/testsuite/sim/cris/c/sigreturn3.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| SOLIB-SY.C DOS 8.3 dup | 2 | DOSCHK - the names SOLIB-SY.C and gdb/testsuite/gdb.base/solib-symbol-lib.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| TYPE-OPA.C DOS 8.3 dup | 2 | DOSCHK - the names TYPE-OPA.C and gdb/testsuite/gdb.base/type-opaque-lib.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| WATCHPOI.EXP DOS 8.3 dup | 2 | DOSCHK - the names WATCHPOI.EXP and gdb/testsuite/gdb.base/watchpoint-solib.exp resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| XTENSA-L.C DOS 8.3 dup | 2 | DOSCHK - the names XTENSA-L.C and gdb/xtensa-linux-nat.c resolve to the same file on a DOS 8.3 system. For DOS, this can be fixed by modifying the file fnchange.lst. |
| 37 | 116 |
| BUG | Total | Description |
|---|---|---|
| deprecated symbol name | 131 | Do not use deprecated symbol_name, see declaration for details |
| deprecated xfer memory | 32 | Do not use deprecated xfer_memory, see declaration for details |
| deprecated sym private | 31 | Do not use deprecated sym_private, see declaration for details |
| deprecated set value type | 24 | Do not use deprecated set_value_type, see declaration for details |
| deprecated sym stab info | 20 | Do not use deprecated sym_stab_info, see declaration for details |
| deprecated safe get selected frame | 19 | Do not use deprecated safe_get_selected_frame, see declaration for details |
| deprecated child ops | 18 | Do not use deprecated child_ops, see declaration for details |
| deprecated ui loop hook | 10 | Do not use deprecated ui_loop_hook, see declaration for details |
| deprecated warn user | 10 | Do not use deprecated warn_user, see declaration for details |
| deprecated command loop hook | 9 | Do not use deprecated command_loop_hook, see declaration for details |
| deprecated print address numeric | 9 | Do not use deprecated print_address_numeric, see declaration for details |
| deprecated add core fns | 8 | Do not use deprecated add_core_fns, see declaration for details |
| deprecated detach hook | 8 | Do not use deprecated detach_hook, see declaration for details |
| deprecated query hook | 8 | Do not use deprecated query_hook, see declaration for details |
| deprecated readline hook | 8 | Do not use deprecated readline_hook, see declaration for details |
| deprecated target wait hook | 8 | Do not use deprecated target_wait_hook, see declaration for details |
| deprecated modify breakpoint hook | 7 | Do not use deprecated modify_breakpoint_hook, see declaration for details |
| deprecated readline begin hook | 7 | Do not use deprecated readline_begin_hook, see declaration for details |
| deprecated readline end hook | 7 | Do not use deprecated readline_end_hook, see declaration for details |
| deprecated show value hack | 7 | Do not use deprecated show_value_hack, see declaration for details |
| deprecated context hook | 6 | Do not use deprecated context_hook, see declaration for details |
| deprecated exec file display hook | 6 | Do not use deprecated exec_file_display_hook, see declaration for details |
| deprecated fp regnum | 6 | Do not use deprecated fp_regnum, see declaration for details |
| deprecated init ui hook | 6 | Do not use deprecated init_ui_hook, see declaration for details |
| deprecated memory changed hook | 6 | Do not use deprecated memory_changed_hook, see declaration for details |
| deprecated print frame info listing hook | 6 | Do not use deprecated print_frame_info_listing_hook, see declaration for details |
| deprecated register changed hook | 6 | Do not use deprecated register_changed_hook, see declaration for details |
| deprecated remove raw breakpoint | 6 | Do not use deprecated remove_raw_breakpoint, see declaration for details |
| deprecated ui load progress hook | 6 | Do not use deprecated ui_load_progress_hook, see declaration for details |
| deprecated warning hook | 6 | Do not use deprecated warning_hook, see declaration for details |
| deprecated delete breakpoint hook | 5 | Do not use deprecated delete_breakpoint_hook, see declaration for details |
| deprecated function start offset | 5 | Do not use deprecated function_start_offset, see declaration for details |
| deprecated set gdb event hooks | 5 | Do not use deprecated set_gdb_event_hooks, see declaration for details |
| deprecated throw reason | 5 | Do not use deprecated throw_reason, see declaration for details |
| deprecated call command hook | 4 | Do not use deprecated call_command_hook, see declaration for details |
| deprecated create breakpoint hook | 4 | Do not use deprecated create_breakpoint_hook, see declaration for details |
| deprecated error begin hook | 4 | Do not use deprecated error_begin_hook, see declaration for details |
| deprecated error hook | 4 | Do not use deprecated error_hook, see declaration for details |
| deprecated insert raw breakpoint | 4 | Do not use deprecated insert_raw_breakpoint, see declaration for details |
| deprecated interactive hook | 4 | Do not use deprecated interactive_hook, see declaration for details |
| deprecated selected frame level changed hook | 4 | Do not use deprecated selected_frame_level_changed_hook, see declaration for details |
| deprecated show load progress | 4 | Do not use deprecated show_load_progress, see declaration for details |
| deprecated target wait loop hook | 4 | Do not use deprecated target_wait_loop_hook, see declaration for details |
| deprecated value modifiable | 4 | Do not use deprecated value_modifiable, see declaration for details |
| deprecated cmd warning | 3 | Do not use deprecated cmd_warning, see declaration for details |
| deprecated set gdbarch data | 3 | Do not use deprecated set_gdbarch_data, see declaration for details |
| deprecated trace start stop hook | 3 | Do not use deprecated trace_start_stop_hook, see declaration for details |
| deprecated add set cmd | 2 | Do not use deprecated add_set_cmd, see declaration for details |
| deprecated annotate exited hook | 2 | Do not use deprecated annotate_exited_hook, see declaration for details |
| deprecated annotate signal hook | 2 | Do not use deprecated annotate_signal_hook, see declaration for details |
| deprecated annotate signalled hook | 2 | Do not use deprecated annotate_signalled_hook, see declaration for details |
| deprecated annotate starting hook | 2 | Do not use deprecated annotate_starting_hook, see declaration for details |
| deprecated annotate stopped hook | 2 | Do not use deprecated annotate_stopped_hook, see declaration for details |
| deprecated attach hook | 2 | Do not use deprecated attach_hook, see declaration for details |
| deprecated current gdbarch select hack | 2 | Do not use deprecated current_gdbarch_select_hack, see declaration for details |
| deprecated file changed hook | 2 | Do not use deprecated file_changed_hook, see declaration for details |
| deprecated ibm6000 target | 2 | Do not use deprecated ibm6000_target, see declaration for details |
| deprecated post add symbol hook | 2 | Do not use deprecated post_add_symbol_hook, see declaration for details |
| deprecated pre add symbol hook | 2 | Do not use deprecated pre_add_symbol_hook, see declaration for details |
| deprecated set hook | 2 | Do not use deprecated set_hook, see declaration for details |
| deprecated set value modifiable | 2 | Do not use deprecated set_value_modifiable, see declaration for details |
| deprecated target resume hook | 2 | Do not use deprecated target_resume_hook, see declaration for details |
| deprecated update frame pc hack | 2 | Do not use deprecated update_frame_pc_hack, see declaration for details |
| deprecated bcache | 1 | Do not use deprecated bcache, see declaration for details |
| deprecated extract floating | 1 | Do not use deprecated extract_floating, see declaration for details |
| deprecated pc in call dummy | 1 | Do not use deprecated pc_in_call_dummy, see declaration for details |
| deprecated trace find hook | 1 | Do not use deprecated trace_find_hook, see declaration for details |
| deprecated update frame base hack | 1 | Do not use deprecated update_frame_base_hack, see declaration for details |
| deprecated value address hack | 1 | Do not use deprecated value_address_hack, see declaration for details |
| deprecated value frame id hack | 1 | Do not use deprecated value_frame_id_hack, see declaration for details |
| deprecated value internalvar hack | 1 | Do not use deprecated value_internalvar_hack, see declaration for details |
| deprecated value lval hack | 1 | Do not use deprecated value_lval_hack, see declaration for details |
| deprecated value regnum hack | 1 | Do not use deprecated value_regnum_hack, see declaration for details |
| deprecated debug xfer memory | 0 | Do not use deprecated debug_xfer_memory, see declaration for details |
| deprecated mips set processor regs hack | 0 | Do not use deprecated mips_set_processor_regs_hack, see declaration for details |
| deprecated serial fd | 0 | Do not use deprecated serial_fd, see declaration for details |
| deprecated store floating | 0 | Do not use deprecated store_floating, see declaration for details |
| deprecated command loop | -1 | Do not use deprecated command_loop, see declaration for details |
| deprecated create tracepoint hook | -1 | Do not use deprecated create_tracepoint_hook, see declaration for details |
| deprecated delete tracepoint hook | -1 | Do not use deprecated delete_tracepoint_hook, see declaration for details |
| deprecated flush hook | -1 | Do not use deprecated flush_hook, see declaration for details |
| deprecated modify tracepoint hook | -1 | Do not use deprecated modify_tracepoint_hook, see declaration for details |
| deprecated obj private | -1 | Do not use deprecated obj_private, see declaration for details |
| 83 | 712 |
| BUG | Total | Description |
|---|---|---|
| get frame base | 22 | Replace get_frame_base with get_frame_id, get_frame_base_address, get_frame_locals_address, or get_frame_args_address. |
| frame register read | 19 | Replace frame_register_read() with get_frame_register(), or possibly introduce a new method safe_get_frame_register() |
| 2 | 41 |
| BUG | Total | Description |
|---|---|---|
| read memory | 134 | Replace read_memory() with regcache_read() et.al. |
| write memory | 124 | Replace write_memory() with regcache_read() et.al. |
| read pc | 47 | Replace READ_PC() with frame_pc_unwind; at present the inferior function call code still uses this |
| write pc | 33 | 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 |
| SOLIB ADD | 9 | Replace SOLIB_ADD with nothing, not needed? |
| PC SOLIB | 7 | Replace PC_SOLIB with nothing, not needed? |
| GCC2 COMPILED FLAG SYMBOL | 6 | Replace GCC2_COMPILED_FLAG_SYMBOL with nothing, not needed? |
| GCC COMPILED FLAG SYMBOL | 5 | Replace GCC_COMPILED_FLAG_SYMBOL with nothing, not needed? |
| SOLIB CREATE INFERIOR HOOK | 5 | Replace SOLIB_CREATE_INFERIOR_HOOK with nothing, not needed? |
| 9 | 370 |
| BUG | Total | Description |
|---|---|---|
| legacy register sim regno | 7 | Do not use legacy register_sim_regno, see declaration for details |
| legacy sim regno ignore | 3 | Do not use legacy sim_regno_ignore, see declaration for details |
| legacy name | 2 | Do not use legacy name, see declaration for details |
| legacy virtual frame pointer | 2 | Do not use legacy virtual_frame_pointer, see declaration for details |
| 4 | 22 |
| BUG | Total | Description |
|---|---|---|
| set register name | 56 | Call to set_gdbarch_register_name |
| set return value | 51 | Call to set_gdbarch_return_value |
| set push dummy call | 44 | Call to set_gdbarch_push_dummy_call |
| set num regs | 43 | Call to set_gdbarch_num_regs |
| set register type | 39 | Call to set_gdbarch_register_type |
| set long double bit | 36 | Call to set_gdbarch_long_double_bit |
| set print insn | 34 | Call to set_gdbarch_print_insn |
| set ptr bit | 34 | Call to set_gdbarch_ptr_bit |
| set sp regnum | 33 | Call to set_gdbarch_sp_regnum |
| set unwind pc | 33 | Call to set_gdbarch_unwind_pc |
| set skip prologue | 32 | Call to set_gdbarch_skip_prologue |
| set unwind dummy id | 32 | Call to set_gdbarch_unwind_dummy_id |
| set pc regnum | 31 | Call to set_gdbarch_pc_regnum |
| set breakpoint from pc | 30 | Call to set_gdbarch_breakpoint_from_pc |
| set inner than | 30 | Call to set_gdbarch_inner_than |
| set regset from core section | 30 | Call to set_gdbarch_regset_from_core_section |
| set num pseudo regs | 29 | Call to set_gdbarch_num_pseudo_regs |
| set skip trampoline code | 29 | Call to set_gdbarch_skip_trampoline_code |
| set long bit | 27 | Call to set_gdbarch_long_bit |
| set long long bit | 24 | Call to set_gdbarch_long_long_bit |
| set long double format | 23 | Call to set_gdbarch_long_double_format |
| set frame align | 22 | Call to set_gdbarch_frame_align |
| set pseudo register read | 21 | Call to set_gdbarch_pseudo_register_read |
| set fp0 regnum | 20 | Call to set_gdbarch_fp0_regnum |
| set pseudo register write | 20 | Call to set_gdbarch_pseudo_register_write |
| set unwind sp | 20 | Call to set_gdbarch_unwind_sp |
| set double bit | 19 | Call to set_gdbarch_double_bit |
| set dwarf2 reg to regnum | 19 | Call to set_gdbarch_dwarf2_reg_to_regnum |
| set int bit | 17 | Call to set_gdbarch_int_bit |
| set write pc | 17 | Call to set_gdbarch_write_pc |
| set register reggroup p | 16 | Call to set_gdbarch_register_reggroup_p |
| set software single step | 16 | Call to set_gdbarch_software_single_step |
| set addr bit | 15 | Call to set_gdbarch_addr_bit |
| set float bit | 15 | Call to set_gdbarch_float_bit |
| set short bit | 15 | Call to set_gdbarch_short_bit |
| set dwarf reg to regnum | 14 | Call to set_gdbarch_dwarf_reg_to_regnum |
| set fetch tls load module address | 14 | Call to set_gdbarch_fetch_tls_load_module_address |
| set register sim regno | 13 | Call to set_gdbarch_register_sim_regno |
| set skip solib resolver | 13 | Call to set_gdbarch_skip_solib_resolver |
| set stab reg to regnum | 13 | Call to set_gdbarch_stab_reg_to_regnum |
| set decr pc after break | 11 | Call to set_gdbarch_decr_pc_after_break |
| set believe pcc promotion | 10 | Call to set_gdbarch_believe_pcc_promotion |
| set frame args skip | 10 | Call to set_gdbarch_frame_args_skip |
| set char signed | 9 | Call to set_gdbarch_char_signed |
| set read pc | 9 | Call to set_gdbarch_read_pc |
| set get longjmp target | 8 | Call to set_gdbarch_get_longjmp_target |
| set in function epilogue p | 8 | Call to set_gdbarch_in_function_epilogue_p |
| set convert register p | 7 | Call to set_gdbarch_convert_register_p |
| set double format | 7 | Call to set_gdbarch_double_format |
| set register to value | 7 | Call to set_gdbarch_register_to_value |
| set value to register | 7 | Call to set_gdbarch_value_to_register |
| set call dummy location | 6 | Call to set_gdbarch_call_dummy_location |
| set cannot fetch register | 6 | Call to set_gdbarch_cannot_fetch_register |
| set cannot store register | 6 | Call to set_gdbarch_cannot_store_register |
| set float format | 6 | Call to set_gdbarch_float_format |
| set have nonsteppable watchpoint | 6 | Call to set_gdbarch_have_nonsteppable_watchpoint |
| set pointer to address | 6 | Call to set_gdbarch_pointer_to_address |
| set ps regnum | 6 | Call to set_gdbarch_ps_regnum |
| set addr bits remove | 5 | Call to set_gdbarch_addr_bits_remove |
| set address to pointer | 5 | Call to set_gdbarch_address_to_pointer |
| set convert from func ptr addr | 5 | Call to set_gdbarch_convert_from_func_ptr_addr |
| set ecoff reg to regnum | 5 | Call to set_gdbarch_ecoff_reg_to_regnum |
| set print registers info | 5 | Call to set_gdbarch_print_registers_info |
| set sofun address maybe missing | 5 | Call to set_gdbarch_sofun_address_maybe_missing |
| set elf make msymbol special | 4 | Call to set_gdbarch_elf_make_msymbol_special |
| set push dummy code | 4 | Call to set_gdbarch_push_dummy_code |
| set deprecated fp regnum | 3 | Call to set_gdbarch_deprecated_fp_regnum |
| set fetch pointer argument | 3 | Call to set_gdbarch_fetch_pointer_argument |
| set frame red zone size | 3 | Call to set_gdbarch_frame_red_zone_size |
| set from file | 3 | Call to set_gdbarch_from_file |
| set memory remove breakpoint | 3 | Call to set_gdbarch_memory_remove_breakpoint |
| set print float info | 3 | Call to set_gdbarch_print_float_info |
| set static transform name | 3 | Call to set_gdbarch_static_transform_name |
| set virtual frame pointer | 3 | Call to set_gdbarch_virtual_frame_pointer |
| set adjust breakpoint address | 2 | Call to set_gdbarch_adjust_breakpoint_address |
| set cannot step breakpoint | 2 | Call to set_gdbarch_cannot_step_breakpoint |
| set in solib return trampoline | 2 | Call to set_gdbarch_in_solib_return_trampoline |
| set integer to address | 2 | Call to set_gdbarch_integer_to_address |
| set memory insert breakpoint | 2 | Call to set_gdbarch_memory_insert_breakpoint |
| set overlay update | 2 | Call to set_gdbarch_overlay_update |
| set single step through delay | 2 | Call to set_gdbarch_single_step_through_delay |
| set smash text address | 2 | Call to set_gdbarch_smash_text_address |
| set stabs argument has addr | 2 | Call to set_gdbarch_stabs_argument_has_addr |
| set value from register | 2 | Call to set_gdbarch_value_from_register |
| set vbit in delta | 2 | Call to set_gdbarch_vbit_in_delta |
| set address class name to type flags | 1 | Call to set_gdbarch_address_class_name_to_type_flags |
| set address class type flags | 1 | Call to set_gdbarch_address_class_type_flags |
| set address class type flags to name | 1 | Call to set_gdbarch_address_class_type_flags_to_name |
| set coff make msymbol special | 1 | Call to set_gdbarch_coff_make_msymbol_special |
| set core read description | 1 | Call to set_gdbarch_core_read_description |
| set core xfer shared libraries | 1 | Call to set_gdbarch_core_xfer_shared_libraries |
| set deprecated function start offset | 1 | Call to set_gdbarch_deprecated_function_start_offset |
| set frame num args | 1 | Call to set_gdbarch_frame_num_args |
| set remote register number | 1 | Call to set_gdbarch_remote_register_number |
| set sdb reg to regnum | 1 | Call to set_gdbarch_sdb_reg_to_regnum |
| set skip permanent breakpoint | 1 | Call to set_gdbarch_skip_permanent_breakpoint |
| set vtable function descriptors | 1 | Call to set_gdbarch_vtable_function_descriptors |
| set bits big endian | 0 | Call to set_gdbarch_bits_big_endian |
| set construct inferior arguments | 0 | Call to set_gdbarch_construct_inferior_arguments |
| set name of malloc | 0 | Call to set_gdbarch_name_of_malloc |
| set print vector info | 0 | Call to set_gdbarch_print_vector_info |
| 101 | 1472 |
Macro
Breakdown of macro definitions (and #undef) in configuration files.
| BUG | Total | Description |
|---|---|---|
| nm I386 DR LOW GET STATUS | 5 | Do not define macros such as I386_DR_LOW_GET_STATUS in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm I386 DR LOW RESET ADDR | 5 | Do not define macros such as I386_DR_LOW_RESET_ADDR in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm I386 DR LOW SET ADDR | 5 | Do not define macros such as I386_DR_LOW_SET_ADDR in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm I386 DR LOW SET CONTROL | 5 | Do not define macros such as I386_DR_LOW_SET_CONTROL in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm I386 USE GENERIC WATCHPOINTS | 5 | Do not define macros such as I386_USE_GENERIC_WATCHPOINTS in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm STOPPED BY WATCHPOINT | 4 | Do not define macros such as STOPPED_BY_WATCHPOINT in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm target insert watchpoint | 4 | Do not define macros such as target_insert_watchpoint in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm target remove watchpoint | 4 | Do not define macros such as target_remove_watchpoint in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm HAVE CONTINUABLE WATCHPOINT | 3 | Do not define macros such as HAVE_CONTINUABLE_WATCHPOINT in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm TARGET HAS HARDWARE WATCHPOINTS | 3 | Do not define macros such as TARGET_HAS_HARDWARE_WATCHPOINTS in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm TARGET REGION SIZE OK FOR HW WATCHPOINT | 3 | Do not define macros such as TARGET_REGION_SIZE_OK_FOR_HW_WATCHPOINT in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm ATTACH NO WAIT | 2 | Do not define macros such as ATTACH_NO_WAIT in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm GDB FPREGSET T | 2 | Do not define macros such as GDB_FPREGSET_T in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm GDB GREGSET T | 2 | Do not define macros such as GDB_GREGSET_T in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm TARGET CAN USE HARDWARE WATCHPOINT | 2 | Do not define macros such as TARGET_CAN_USE_HARDWARE_WATCHPOINT in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm ADD SHARED SYMBOL FILES | 1 | Do not define macros such as ADD_SHARED_SYMBOL_FILES in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm CANNOT STEP HW WATCHPOINTS | 1 | Do not define macros such as CANNOT_STEP_HW_WATCHPOINTS in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm DEPRECATED IBM6000 TARGET | 1 | Do not define macros such as DEPRECATED_IBM6000_TARGET in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm FILL FPXREGSET | 1 | Do not define macros such as FILL_FPXREGSET in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm GET THREAD SIGNALS | 1 | Do not define macros such as GET_THREAD_SIGNALS in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm IN SOLIB DYNSYM RESOLVE CODE | 1 | Do not define macros such as IN_SOLIB_DYNSYM_RESOLVE_CODE in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm PC SOLIB | 1 | Do not define macros such as PC_SOLIB in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm PROCFS DONT TRACE FAULTS | 1 | Do not define macros such as PROCFS_DONT_TRACE_FAULTS in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm PROCFS NEED CLEAR CURSIG FOR KILL | 1 | Do not define macros such as PROCFS_NEED_CLEAR_CURSIG_FOR_KILL in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm REALTIME HI | 1 | Do not define macros such as REALTIME_HI in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm REALTIME LO | 1 | Do not define macros such as REALTIME_LO in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm SOLIB ADD | 1 | Do not define macros such as SOLIB_ADD in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm SOLIB CREATE INFERIOR HOOK | 1 | Do not define macros such as SOLIB_CREATE_INFERIOR_HOOK in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm START INFERIOR TRAPS EXPECTED | 1 | Do not define macros such as START_INFERIOR_TRAPS_EXPECTED in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm TARGET HAS DR LEN 8 | 1 | Do not define macros such as TARGET_HAS_DR_LEN_8 in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm TARGET REGION OK FOR HW WATCHPOINT | 1 | Do not define macros such as TARGET_REGION_OK_FOR_HW_WATCHPOINT in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm THREAD STATE CLEAR TRACED | 1 | Do not define macros such as THREAD_STATE_CLEAR_TRACED in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm THREAD STATE FLAVOR | 1 | Do not define macros such as THREAD_STATE_FLAVOR in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm THREAD STATE SET TRACED | 1 | Do not define macros such as THREAD_STATE_SET_TRACED in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm THREAD STATE SIZE | 1 | Do not define macros such as THREAD_STATE_SIZE in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm target insert hw breakpoint | 1 | Do not define macros such as target_insert_hw_breakpoint in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm target remove hw breakpoint | 1 | Do not define macros such as target_remove_hw_breakpoint in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| nm target stopped data address | 1 | Do not define macros such as target_stopped_data_address in a tm, nm or xm file, in fact do not provide a tm, nm or xm file |
| 38 | 77 |
Fixed
Problems that have been expunged from the source code.
| BUG | Total | Description |
|---|---|---|
| PTR | 6 | Do not use PTR, ISO C 90 implies `void *' |
| IN SOLIB DYNSYM RESOLVE CODE | 5 | Replace IN_SOLIB_DYNSYM_RESOLVE_CODE with nothing, not needed? |
| TARGET HAS HARDWARE WATCHPOINTS | 5 | Replace TARGET_HAS_HARDWARE_WATCHPOINTS with nothing, not needed |
| SOLIB LOADED LIBRARY PATHNAME | 4 | Replace SOLIB_LOADED_LIBRARY_PATHNAME with nothing, not needed? |
| ADD SHARED SYMBOL FILES | 3 | Replace ADD_SHARED_SYMBOL_FILES with nothing, not needed? |
| FUNCTION EPILOGUE SIZE | 2 | Replace FUNCTION_EPILOGUE_SIZE with nothing, not needed? |
| var boolean | 2 | Replace var_boolean with add_setshow_boolean_cmd |
| asprintf | 1 | Do not use asprintf(), instead use xstrprintf() |
| dirent.h | 1 | Do not include dirent.h, instead include gdb_dirent.h |
| d namelen | 1 | Do not use dirent.d_namelen, instead use NAMELEN |
| hash | 1 | Do not use ` #...', instead use `#...'(some compilers only correctly parse a C preprocessor directive when `#' is the first character on the line) |
| strlen d name | 1 | Do not use strlen dirent.d_name, instead use NAMELEN |
| ARGSUSED | 0 | Do not use ARGSUSED, unnecessary |
| ATTRIBUTE UNUSED | 0 | Do not use ATTRIBUTE_UNUSED, do not bother (GDB is compiled with -Werror and, consequently, is not able to tolerate false warnings. Since -Wunused-param produces such warnings, neither that warning flag nor ATTRIBUTE_UNUSED are used by GDB |
| BIG ENDIAN | 0 | Do not use BIG_ENDIAN, instead use BFD_ENDIAN_BIG |
| CYGWIN32 | 0 | Do not use __CYGWIN32__, instead use __CYGWIN__ or, better, an explicit autoconf tests |
| FUNCTION | 0 | Do not use __FUNCTION__, ISO C 90 does not support this macro |
| HAVE VFORK | 0 | Do not use HAVE_VFORK, instead include "gdb_vfork.h" and call vfork() unconditionally |
| LITTLE ENDIAN | 0 | Do not use LITTLE_ENDIAN, instead use BFD_ENDIAN_LITTLE |
| PARAMS | 0 | Do not use PARAMS(), ISO C 90 implies prototypes |
| PROCESS LINENUMBER HOOK | 0 | Replace PROCESS_LINENUMBER_HOOK with nothing, not needed? |
| REGISTER U ADDR | 0 | Replace REGISTER_U_ADDR with nothing, not needed? |
| abort | 0 | Do not use abort, instead use internal_error; GDB should never abort |
| assert | 0 | Do not use assert, instead use gdb_assert or internal_error; assert calls abort and GDB should never call abort |
| assert.h | 0 | Do not include assert.h, instead include "gdb_assert.h" |
| basename | 0 | Do not use basename, instead use lbasename |
| bcmp | 0 | Do not use bcmp(), ISO C 90 implies memcmp() |
| bcopy | 0 | Do not use bcopy(), ISO C 90 implies memcpy() and memmove() |
| boolean | 0 | Do not use `boolean', use `int' instead |
| bzero | 0 | Do not use bzero(), instead use memset() |
| error not internal-warning | 0 | Do not use error("internal-warning"), instead use internal_warning |
| false | 0 | Definitely do not use `false' in boolean expressions |
| floatformat from double | 0 | Do not use floatformat_from_double() from libierty, instead use floatformat_from_doublest() |
| floatformat to double | 0 | Do not use floatformat_to_double() from libierty, instead use floatformat_to_doublest() |
| frame unwind unsigned register | 0 | Replace frame_unwind_unsigned_register with frame_unwind_register_unsigned |
| func | 0 | Do not use __func__, ISO C 90 does not support this macro |
| generic target write pc | 0 | Replace generic_target_write_pc with a per-architecture implementation, this relies on PC_REGNUM which is being eliminated |
| generic use struct convention | 0 | Replace generic_use_struct_convention with nothing, EXTRACT_STRUCT_VALUE_ADDRESS is a predicate |
| gnu-regex.h | 0 | Do not include gnu-regex.h, instead include gdb_regex.h |
| multi-line string | 0 | Multi-line string must have the newline escaped |
| read register | 0 | Replace read_register() with regcache_read() et.al. |
| read sp | 0 | Replace read_sp() with frame_sp_unwind |
| regex.h | 0 | Do not include regex.h, instead include gdb_regex.h |
| register cached | 0 | Replace register_cached() with nothing, does not have a regcache parameter |
| sec ptr | 0 | Instead of sec_ptr, use struct bfd_section |
| setlinebuf | 0 | Do not use setlinebuf(), ISO C 90 implies setvbuf() |
| set register cached | 0 | Replace set_register_cached() with nothing, does not have a regcache parameter |
| stat.h | 0 | Do not include stat.h or sys/stat.h, instead include gdb_stat.h |
| strdup | 0 | Do not use strdup(), instead use xstrdup() |
| strerror | 0 | Do not use strerror(), instead use safe_strerror() |
| strnicmp | 0 | Do not use strnicmp(), instead use strncasecmp() |
| strsave | 0 | Do not use strsave(), instead use xstrdup() et.al. |
| true | 0 | Do not try to use `true' in boolean expressions |
| vasprintf | 0 | Do not use vasprintf(), instead use xstrvprintf |
| vfork.h | 0 | Do not include vfork.h, instead include gdb_vfork.h |
| vsprint | 0 | Do not use vsprint(), instead use xstrvprintf |
| wait.h | 0 | Do not include wait.h or sys/wait.h, instead include gdb_wait.h |
| write register | 0 | Replace write_register() with regcache_read() et.al. |
| xasprintf | 0 | Do not use xasprintf(), instead use xstrprintf |
| xregex.h | 0 | Do not include xregex.h, instead include gdb_regex.h |
| xvasprintf | 0 | Do not use xvasprintf(), instead use xstrvprintf |
| 61 | 32 |
Please send FSF & GNU inquiries & questions to gnu@gnu.org. There are also other ways to contact the FSF.
This page is maintained by the GDB developers.
Copyright Free Software Foundation, Inc., 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA.
Verbatim copying and distribution of this entire article is permitted in any medium, provided this notice is preserved.
Last modified 2009-07-04.