diff options
108 files changed, 240 insertions, 124 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 9acc579..edf7c34 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,88 @@ +2019-10-26 Tom de Vries <tdevries@suse.de> + + * aarch64-linux-tdep.c: Fix typos in comments. + * aarch64-tdep.c: Same. + * ada-lang.c: Same. + * amd64-nat.c: Same. + * arc-tdep.c: Same. + * arch/aarch64-insn.c: Same. + * block.c: Same. + * breakpoint.h: Same. + * btrace.h: Same. + * c-varobj.c: Same. + * cli/cli-decode.c: Same. + * cli/cli-script.c: Same. + * cli/cli-utils.h: Same. + * coff-pe-read.c: Same. + * coffread.c: Same. + * compile/compile-cplus-symbols.c: Same. + * compile/compile-object-run.c: Same. + * completer.c: Same. + * corelow.c: Same. + * cp-support.c: Same. + * demangle.c: Same. + * dwarf-index-write.c: Same. + * dwarf2-frame.c: Same. + * dwarf2-frame.h: Same. + * eval.c: Same. + * frame-base.h: Same. + * frame.h: Same. + * gdbcmd.h: Same. + * gdbtypes.h: Same. + * gnu-nat.c: Same. + * guile/scm-objfile.c: Same. + * i386-tdep.c: Same. + * i386-tdep.h: Same. + * infcall.c: Same. + * infcall.h: Same. + * linux-nat.c: Same. + * m68k-tdep.c: Same. + * macroexp.c: Same. + * memattr.c: Same. + * mi/mi-cmd-disas.c: Same. + * mi/mi-getopt.h: Same. + * mi/mi-main.c: Same. + * minsyms.c: Same. + * nat/aarch64-sve-linux-sigcontext.h: Same. + * objfiles.h: Same. + * ppc-linux-nat.c: Same. + * ppc-linux-tdep.c: Same. + * ppc-tdep.h: Same. + * progspace.h: Same. + * prologue-value.h: Same. + * python/py-evtregistry.c: Same. + * python/py-instruction.h: Same. + * record-btrace.c: Same. + * record-full.c: Same. + * remote.c: Same. + * rs6000-tdep.c: Same. + * ser-tcp.c: Same. + * sol-thread.c: Same. + * sparc-sol2-tdep.c: Same. + * sparc64-tdep.c: Same. + * stabsread.c: Same. + * symfile.c: Same. + * symtab.h: Same. + * target.c: Same. + * tracepoint.c: Same. + * tui/tui-data.h: Same. + * tui/tui-io.c: Same. + * tui/tui-win.c: Same. + * tui/tui.c: Same. + * unittests/rsp-low-selftests.c: Same. + * user-regs.h: Same. + * utils.c: Same. + * utils.h: Same. + * valarith.c: Same. + * valops.c: Same. + * valprint.c: Same. + * valprint.h: Same. + * value.c: Same. + * value.h: Same. + * varobj.c: Same. + * x86-nat.h: Same. + * xtensa-tdep.c: Same. + 2019-10-25 Ali Tamur <tamur@google.com> * charset.c (find_charset_names): Reflect API change. diff --git a/gdb/aarch64-linux-tdep.c b/gdb/aarch64-linux-tdep.c index 3ec08d1..c8e30a4 100644 --- a/gdb/aarch64-linux-tdep.c +++ b/gdb/aarch64-linux-tdep.c @@ -1347,7 +1347,7 @@ aarch64_linux_get_syscall_number (struct gdbarch *gdbarch, thread_info *thread) This function will only ever get called when stopped at the entry or exit of a syscall, so by checking for 0 in x0 (arg0/retval), x1 (arg1), x8 (syscall), x29 (FP) and x30 (LR) we can infer: - 1) Either inferior is at exit from sucessful execve. + 1) Either inferior is at exit from successful execve. 2) Or inferior is at entry to a call to io_setup with invalid arguments and a corrupted FP and LR. It should be safe enough to assume case 1. */ diff --git a/gdb/aarch64-tdep.c b/gdb/aarch64-tdep.c index 2e428ca..af186e4 100644 --- a/gdb/aarch64-tdep.c +++ b/gdb/aarch64-tdep.c @@ -1414,7 +1414,7 @@ struct aarch64_call_info }; /* Pass a value in a sequence of consecutive X registers. The caller - is responsbile for ensuring sufficient registers are available. */ + is responsible for ensuring sufficient registers are available. */ static void pass_in_x (struct gdbarch *gdbarch, struct regcache *regcache, diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c index 6175219..5b54971 100644 --- a/gdb/ada-lang.c +++ b/gdb/ada-lang.c @@ -6443,7 +6443,7 @@ ada_is_ignored_field (struct type *type, int field_num) /* Anonymous field names should not be printed. brobecker/2007-02-20: I don't think this can actually happen - but we don't want to print the value of annonymous fields anyway. */ + but we don't want to print the value of anonymous fields anyway. */ if (name == NULL) return 1; @@ -10117,7 +10117,7 @@ ada_value_cast (struct type *type, struct value *arg2) information nor the associated type structure in GDB are able to express such dynamic types. So what the debugger does is to create "fixed" versions of the type that applies to the specific object. - We also informally refer to this opperation as "fixing" an object, + We also informally refer to this operation as "fixing" an object, which means creating its associated fixed type. Example: when printing the value of variable "Yes" above, its fixed diff --git a/gdb/amd64-nat.c b/gdb/amd64-nat.c index a65edc2..eb54b49 100644 --- a/gdb/amd64-nat.c +++ b/gdb/amd64-nat.c @@ -34,7 +34,7 @@ register set indexed by register number, and the number of registers supported by the mapping. We don't need mappings for the floating-point and SSE registers, since the difference between - 64-bit and 32-bit variants are negligable. The difference in the + 64-bit and 32-bit variants are negligible. The difference in the number of SSE registers is already handled by the target code. */ /* General-purpose register mapping for native 32-bit code. */ diff --git a/gdb/arc-tdep.c b/gdb/arc-tdep.c index 6bcd9b1..13b1c7f 100644 --- a/gdb/arc-tdep.c +++ b/gdb/arc-tdep.c @@ -969,7 +969,7 @@ arc_is_in_prologue (struct gdbarch *gdbarch, const struct arc_instruction &insn, /* Store of some register. May or may not update base address register. */ if (insn.insn_class == STORE || insn.insn_class == PUSH) { - /* There is definetely at least one operand - register/value being + /* There is definitely at least one operand - register/value being stored. */ gdb_assert (insn.operands_count > 0); @@ -1886,7 +1886,7 @@ arc_tdesc_init (struct gdbarch_info info, const struct target_desc **tdesc, } } - /* Mandatory AUX registeres are intentionally few and are common between + /* Mandatory AUX registers are intentionally few and are common between ARCompact and ARC v2, so same code can be used for both. */ feature = tdesc_find_feature (tdesc_loc, aux_minimal_feature_name); if (feature == NULL) diff --git a/gdb/arch/aarch64-insn.c b/gdb/arch/aarch64-insn.c index 48d0be8..89bdabc 100644 --- a/gdb/arch/aarch64-insn.c +++ b/gdb/arch/aarch64-insn.c @@ -47,7 +47,7 @@ extract_signed_bitfield (uint32_t insn, unsigned width, unsigned offset) INSN is the instruction opcode. MASK specifies the bits within the opcode that are to be tested - agsinst for a match with PATTERN. */ + against for a match with PATTERN. */ static int decode_masked_match (uint32_t insn, uint32_t mask, uint32_t pattern) diff --git a/gdb/block.c b/gdb/block.c index 5ba44d4..40cd3f4 100644 --- a/gdb/block.c +++ b/gdb/block.c @@ -343,7 +343,7 @@ block_set_using (struct block *block, } /* If BLOCK_NAMESPACE (block) is NULL, allocate it via OBSTACK and - ititialize its members to zero. */ + initialize its members to zero. */ static void block_initialize_namespace (struct block *block, struct obstack *obstack) diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h index da26f64..5c8f17c 100644 --- a/gdb/breakpoint.h +++ b/gdb/breakpoint.h @@ -375,7 +375,7 @@ public: agent_expr_up cmd_bytecode; /* Signals that breakpoint conditions and/or commands need to be - re-synched with the target. This has no use other than + re-synced with the target. This has no use other than target-side breakpoints. */ bool needs_update = false; @@ -672,7 +672,7 @@ enum watchpoint_triggered /* Some targets (e.g., embedded PowerPC) need two debug registers to set a watchpoint over a memory region. If this flag is true, GDB will use - only one register per watchpoint, thus assuming that all acesses that + only one register per watchpoint, thus assuming that all accesses that modify a memory location happen at its starting address. */ extern bool target_exact_watchpoints; diff --git a/gdb/btrace.h b/gdb/btrace.h index acde64a..d29fec8 100644 --- a/gdb/btrace.h +++ b/gdb/btrace.h @@ -445,7 +445,7 @@ extern int btrace_find_insn_by_number (struct btrace_insn_iterator *, unsigned int number); /* Dereference a branch trace call iterator. Return a pointer to the - function the iterator points to or NULL if the interator points past + function the iterator points to or NULL if the iterator points past the end of the branch trace. */ extern const struct btrace_function * btrace_call_get (const struct btrace_call_iterator *); diff --git a/gdb/c-varobj.c b/gdb/c-varobj.c index 50ce202..aad1859 100644 --- a/gdb/c-varobj.c +++ b/gdb/c-varobj.c @@ -82,7 +82,7 @@ adjust_value_for_child_access (struct value **value, /* Pointers to structures are treated just like structures when accessing children. Don't - dererences pointers to other types. */ + dereference pointers to other types. */ if (TYPE_CODE (*type) == TYPE_CODE_PTR) { struct type *target_type = get_target_type (*type); @@ -765,7 +765,7 @@ cplus_describe_child (const struct varobj *parent, int index, --type_index; /* If the type is anonymous and the field has no name, - set an appopriate name. */ + set an appropriate name. */ field_name = TYPE_FIELD_NAME (type, type_index); if (field_name == NULL || *field_name == '\0') { diff --git a/gdb/cli/cli-decode.c b/gdb/cli/cli-decode.c index debffbc..7ace72f 100644 --- a/gdb/cli/cli-decode.c +++ b/gdb/cli/cli-decode.c @@ -1038,7 +1038,7 @@ apropos_cmd (struct ui_file *stream, command that requires subcommands. Also called by saying just "help".) - I am going to split this into two seperate comamnds, help_cmd and + I am going to split this into two separate commands, help_cmd and help_list. */ void @@ -1570,7 +1570,7 @@ undef_cmd_error (const char *cmdtype, const char *q) unless ALLOW_UNKNOWN is negative. CMDTYPE precedes the word "command" in the error message. - If INGNORE_HELP_CLASSES is nonzero, ignore any command list + If IGNORE_HELP_CLASSES is nonzero, ignore any command list elements which are actually help classes rather than commands (i.e. the function field of the struct cmd_list_element is 0). */ diff --git a/gdb/cli/cli-script.c b/gdb/cli/cli-script.c index 3137955..8abd48c 100644 --- a/gdb/cli/cli-script.c +++ b/gdb/cli/cli-script.c @@ -251,7 +251,7 @@ print_command_lines (struct ui_out *uiout, struct command_line *cmd, } /* An if command. Recursively print both arms before - continueing. */ + continuing. */ if (list->control_type == if_control) { uiout->field_fmt (NULL, "if %s", list->line); diff --git a/gdb/cli/cli-utils.h b/gdb/cli/cli-utils.h index 33b8663..43e810f 100644 --- a/gdb/cli/cli-utils.h +++ b/gdb/cli/cli-utils.h @@ -56,7 +56,7 @@ extern void report_unrecognized_option_error (const char *command, /* Builds the help string for a command documented by PREFIX, followed by the extract_info_print_args help for ENTITY_KIND. If - DOCUMENT_N_FLAG is true then help text descibing the -n flag is also + DOCUMENT_N_FLAG is true then help text describing the -n flag is also included. */ const char *info_print_args_help (const char *prefix, diff --git a/gdb/coff-pe-read.c b/gdb/coff-pe-read.c index 11ce46d..b05357b 100644 --- a/gdb/coff-pe-read.c +++ b/gdb/coff-pe-read.c @@ -614,7 +614,7 @@ read_pe_exported_syms (minimal_symbol_reader &reader, /* Extract from ABFD the offset of the .text section. This offset is mainly related to the offset within the file. The value was previously expected to be 0x1000 for all files, - but some Windows OS core DLLs seem to use 0x10000 section alignement + but some Windows OS core DLLs seem to use 0x10000 section alignment which modified the return value of that function. Still return default 0x1000 value if ABFD is NULL or if '.text' section is not found, but that should not happen... */ diff --git a/gdb/coffread.c b/gdb/coffread.c index e582d8f..12f36b7 100644 --- a/gdb/coffread.c +++ b/gdb/coffread.c @@ -456,7 +456,7 @@ record_minimal_symbol (minimal_symbol_reader &reader, { /* Because the value of these symbols is within a function code range, these symbols interfere with the symbol-from-address - reverse lookup; this manifests itselfs in backtraces, or any + reverse lookup; this manifests itself in backtraces, or any other commands that prints symbolic addresses. Just pretend these symbols do not exist. */ return NULL; diff --git a/gdb/compile/compile-cplus-symbols.c b/gdb/compile/compile-cplus-symbols.c index 201c447..10ee856 100644 --- a/gdb/compile/compile-cplus-symbols.c +++ b/gdb/compile/compile-cplus-symbols.c @@ -194,7 +194,7 @@ convert_one_symbol (compile_cplus_instance *instance, if (scope.nested_type () != GCC_TYPE_NONE) { /* We found a symbol for this type that was defined inside - some other symbol, e.g., a class tyepdef defined. */ + some other symbol, e.g., a class typedef defined. */ return; } diff --git a/gdb/compile/compile-object-run.c b/gdb/compile/compile-object-run.c index 32e46f9..8173cfe 100644 --- a/gdb/compile/compile-object-run.c +++ b/gdb/compile/compile-object-run.c @@ -77,7 +77,7 @@ do_module_cleanup (void *arg, int registers_valid) addr_value = value_from_pointer (ptr_type, data->out_value_addr); - /* SCOPE_DATA would be stale unlesse EXECUTEDP != NULL. */ + /* SCOPE_DATA would be stale unless EXECUTEDP != NULL. */ compile_print_value (value_ind (addr_value), data->scope_data); } } diff --git a/gdb/completer.c b/gdb/completer.c index 07facfb..5f75aaa 100644 --- a/gdb/completer.c +++ b/gdb/completer.c @@ -865,7 +865,7 @@ complete_explicit_location (completion_tracker &tracker, else if (quoted_arg_end[1] == ' ') { /* We're maybe past the explicit location argument. - Skip the argument without interpretion, assuming the + Skip the argument without interpretation, assuming the user may want to create pending breakpoint. Offer the keyword and explicit location options as possible completions. */ diff --git a/gdb/corelow.c b/gdb/corelow.c index cea9210..b32fa95 100644 --- a/gdb/corelow.c +++ b/gdb/corelow.c @@ -330,7 +330,7 @@ maybe_say_no_core_file_now (int from_tty) printf_filtered (_("No core file now.\n")); } -/* Backward compatability with old way of specifying core files. */ +/* Backward compatibility with old way of specifying core files. */ void core_file_command (const char *filename, int from_tty) diff --git a/gdb/cp-support.c b/gdb/cp-support.c index 253369b..fd7ddc1 100644 --- a/gdb/cp-support.c +++ b/gdb/cp-support.c @@ -985,7 +985,7 @@ cp_find_first_component_aux (const char *name, int permissive) /* Operator names can show up in unexpected places. Since these can contain parentheses or angle brackets, they can screw up the recursion. But not every string 'operator' is part of an - operater name: e.g. you could have a variable 'cooperator'. So + operator name: e.g. you could have a variable 'cooperator'. So this variable tells us whether or not we should treat the string 'operator' as starting an operator. */ int operator_possible = 1; diff --git a/gdb/demangle.c b/gdb/demangle.c index cfe3421..d8b0310 100644 --- a/gdb/demangle.c +++ b/gdb/demangle.c @@ -75,7 +75,7 @@ show_asm_demangle (struct ui_file *file, int from_tty, static const char *current_demangling_style_string; -/* The array of names of the known demanglyng styles. Generated by +/* The array of names of the known demangling styles. Generated by _initialize_demangler from libiberty_demanglers[] array. */ static const char **demangling_style_names; diff --git a/gdb/dwarf-index-write.c b/gdb/dwarf-index-write.c index 420c53f..48f385e 100644 --- a/gdb/dwarf-index-write.c +++ b/gdb/dwarf-index-write.c @@ -951,7 +951,7 @@ private: { public: - /* Object costructor to be called for current DWARF2_PER_OBJFILE. + /* Object constructor to be called for current DWARF2_PER_OBJFILE. All .debug_str section strings are automatically stored. */ debug_str_lookup (struct dwarf2_per_objfile *dwarf2_per_objfile) : m_abfd (dwarf2_per_objfile->objfile->obfd), diff --git a/gdb/dwarf2-frame.c b/gdb/dwarf2-frame.c index f8c6d5d..c41db79 100644 --- a/gdb/dwarf2-frame.c +++ b/gdb/dwarf2-frame.c @@ -1343,7 +1343,7 @@ dwarf2_frame_sniffer (const struct frame_unwind *self, if (!dwarf2_frame_unwinders_enabled_p) return 0; - /* Grab an address that is guarenteed to reside somewhere within the + /* Grab an address that is guaranteed to reside somewhere within the function. get_frame_pc(), with a no-return next function, can end up returning something past the end of this function's body. If the frame we're sniffing for is a signal frame whose start diff --git a/gdb/dwarf2-frame.h b/gdb/dwarf2-frame.h index 3c54f09..d6dbeff 100644 --- a/gdb/dwarf2-frame.h +++ b/gdb/dwarf2-frame.h @@ -41,7 +41,7 @@ enum dwarf2_frame_reg_rule DWARF2_FRAME_REG_UNSPECIFIED = 0, /* The term "undefined" comes from the DWARF2 CFI spec which this - code is moddeling; it indicates that the register's value is + code is modeling; it indicates that the register's value is "undefined". GCC uses the less formal term "unsaved". Its definition is a combination of REG_UNDEFINED and REG_UNSPECIFIED. The failure to differentiate the two helps explain a few problems @@ -342,7 +342,7 @@ evaluate_struct_tuple (struct value *struct_val, /* Recursive helper function for setting elements of array tuples. The target is ARRAY (which has bounds LOW_BOUND to HIGH_BOUND); the element value is ELEMENT; EXP, POS and NOSIDE are as usual. - Evaluates index expresions and sets the specified element(s) of + Evaluates index expressions and sets the specified element(s) of ARRAY to ELEMENT. Returns last index value. */ static LONGEST diff --git a/gdb/frame-base.h b/gdb/frame-base.h index 4b23d08..fe51afb 100644 --- a/gdb/frame-base.h +++ b/gdb/frame-base.h @@ -36,7 +36,7 @@ struct regcache; debug info. It is assumed that dwarf2, stabs, ... will each provide their own methods. - A typical implmentation will return the same value for base, + A typical implementation will return the same value for base, locals-base and args-base. That value, however, will likely be different to the frame ID's stack address. */ diff --git a/gdb/frame.h b/gdb/frame.h index 624bc87..936ef50 100644 --- a/gdb/frame.h +++ b/gdb/frame.h @@ -21,7 +21,7 @@ #define FRAME_H 1 /* The following is the intended naming schema for frame functions. - It isn't 100% consistent, but it is aproaching that. Frame naming + It isn't 100% consistent, but it is approaching that. Frame naming schema: Prefixes: diff --git a/gdb/gdbcmd.h b/gdb/gdbcmd.h index 1b47719..675c25c 100644 --- a/gdb/gdbcmd.h +++ b/gdb/gdbcmd.h @@ -2,7 +2,7 @@ of the possible command languages. If necessary, a hook (that may be present or not) must be used and set to the appropriate routine by any command language that cares about it. If you are having to include this - file you are possibly doing things the old way. This file will disapear. + file you are possibly doing things the old way. This file will dissapear. fnasser@redhat.com */ /* Header file for GDB-specific command-line stuff. diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index 5539479..d016549 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,15 @@ +2019-10-26 Tom de Vries <tdevries@suse.de> + + * linux-aarch64-low.c: Fix typos in comments. + * linux-arm-low.c: Same. + * linux-low.c: Same. + * linux-ppc-low.c: Same. + * proc-service.c: Same. + * regcache.h: Same. + * server.c: Same. + * tracepoint.c: Same. + * win32-low.c: Same. + 2019-10-25 Tom Tromey <tromey@adacore.com> * utils.c (xstrdup): Remove. diff --git a/gdb/gdbserver/linux-aarch64-low.c b/gdb/gdbserver/linux-aarch64-low.c index ad04817..87a21a0 100644 --- a/gdb/gdbserver/linux-aarch64-low.c +++ b/gdb/gdbserver/linux-aarch64-low.c @@ -1948,7 +1948,7 @@ aarch64_install_fast_tracepoint_jump_pad (CORE_ADDR tpoint, for (i = 30; i >= 0; i -= 2) p += emit_stp_q_offset (p, i, i + 1, sp, i * 16); - /* Push general puspose registers on the stack. Note that we do not need + /* Push general purpose registers on the stack. Note that we do not need to push x31 as it represents the xzr register and not the stack pointer in a STR instruction. @@ -2116,7 +2116,7 @@ aarch64_install_fast_tracepoint_jump_pad (CORE_ADDR tpoint, ; This instruction is a normal store with memory ordering ; constraints. Thanks to this we do not have to put a data ; barrier instruction to make sure all data read and writes are done - ; before this instruction is executed. Furthermore, this instrucion + ; before this instruction is executed. Furthermore, this instruction ; will trigger an event, letting other threads know they can grab ; the lock. STLR xzr, [x0] @@ -2311,7 +2311,7 @@ aarch64_emit_prologue (void) the current stack pointer in the frame pointer. This way, it is not clobbered when calling C functions. - Finally, throughtout every operation, we are using register x0 as the + Finally, throughout every operation, we are using register x0 as the top of the stack, and x1 as a scratch register. */ p += emit_stp (p, x0, x1, sp, preindex_memory_operand (-2 * 16)); diff --git a/gdb/gdbserver/linux-arm-low.c b/gdb/gdbserver/linux-arm-low.c index 0e30af3..22b6be4 100644 --- a/gdb/gdbserver/linux-arm-low.c +++ b/gdb/gdbserver/linux-arm-low.c @@ -249,7 +249,7 @@ get_next_pcs_is_thumb (struct arm_get_next_pcs *self) return arm_is_thumb_mode (); } -/* Read memory from the inferiror. +/* Read memory from the inferior. BYTE_ORDER is ignored and there to keep compatiblity with GDB's read_memory_unsigned_integer. */ static ULONGEST diff --git a/gdb/gdbserver/linux-low.c b/gdb/gdbserver/linux-low.c index 0e4b14e..d6b6ce7 100644 --- a/gdb/gdbserver/linux-low.c +++ b/gdb/gdbserver/linux-low.c @@ -1613,7 +1613,7 @@ linux_detach (process_info *process) complete_ongoing_step_over (); /* Stop all threads before detaching. First, ptrace requires that - the thread is stopped to sucessfully detach. Second, thread_db + the thread is stopped to successfully detach. Second, thread_db may need to uninstall thread event breakpoints from memory, which only works with a stopped process anyway. */ stop_all_lwps (0, NULL); diff --git a/gdb/gdbserver/linux-ppc-low.c b/gdb/gdbserver/linux-ppc-low.c index c062282..d6c10c3 100644 --- a/gdb/gdbserver/linux-ppc-low.c +++ b/gdb/gdbserver/linux-ppc-low.c @@ -1588,7 +1588,7 @@ ppc_install_fast_tracepoint_jump_pad (CORE_ADDR tpoint, CORE_ADDR tpaddr, 6. Restore SP 7. Build a jump for back to the program 8. Copy/relocate original instruction - 9. Build a jump for replacing orignal instruction. */ + 9. Build a jump for replacing original instruction. */ /* Adjust stack pointer. */ if (is_64) diff --git a/gdb/gdbserver/proc-service.c b/gdb/gdbserver/proc-service.c index c5ebff2..512d6c6 100644 --- a/gdb/gdbserver/proc-service.c +++ b/gdb/gdbserver/proc-service.c @@ -21,7 +21,7 @@ #include "server.h" /* This file is currently tied to GNU/Linux. It should scale well to - another libthread_db implementation, with the approriate gdbserver + another libthread_db implementation, with the appropriate gdbserver hooks, but for now this means we can use GNU/Linux's target data. */ #include "linux-low.h" diff --git a/gdb/gdbserver/regcache.h b/gdb/gdbserver/regcache.h index d3a350a..4b2ea1b 100644 --- a/gdb/gdbserver/regcache.h +++ b/gdb/gdbserver/regcache.h @@ -42,7 +42,7 @@ struct regcache : public reg_buffer_common int registers_owned = 0; unsigned char *registers = nullptr; #ifndef IN_PROCESS_AGENT - /* One of REG_UNAVAILBLE or REG_VALID. */ + /* One of REG_UNAVAILABLE or REG_VALID. */ unsigned char *register_status = nullptr; #endif diff --git a/gdb/gdbserver/server.c b/gdb/gdbserver/server.c index 25a2be8..59e8a55 100644 --- a/gdb/gdbserver/server.c +++ b/gdb/gdbserver/server.c @@ -79,7 +79,7 @@ static bool exit_requested; /* --once: Exit after the first connection has closed. */ bool run_once; -/* Whether to report TARGET_WAITKING_NO_RESUMED events. */ +/* Whether to report TARGET_WAITKIND_NO_RESUMED events. */ static bool report_no_resumed; bool non_stop; diff --git a/gdb/gdbserver/tracepoint.c b/gdb/gdbserver/tracepoint.c index 0d02639..2bd75df 100644 --- a/gdb/gdbserver/tracepoint.c +++ b/gdb/gdbserver/tracepoint.c @@ -1002,7 +1002,7 @@ EXTERN_C_POP /* Control structure holding the read/write/etc. pointers into the trace buffer. We need more than one of these to implement a - transaction-like mechanism to garantees that both GDBserver and the + transaction-like mechanism to guarantees that both GDBserver and the in-process agent can try to change the trace buffer simultaneously. */ @@ -5154,7 +5154,7 @@ traceframe_walk_blocks (unsigned char *database, unsigned int datasize, return NULL; } -/* Look for the block of type TYPE_WANTED in the trameframe starting +/* Look for the block of type TYPE_WANTED in the traceframe starting at DATABASE of DATASIZE bytes long. TFNUM is the traceframe number. */ @@ -6310,7 +6310,7 @@ download_trace_state_variables (void) into GDBserver's trace buffer. This always uploads either all or no trace frames. This is the counter part of `trace_alloc_trace_buffer'. See its description of the atomic - synching mechanism. */ + syncing mechanism. */ static void upload_fast_traceframes (void) diff --git a/gdb/gdbserver/win32-low.c b/gdb/gdbserver/win32-low.c index 7088ba4..449ed5f 100644 --- a/gdb/gdbserver/win32-low.c +++ b/gdb/gdbserver/win32-low.c @@ -146,7 +146,7 @@ win32_set_thread_context (win32_thread_info *th) will often not be true. In those cases, the context returned by GetThreadContext will not be correct by the time the thread stops, hence we can't set that context back into the thread when - resuming - it will most likelly crash the inferior. + resuming - it will most likely crash the inferior. Unfortunately, there is no way to know when the thread will really stop. To work around it, we'll only write the context back to the thread when either the user or GDB explicitly change @@ -1426,7 +1426,7 @@ get_child_debug_event (struct target_waitstatus *ourstatus) else #endif { - /* Keep the wait time low enough for confortable remote + /* Keep the wait time low enough for comfortable remote interruption, but high enough so gdbserver doesn't become a bottleneck. */ if (!WaitForDebugEvent (¤t_event, 250)) diff --git a/gdb/gdbtypes.h b/gdb/gdbtypes.h index d431cb6..6d6ff59 100644 --- a/gdb/gdbtypes.h +++ b/gdb/gdbtypes.h @@ -292,7 +292,7 @@ DEF_ENUM_FLAGS_TYPE (enum type_instance_flag_value, type_instance_flags); #define TYPE_GNU_IFUNC(t) (TYPE_MAIN_TYPE (t)->flag_gnu_ifunc) /* * Type owner. If TYPE_OBJFILE_OWNED is true, the type is owned by - the objfile retrieved as TYPE_OBJFILE. Otherweise, the type is + the objfile retrieved as TYPE_OBJFILE. Otherwise, the type is owned by an architecture; TYPE_OBJFILE is NULL in this case. */ #define TYPE_OBJFILE_OWNED(t) (TYPE_MAIN_TYPE (t)->flag_objfile_owned) diff --git a/gdb/gnu-nat.c b/gdb/gnu-nat.c index b55c75b..93a50be 100644 --- a/gdb/gnu-nat.c +++ b/gdb/gnu-nat.c @@ -1731,7 +1731,7 @@ S_exception_raise_request (mach_port_t port, mach_port_t reply_port, } } else - /* A supppressed exception, which ignore. */ + /* A suppressed exception, which ignore. */ { inf->wait.suppress = 1; mach_port_deallocate (mach_task_self (), reply_port); diff --git a/gdb/guile/scm-objfile.c b/gdb/guile/scm-objfile.c index c70de76..7891ec0 100644 --- a/gdb/guile/scm-objfile.c +++ b/gdb/guile/scm-objfile.c @@ -346,7 +346,7 @@ gdbscm_execute_objfile_script (const struct extension_language_defn *extlang, ofscm_current_objfile = NULL; } -/* (current-objfile) -> <gdb:obfjile> +/* (current-objfile) -> <gdb:objfile> Return the current objfile, or #f if there isn't one. Ideally this would be named ofscm_current_objfile, but that name is taken by the variable recording the current objfile. */ diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c index ef2a9a7..52d286a 100644 --- a/gdb/i386-tdep.c +++ b/gdb/i386-tdep.c @@ -1622,7 +1622,7 @@ i386_analyze_frame_setup (struct gdbarch *gdbarch, /* Check for some special instructions that might be migrated by GCC into the prologue and skip them. At this point in the prologue, code should only touch the scratch registers %eax, - %ecx and %edx, so while the number of posibilities is sheer, + %ecx and %edx, so while the number of possibilities is sheer, it is limited. Make sure we only skip these instructions if we later see the diff --git a/gdb/i386-tdep.h b/gdb/i386-tdep.h index 456dd54..bcfa719 100644 --- a/gdb/i386-tdep.h +++ b/gdb/i386-tdep.h @@ -260,7 +260,7 @@ struct gdbarch_tdep /* Floating-point registers. */ -/* All FPU control regusters (except for FIOFF and FOOFF) are 16-bit +/* All FPU control registers (except for FIOFF and FOOFF) are 16-bit (at most) in the FPU, but are zero-extended to 32 bits in GDB's register cache. */ diff --git a/gdb/infcall.c b/gdb/infcall.c index 583f0de..9fc30ac 100644 --- a/gdb/infcall.c +++ b/gdb/infcall.c @@ -841,7 +841,7 @@ call_function_by_hand_dummy (struct value *function, void parameterless generic dummy frame calls to frameless functions will create a sequence of effectively identical frames (SP, FP and TOS and PC the same). This, not - suprisingly, results in what appears to be a stack in an + surprisingly, results in what appears to be a stack in an infinite loop --- when GDB tries to find a generic dummy frame on the internal dummy frame stack, it will always find the first one. diff --git a/gdb/infcall.h b/gdb/infcall.h index 9966fc6..c3f3fc8 100644 --- a/gdb/infcall.h +++ b/gdb/infcall.h @@ -43,7 +43,7 @@ extern CORE_ADDR find_function_addr (struct value *function, function returned. May fail to return, if a breakpoint or signal is hit during the execution of the function. - DFEAULT_RETURN_TYPE is used as function return type if the return + DEFAULT_RETURN_TYPE is used as function return type if the return type is unknown. This is used when calling functions with no debug info. diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c index 22e8303..bc39796 100644 --- a/gdb/linux-nat.c +++ b/gdb/linux-nat.c @@ -1468,7 +1468,7 @@ linux_nat_target::detach (inferior *inf, int from_tty) inferiors running. */ /* Stop all threads before detaching. ptrace requires that the - thread is stopped to sucessfully detach. */ + thread is stopped to successfully detach. */ iterate_over_lwps (ptid_t (pid), stop_callback); /* ... and wait until all of them have reported back that they're no longer running. */ @@ -3726,7 +3726,7 @@ linux_nat_target::kill () ptid_t ptid = ptid_t (inferior_ptid.pid ()); /* Stop all threads before killing them, since ptrace requires - that the thread is stopped to sucessfully PTRACE_KILL. */ + that the thread is stopped to successfully PTRACE_KILL. */ iterate_over_lwps (ptid, stop_callback); /* ... and wait until all of them have reported back that they're no longer running. */ diff --git a/gdb/m68k-tdep.c b/gdb/m68k-tdep.c index fb18cad..58cbf90 100644 --- a/gdb/m68k-tdep.c +++ b/gdb/m68k-tdep.c @@ -257,7 +257,7 @@ m68k_value_to_register (struct frame_info *frame, int regnum, The 68020/030/040/060 do support an FPU, either as a coprocessor (68881/2) or built-in (68040/68060). That's why System V release 4 - (SVR4) instroduces a new calling convention specified by the SVR4 + (SVR4) introduces a new calling convention specified by the SVR4 psABI. Integer values are returned in %d0/%d1, pointer return values in %a0 and floating values in %fp0. When calling functions returning a structure the caller should pass a pointer to a buffer diff --git a/gdb/macroexp.c b/gdb/macroexp.c index 33a72a7..65232bc 100644 --- a/gdb/macroexp.c +++ b/gdb/macroexp.c @@ -1333,7 +1333,7 @@ expand (const char *id, /* If the single token in SRC_FIRST followed by the tokens in SRC_REST - constitute a macro invokation not forbidden in NO_LOOP, append its + constitute a macro invocation not forbidden in NO_LOOP, append its expansion to DEST and return non-zero. Otherwise, return zero, and leave DEST unchanged. diff --git a/gdb/memattr.c b/gdb/memattr.c index 7a961af..8301a3f 100644 --- a/gdb/memattr.c +++ b/gdb/memattr.c @@ -133,7 +133,7 @@ create_user_mem_region (CORE_ADDR lo, CORE_ADDR hi, int ix = std::distance (user_mem_region_list.begin (), it); /* Check for an overlapping memory region. We only need to check - in the vicinity - at most one before and one after the + in the vincinity - at most one before and one after the insertion point. */ for (int i = ix - 1; i < ix + 1; i++) { diff --git a/gdb/mi/mi-cmd-disas.c b/gdb/mi/mi-cmd-disas.c index 7d38b8c..2f919e0 100644 --- a/gdb/mi/mi-cmd-disas.c +++ b/gdb/mi/mi-cmd-disas.c @@ -36,7 +36,7 @@ FILENAME: The name of the file where we want disassemble from. LINE: The line around which we want to disassemble. It will - disassemble the function that contins that line. + disassemble the function that contains that line. HOW_MANY: Number of disassembly lines to display. With source, it is the number of disassembly lines only, not counting the source lines. diff --git a/gdb/mi/mi-getopt.h b/gdb/mi/mi-getopt.h index 1513bb5..92e1938 100644 --- a/gdb/mi/mi-getopt.h +++ b/gdb/mi/mi-getopt.h @@ -58,7 +58,7 @@ extern int mi_getopt_allow_unknown (const char *prefix, int argc, /* mi_valid_noargs determines if ARGC/ARGV are a valid set of parameters to satisfy an MI function that is not supposed to - recieve any arguments. + receive any arguments. An MI function that should not receive arguments can still be passed parameters after the special option '--' such as below. diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c index 9a0b7f9..0e99fa3 100644 --- a/gdb/mi/mi-main.c +++ b/gdb/mi/mi-main.c @@ -1234,12 +1234,12 @@ mi_cmd_data_evaluate_expression (const char *command, char **argv, int argc) the ``x'' command. WORD-SIZE: size of each ``word''; 1,2,4, or 8 bytes. NR_ROW: Number of rows. - NR_COL: The number of colums (words per row). + NR_COL: The number of columns (words per row). ASCHAR: (OPTIONAL) Append an ascii character dump to each row. Use ASCHAR for unprintable characters. Reads SIZE*NR_ROW*NR_COL bytes starting at ADDR from memory and - displayes them. Returns: + displays them. Returns: {addr="...",rowN={wordN="..." ,... [,ascii="..."]}, ...} @@ -1359,7 +1359,7 @@ mi_cmd_data_read_memory (const char *command, char **argv, int argc) uiout->field_core_addr ("next-page", gdbarch, addr + total_bytes); uiout->field_core_addr ("prev-page", gdbarch, addr - total_bytes); - /* Build the result as a two dimentional table. */ + /* Build the result as a two dimensional table. */ { int row; int row_byte; diff --git a/gdb/minsyms.c b/gdb/minsyms.c index c41e5c3..0267472 100644 --- a/gdb/minsyms.c +++ b/gdb/minsyms.c @@ -737,7 +737,7 @@ lookup_minimal_symbol_by_pc_section (CORE_ADDR pc_in, struct obj_section *sectio or equal to the desired pc value, we accomplish two things: (1) the case where the pc value is larger than any minimal symbol address is trivially solved, (2) the address associated - with the hi index is always the one we want when the interation + with the hi index is always the one we want when the iteration terminates. In essence, we are iterating the test interval down until the pc value is pushed out of it from the high end. diff --git a/gdb/nat/aarch64-sve-linux-sigcontext.h b/gdb/nat/aarch64-sve-linux-sigcontext.h index 875c12c..750b210 100644 --- a/gdb/nat/aarch64-sve-linux-sigcontext.h +++ b/gdb/nat/aarch64-sve-linux-sigcontext.h @@ -133,7 +133,7 @@ struct sve_context { struct user_sve_header { __u32 size; /* total meaningful regset content in bytes */ - __u32 max_size; /* maxmium possible size for this thread */ + __u32 max_size; /* maximum possible size for this thread */ __u16 vl; /* current vector length */ __u16 max_vl; /* maximum possible vector length */ __u16 flags; diff --git a/gdb/objfiles.h b/gdb/objfiles.h index aa8e320..0c04458 100644 --- a/gdb/objfiles.h +++ b/gdb/objfiles.h @@ -69,8 +69,8 @@ struct partial_symbol; testcase are broken for some targets. In this test the functions are all implemented as part of one file and the testcase is not necessarily linked with a start file (depending on the target). - What happens is, that the first frame is printed normaly and - following frames are treated as being inside the enttry file then. + What happens is, that the first frame is printed normally and + following frames are treated as being inside the entry file then. This way, only the #0 frame is printed in the backtrace output.'' Ref "frame.c" "NOTE: vinschen/2003-04-01". diff --git a/gdb/ppc-linux-nat.c b/gdb/ppc-linux-nat.c index 532813d..e142755 100644 --- a/gdb/ppc-linux-nat.c +++ b/gdb/ppc-linux-nat.c @@ -616,7 +616,7 @@ fetch_register (struct regcache *regcache, int tid, int regno) if (altivec_register_p (gdbarch, regno)) { /* If this is the first time through, or if it is not the first - time through, and we have comfirmed that there is kernel + time through, and we have confirmed that there is kernel support for such a ptrace request, then go and fetch the register. */ if (have_ptrace_getvrregs) diff --git a/gdb/ppc-linux-tdep.c b/gdb/ppc-linux-tdep.c index 5a70852..fd88a89 100644 --- a/gdb/ppc-linux-tdep.c +++ b/gdb/ppc-linux-tdep.c @@ -150,7 +150,7 @@ static struct target_so_ops powerpc_so_ops; Examine the PLT again. Note that the loading of the shared library has initialized the PLT to code which loads a constant (which I think is an index into the GOT) into r11 and then - branchs a short distance to the code which actually does the + branches a short distance to the code which actually does the resolving. (gdb) x/2i 0x100409d4 diff --git a/gdb/ppc-tdep.h b/gdb/ppc-tdep.h index 1749f47..ceae089 100644 --- a/gdb/ppc-tdep.h +++ b/gdb/ppc-tdep.h @@ -433,7 +433,7 @@ extern int ppc_process_record (struct gdbarch *gdbarch, /* Instruction size. */ #define PPC_INSN_SIZE 4 -/* Estimate for the maximum number of instrctions in a function epilogue. */ +/* Estimate for the maximum number of instructions in a function epilogue. */ #define PPC_MAX_EPILOGUE_INSTRUCTIONS 52 #endif /* ppc-tdep.h */ diff --git a/gdb/progspace.h b/gdb/progspace.h index 5a053b9..c281648 100644 --- a/gdb/progspace.h +++ b/gdb/progspace.h @@ -140,7 +140,7 @@ struct program_space typedef next_adapter<struct objfile> objfiles_range; - /* Return an iterarable object that can be used to iterate over all + /* Return an iterable object that can be used to iterate over all objfiles. The basic use is in a foreach, like: for (objfile *objf : pspace->objfiles ()) { ... } */ diff --git a/gdb/prologue-value.h b/gdb/prologue-value.h index 74009d8..622830e 100644 --- a/gdb/prologue-value.h +++ b/gdb/prologue-value.h @@ -121,7 +121,7 @@ enum prologue_value_kind understand and maintain. In the approach used here: - It's easier to see that the analyzer is correct: you just see - whether the analyzer properly (albiet conservatively) simulates + whether the analyzer properly (albeit conservatively) simulates the effect of each instruction. - It's easier to extend the analyzer: you can add support for new diff --git a/gdb/python/py-evtregistry.c b/gdb/python/py-evtregistry.c index cf3505e..4ea2bcc 100644 --- a/gdb/python/py-evtregistry.c +++ b/gdb/python/py-evtregistry.c @@ -112,7 +112,7 @@ gdbpy_initialize_eventregistry (void) (PyObject *) &eventregistry_object_type); } -/* Retern the number of listeners currently connected to this +/* Return the number of listeners currently connected to this registry. */ int diff --git a/gdb/python/py-instruction.h b/gdb/python/py-instruction.h index d0d5306..2e5a9ac 100644 --- a/gdb/python/py-instruction.h +++ b/gdb/python/py-instruction.h @@ -24,7 +24,7 @@ /* Python type object for the abstract gdb.Instruction class. This class contains getters for four elements: "pc" (int), "data" (buffer), "decode" - (str) and "size" (int) that must be overriden by sub classes. */ + (str) and "size" (int) that must be overridden by sub classes. */ extern PyTypeObject py_insn_type; #endif /* PYTHON_PY_INSTRUCTION_H */ diff --git a/gdb/record-btrace.c b/gdb/record-btrace.c index 1171fa9..3fd4007 100644 --- a/gdb/record-btrace.c +++ b/gdb/record-btrace.c @@ -1861,7 +1861,7 @@ record_btrace_frame_dealloc_cache (struct frame_info *self, void *this_cache) } /* btrace recording does not store previous memory content, neither the stack - frames content. Any unwinding would return errorneous results as the stack + frames content. Any unwinding would return erroneous results as the stack contents no longer matches the changed PC value restored from history. Therefore this unwinder reports any possibly unwound registers as <unavailable>. */ diff --git a/gdb/record-full.c b/gdb/record-full.c index a940274..5502a01 100644 --- a/gdb/record-full.c +++ b/gdb/record-full.c @@ -53,7 +53,7 @@ mode, and we build up an execution log in which, for each executed instruction, we record all changes in memory and register state. This is invisible to the user, to whom it just looks like an - ordinary debugging session (except for performance degredation). + ordinary debugging session (except for performance degradation). In replay mode, instead of actually letting the inferior run as a process, we simulate its execution by playing back the recorded diff --git a/gdb/remote.c b/gdb/remote.c index 4e2f82a..8ea52d3 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -2829,7 +2829,7 @@ remote_target::thread_name (struct thread_info *info) /* About these extended threadlist and threadinfo packets. They are variable length packets but, the fields within them are often fixed - length. They are redundent enough to send over UDP as is the + length. They are redundant enough to send over UDP as is the remote protocol in general. There is a matching unit test module in libstub. */ diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c index 33cdc78..9c4df4a 100644 --- a/gdb/rs6000-tdep.c +++ b/gdb/rs6000-tdep.c @@ -1360,7 +1360,7 @@ rs6000_skip_stack_check (struct gdbarch *gdbarch, const CORE_ADDR start_pc) return pc; } - /* Third sequence: No probe; instead, a comparizon between the stack size + /* Third sequence: No probe; instead, a comparison between the stack size limit (saved in a run-time global variable) and the current stack pointer: diff --git a/gdb/ser-tcp.c b/gdb/ser-tcp.c index 079f248..37f64b5 100644 --- a/gdb/ser-tcp.c +++ b/gdb/ser-tcp.c @@ -307,7 +307,7 @@ net_open (struct serial *scb, const char *name) /* Flag to indicate whether we've got a connection refused. It will be true if any of the connections tried was refused. */ bool got_connrefused; - /* If a connection succeeeds, SUCCESS_AINFO will point to the + /* If a connection succeeds, SUCCESS_AINFO will point to the 'struct addrinfo' that succeed. */ struct addrinfo *success_ainfo = NULL; unsigned int polls = 0; diff --git a/gdb/sol-thread.c b/gdb/sol-thread.c index f723bd5..6b4cf20 100644 --- a/gdb/sol-thread.c +++ b/gdb/sol-thread.c @@ -22,7 +22,7 @@ to provide access to the Solaris user-mode thread implementation. Solaris threads are true user-mode threads, which are invoked via - the thr_* and pthread_* (native and POSIX respectivly) interfaces. + the thr_* and pthread_* (native and POSIX respectively) interfaces. These are mostly implemented in user-space, with all thread context kept in various structures that live in the user's heap. These should not be confused with lightweight processes (LWPs), which are @@ -253,7 +253,7 @@ td_err_string (td_err_e errcode) return buf; } -/* Return the libthread_db state string assicoated with STATECODE. +/* Return the libthread_db state string associated with STATECODE. If STATECODE is unknown, return an appropriate message. */ static const char * diff --git a/gdb/sparc-sol2-tdep.c b/gdb/sparc-sol2-tdep.c index 89755a1..0a8a9be 100644 --- a/gdb/sparc-sol2-tdep.c +++ b/gdb/sparc-sol2-tdep.c @@ -239,7 +239,7 @@ sparc_sol2_static_transform_name (const char *name) be incorrect in some places, at least for SPARC. The globalization prefix is encoded into an N_OPT stab, with the form "G=<prefix>". The globalization prefix always seems to start - with a dollar sign '$'; a dot '.' is used as a seperator. So we + with a dollar sign '$'; a dot '.' is used as a separator. So we simply strip everything up until the last dot. */ if (name[0] == '$') diff --git a/gdb/sparc64-tdep.c b/gdb/sparc64-tdep.c index fa55fcb..873fbaa 100644 --- a/gdb/sparc64-tdep.c +++ b/gdb/sparc64-tdep.c @@ -1199,7 +1199,7 @@ sparc64_16_byte_align_p (struct type *type) /* Store floating fields of element ELEMENT of an "parameter array" that has type TYPE and is stored at BITPOS in VALBUF in the - apropriate registers of REGCACHE. This function can be called + appropriate registers of REGCACHE. This function can be called recursively and therefore handles floating types in addition to structures. */ diff --git a/gdb/stabsread.c b/gdb/stabsread.c index 2896d98..b99ac20 100644 --- a/gdb/stabsread.c +++ b/gdb/stabsread.c @@ -4548,7 +4548,7 @@ cleanup_undefined_types_1 (void) undef_types_length = 0; } -/* Try to fix all the undefined types we ecountered while processing +/* Try to fix all the undefined types we encountered while processing this unit. */ void diff --git a/gdb/stubs/ChangeLog b/gdb/stubs/ChangeLog index 3626f75..835aed9 100644 --- a/gdb/stubs/ChangeLog +++ b/gdb/stubs/ChangeLog @@ -1,3 +1,10 @@ +2019-10-26 Tom de Vries <tdevries@suse.de> + + * ia64vms-stub.c: Fix typos in comments. + * m32r-stub.c: Same. + * m68k-stub.c: Same. + * sh-stub.c: Same. + 2019-01-21 Tom Tromey <tom@tromey.com> * ia64vms-stub.c: Fix includes. diff --git a/gdb/stubs/ia64vms-stub.c b/gdb/stubs/ia64vms-stub.c index 6e8ec4d..ccf4203 100644 --- a/gdb/stubs/ia64vms-stub.c +++ b/gdb/stubs/ia64vms-stub.c @@ -1924,7 +1924,7 @@ sock_write (const unsigned char *buf, int len) } } -/* Compute the cheksum and send the packet. */ +/* Compute the checksum and send the packet. */ static void send_pkt (void) @@ -2251,7 +2251,7 @@ excp_handler (struct chf$signal_array *sig, /* Self protection. FIXME: Should be per thread ? */ static int in_handler = 0; - /* Completly ignore some conditions (signaled indirectly by this stub). */ + /* Completely ignore some conditions (signaled indirectly by this stub). */ switch (code) { case LIB$_KEYNOTFOU & STS$M_COND_ID: diff --git a/gdb/stubs/m32r-stub.c b/gdb/stubs/m32r-stub.c index 4d54f72..40d0ca0 100644 --- a/gdb/stubs/m32r-stub.c +++ b/gdb/stubs/m32r-stub.c @@ -542,7 +542,7 @@ handle_exception (int exceptionVector) /* qCRC support */ -/* Table used by the crc32 function to calcuate the checksum. */ +/* Table used by the crc32 function to calculate the checksum. */ static unsigned long crc32_table[256] = { 0, 0 }; static unsigned long diff --git a/gdb/stubs/m68k-stub.c b/gdb/stubs/m68k-stub.c index 4ef4069..7f78a96 100644 --- a/gdb/stubs/m68k-stub.c +++ b/gdb/stubs/m68k-stub.c @@ -45,7 +45,7 @@ * program counter and status register onto the supervisor stack and then * transfers execution to a location specified in it's vector table. * The handlers for the exception vectors are hardwired to jmp to an address - * given by the relation: (exception - 256) * 6. These are decending + * given by the relation: (exception - 256) * 6. These are descending * addresses starting from -6, -12, -18, ... By allowing 6 bytes for * each entry, a jsr, jmp, bsr, ... can be used to enter the exception * handler. Using a jsr to handle an exception has an added benefit of diff --git a/gdb/stubs/sh-stub.c b/gdb/stubs/sh-stub.c index 76c98a5..c0d14f1 100644 --- a/gdb/stubs/sh-stub.c +++ b/gdb/stubs/sh-stub.c @@ -111,7 +111,7 @@ r... = register contents or... WAA The process exited, and AA is the exit status. This is only - applicable for certains sorts of + applicable for certain sorts of targets. kill request k @@ -137,7 +137,7 @@ Responses can be run-length encoded to save space. A '*' means that the next character is an ASCII encoding giving a repeat count which - stands for that many repititions of the character preceding the '*'. + stands for that many repetitions of the character preceding the '*'. The encoding is n+29, yielding a printable character where n >=3 (which is where rle starts to win). Don't use an n > 126. diff --git a/gdb/symfile.c b/gdb/symfile.c index d2ed1cc..c5d226e 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -1044,7 +1044,7 @@ finish_new_objfile (struct objfile *objfile, symfile_add_flags add_flags) For NAME description see the objfile constructor. ADD_FLAGS encodes verbosity, whether this is main symbol file or - extra, such as dynamically loaded code, and what to do with breakpoins. + extra, such as dynamically loaded code, and what to do with breakpoints. ADDRS is as described for syms_from_objfile_1, above. ADDRS is ignored when SYMFILE_MAINLINE bit is set in ADD_FLAGS. diff --git a/gdb/symtab.h b/gdb/symtab.h index dc65409..5300383 100644 --- a/gdb/symtab.h +++ b/gdb/symtab.h @@ -1044,7 +1044,7 @@ struct symbol_block_ops register, the CFA as defined by DWARF unwinding information, ... So this specific method is supposed to compute the frame base address such - as for nested fuctions, the static link computes the same address. For + as for nested functions, the static link computes the same address. For instance, considering DWARF debugging information, the static link is computed with DW_AT_static_link and this method must be used to compute the corresponding DW_AT_frame_base attribute. */ diff --git a/gdb/target.c b/gdb/target.c index 78bdfeb..7c9befc 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -140,7 +140,7 @@ static int show_memory_breakpoints = 0; /* These globals control whether GDB attempts to perform these operations; they are useful for targets that need to prevent - inadvertant disruption, such as in non-stop mode. */ + inadvertent disruption, such as in non-stop mode. */ bool may_write_registers = true; diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index b4228ce..417c289 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,15 @@ +2019-10-26 Tom de Vries <tdevries@suse.de> + + * gdb.base/bigcore.c: Fix typos in comments. + * gdb.base/ctf-ptype.c: Same. + * gdb.base/long_long.c: Same. + * gdb.dwarf2/dw2-op-out-param.S: Same. + * gdb.python/py-evthreads.c: Same. + * gdb.reverse/i387-stack-reverse.c: Same. + * gdb.trace/tfile.c: Same. + * lib/compiler.c: Same. + * lib/compiler.cc: Same. + 2019-10-25 Tom de Vries <tdevries@suse.de> * gdb.reverse/sigall-precsave.exp: Use -wrap and $gdb_test_name in diff --git a/gdb/testsuite/gdb.base/bigcore.c b/gdb/testsuite/gdb.base/bigcore.c index 0507744..3ac7ca9 100644 --- a/gdb/testsuite/gdb.base/bigcore.c +++ b/gdb/testsuite/gdb.base/bigcore.c @@ -121,7 +121,7 @@ maximize_rlimit (int resource, const char *prefix) print_string ("\n"); } -/* Maintain a doublely linked list. */ +/* Maintain a doubly linked list. */ struct list { struct list *next; @@ -194,7 +194,7 @@ main () } /* Compute an initial chunk size. The math is dodgy but it works - for the moment. Perhaphs there's a constant around somewhere. + for the moment. Perhaps there's a constant around somewhere. Limit this to max_core_size bytes - no point in trying to allocate more than can be written to the corefile. */ { diff --git a/gdb/testsuite/gdb.base/ctf-ptype.c b/gdb/testsuite/gdb.base/ctf-ptype.c index 2d2e881..5d3580d 100644 --- a/gdb/testsuite/gdb.base/ctf-ptype.c +++ b/gdb/testsuite/gdb.base/ctf-ptype.c @@ -17,7 +17,7 @@ /* * Test file with lots of different types, for testing the - * "ptype" command on CTF data. It's devired from ptype.c. + * "ptype" command on CTF data. It's derived from ptype.c. */ /* diff --git a/gdb/testsuite/gdb.base/long_long.c b/gdb/testsuite/gdb.base/long_long.c index 9d0393d..2451585 100644 --- a/gdb/testsuite/gdb.base/long_long.c +++ b/gdb/testsuite/gdb.base/long_long.c @@ -80,7 +80,7 @@ int known_types() int main() { - /* Pack Byte, Half, Word and Giant arrays with byte-orderd values. + /* Pack Byte, Half, Word and Giant arrays with byte-ordered values. That way "(gdb) x" gives the same output on different architectures. */ pack (b, 1, 2); diff --git a/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S b/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S index 09b5155..169eb04 100644 --- a/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S +++ b/gdb/testsuite/gdb.dwarf2/dw2-op-out-param.S @@ -28,7 +28,7 @@ like gdb to behave in a user friendly, and helpful way when presented with such dwarf. */ -/* There are 4 test cases in this assembler file. In each case funcion +/* There are 4 test cases in this assembler file. In each case function main calls each test function in turn, each test case then calls the breakpt function. diff --git a/gdb/testsuite/gdb.python/py-evthreads.c b/gdb/testsuite/gdb.python/py-evthreads.c index 2f55815..f824945 100644 --- a/gdb/testsuite/gdb.python/py-evthreads.c +++ b/gdb/testsuite/gdb.python/py-evthreads.c @@ -33,7 +33,7 @@ void* thread3 (void* d) void* thread2 (void* d) { /* Do not quit thread3 asynchronously wrt thread2 stop - wait first on - thread3_id to stop. It would complicate testcase receiption of the + thread3_id to stop. It would complicate testcase reception of the events. */ pthread_create (&thread3_id, NULL, thread3, NULL); pthread_join (thread3_id, NULL); diff --git a/gdb/testsuite/gdb.reverse/i387-stack-reverse.c b/gdb/testsuite/gdb.reverse/i387-stack-reverse.c index 8c690f7..93d3f46 100644 --- a/gdb/testsuite/gdb.reverse/i387-stack-reverse.c +++ b/gdb/testsuite/gdb.reverse/i387-stack-reverse.c @@ -14,7 +14,7 @@ void empty_fpu_stack() "ffree %st(7)"); } -/* tests floating point arithmatic */ +/* tests floating point arithmetic */ void test_arith_floats() { diff --git a/gdb/testsuite/gdb.trace/tfile.c b/gdb/testsuite/gdb.trace/tfile.c index 51d464e..402a2d5 100644 --- a/gdb/testsuite/gdb.trace/tfile.c +++ b/gdb/testsuite/gdb.trace/tfile.c @@ -176,7 +176,7 @@ write_basic_trace_file (void) /* Make up a simulated trace buffer. */ /* (Encapsulate better if we're going to do lots of this; note that - buffer endianness is the target program's enddianness.) */ + buffer endianness is the target program's endianness.) */ trptr = trbuf; tfile_write_16 (1); diff --git a/gdb/testsuite/lib/compiler.c b/gdb/testsuite/lib/compiler.c index 9b8400a..917b4ee 100644 --- a/gdb/testsuite/lib/compiler.c +++ b/gdb/testsuite/lib/compiler.c @@ -45,7 +45,7 @@ set compiler_info [join {gcc __GNUC__ __GNUC_MINOR__ "unknown"} -] #if defined (__xlc__) /* IBM'x xlc compiler. NOTE: __xlc__ expands to a double quoted string of four - numbers seperated by '.'s: currently "7.0.0.0" */ + numbers separated by '.'s: currently "7.0.0.0" */ set need_a_set [regsub -all {\.} [join {xlc __xlc__} -] - compiler_info] #endif diff --git a/gdb/testsuite/lib/compiler.cc b/gdb/testsuite/lib/compiler.cc index 9faeaa2..a9e69ba 100755 --- a/gdb/testsuite/lib/compiler.cc +++ b/gdb/testsuite/lib/compiler.cc @@ -33,7 +33,7 @@ set compiler_info [join {gcc __GNUC__ __GNUC_MINOR__ "unknown"} -] #if defined (__xlc__) /* IBM'x xlc compiler. NOTE: __xlc__ expands to a double quoted string of four - numbers seperated by '.'s: currently "7.0.0.0" */ + numbers separated by '.'s: currently "7.0.0.0" */ set need_a_set [regsub -all {\.} [join {xlc __xlc__} -] - compiler_info] #endif diff --git a/gdb/tracepoint.c b/gdb/tracepoint.c index 4db7060..d627839 100644 --- a/gdb/tracepoint.c +++ b/gdb/tracepoint.c @@ -2136,7 +2136,7 @@ tfind_1 (enum trace_find_type type, int num, DON'T give an error, but DO change the state of traceframe_number etc. to invalid. - The rationalle is that if you typed the command, you + The rationale is that if you typed the command, you might just have committed a typo or something, and you'd like to NOT lose your current debugging state. However if you're in a user-defined command or especially in a diff --git a/gdb/tui/tui-data.h b/gdb/tui/tui-data.h index 4710c76..8af1573 100644 --- a/gdb/tui/tui-data.h +++ b/gdb/tui/tui-data.h @@ -181,7 +181,7 @@ public: { } - /* Set whether this window is highglighted. */ + /* Set whether this window is highlighted. */ void set_highlight (bool highlight) { is_highlighted = highlight; diff --git a/gdb/tui/tui-io.c b/gdb/tui/tui-io.c index 6bb495b..964f2e3 100644 --- a/gdb/tui/tui-io.c +++ b/gdb/tui/tui-io.c @@ -90,7 +90,7 @@ key_is_start_sequence (int ch) be garbled. This is implemented with a pipe that TUI reads and readline writes to. A gdb input handler is created so that reading the pipe is handled automatically. This will probably not work on - non-Unix platforms. The best fix is to make readline clean enougth + non-Unix platforms. The best fix is to make readline clean enough so that is never write on stdout. Note SCz/2002-09-01: we now use more readline hooks and it seems diff --git a/gdb/tui/tui-win.c b/gdb/tui/tui-win.c index 41c61f1..6df5ea2 100644 --- a/gdb/tui/tui-win.c +++ b/gdb/tui/tui-win.c @@ -1050,7 +1050,7 @@ tui_adjust_win_heights (struct tui_win_info *primary_win_info, second_win = *(tui_source_windows ().begin ()); } if (primary_win_info == TUI_CMD_WIN) - { /* Split the change in height accross the 1st & 2nd + { /* Split the change in height across the 1st & 2nd windows, adjusting them as well. */ /* Subtract the locator. */ int first_split_diff = diff / 2; @@ -1072,7 +1072,7 @@ tui_adjust_win_heights (struct tui_win_info *primary_win_info, second_split_diff++; } } - /* Make sure that the minimum hieghts are + /* Make sure that the minimum heights are honored. */ while ((first_win->height + first_split_diff) < 3) { diff --git a/gdb/tui/tui.c b/gdb/tui/tui.c index e765d58..1568351 100644 --- a/gdb/tui/tui.c +++ b/gdb/tui/tui.c @@ -405,7 +405,7 @@ tui_enable (void) if (tui_active) return; - /* To avoid to initialize curses when gdb starts, there is a defered + /* To avoid to initialize curses when gdb starts, there is a deferred curses initialization. This initialization is made only once and the first time the curses mode is entered. */ if (tui_finish_init) diff --git a/gdb/unittests/rsp-low-selftests.c b/gdb/unittests/rsp-low-selftests.c index e316a22..541d030 100644 --- a/gdb/unittests/rsp-low-selftests.c +++ b/gdb/unittests/rsp-low-selftests.c @@ -34,7 +34,7 @@ static void test_hex2bin_byte_vector () bv = hex2bin (""); SELF_CHECK (bv.size () == 0); - /* Test a well-formated hex string. */ + /* Test a well-formatted hex string. */ bv = hex2bin ("abcd01"); SELF_CHECK (bv.size () == 3); SELF_CHECK (bv[0] == 0xab); diff --git a/gdb/user-regs.h b/gdb/user-regs.h index 0714b06..06aee24 100644 --- a/gdb/user-regs.h +++ b/gdb/user-regs.h @@ -36,7 +36,7 @@ /* TODO: cagney/2003-06-27: Need to think more about how these registers are added, read, and modified. At present they are kind of assumed to be read-only. Should it, for instance, return a - register descriptor that contains all the relvent access methods. */ + register descriptor that contains all the relevant access methods. */ struct frame_info; struct gdbarch; diff --git a/gdb/utils.c b/gdb/utils.c index 3afb8e5..0c133d1 100644 --- a/gdb/utils.c +++ b/gdb/utils.c @@ -1281,7 +1281,7 @@ init_page_info (void) || getenv ("EMACS") || getenv ("INSIDE_EMACS")) { /* The number of lines per page is not mentioned in the terminal - description or EMACS evironment variable is set. This probably + description or EMACS environment variable is set. This probably means that paging is not useful, so disable paging. */ lines_per_page = UINT_MAX; } @@ -3098,7 +3098,7 @@ parse_pid_to_attach (const char *args) return pid; } -/* Substitute all occurences of string FROM by string TO in *STRINGP. *STRINGP +/* Substitute all occurrences of string FROM by string TO in *STRINGP. *STRINGP must come from xrealloc-compatible allocator and it may be updated. FROM needs to be delimited by IS_DIR_SEPARATOR or DIRNAME_SEPARATOR (or be located at the start or end of *STRINGP. */ diff --git a/gdb/utils.h b/gdb/utils.h index e837621..79c8a6f 100644 --- a/gdb/utils.h +++ b/gdb/utils.h @@ -125,7 +125,7 @@ void reset_prompt_for_continue_wait_time (void); /* Return the time spent in prompt_for_continue. */ std::chrono::steady_clock::duration get_prompt_for_continue_wait_time (); -/* Parsing utilites. */ +/* Parsing utilities. */ extern int parse_pid_to_attach (const char *args); diff --git a/gdb/valarith.c b/gdb/valarith.c index 8f3c502..7f1b24f 100644 --- a/gdb/valarith.c +++ b/gdb/valarith.c @@ -335,7 +335,7 @@ value_user_defined_op (struct value **argp, gdb::array_view<value *> args, arg1.operator @ (arg1,arg2) and return that value (where '@' is any binary operator which is legal for GNU C++). - OP is the operatore, and if it is BINOP_ASSIGN_MODIFY, then OTHEROP + OP is the operator, and if it is BINOP_ASSIGN_MODIFY, then OTHEROP is the opcode saying how to modify it. Otherwise, OTHEROP is unused. */ diff --git a/gdb/valops.c b/gdb/valops.c index e197cef..bfdd781 100644 --- a/gdb/valops.c +++ b/gdb/valops.c @@ -2820,7 +2820,7 @@ find_overload_match (gdb::array_view<value *> args, contained in QUALIFIED_NAME until it either finds a good match or runs out of namespaces. It stores the overloaded functions in *OLOAD_SYMS, and the badness vector in *OLOAD_CHAMP_BV. If NO_ADL, - argument dependent lookup is not performned. */ + argument dependent lookup is not performed. */ static int find_oload_champ_namespace (gdb::array_view<value *> args, diff --git a/gdb/valprint.c b/gdb/valprint.c index 5616db1..e0eb8e1 100644 --- a/gdb/valprint.c +++ b/gdb/valprint.c @@ -1720,7 +1720,7 @@ print_decimal_chars (struct ui_file *stream, const gdb_byte *valaddr, * decimal. * * Given a hex number (in nibbles) as XYZ, we start by taking X and - * decemalizing it as "x1 x2" in two decimal nibbles. Then we multiply + * decimalizing it as "x1 x2" in two decimal nibbles. Then we multiply * the nibbles by 16, add Y and re-decimalize. Repeat with Z. * * The trick is that "digits" holds a base-10 number, but sometimes diff --git a/gdb/valprint.h b/gdb/valprint.h index 6d57554..24b731a 100644 --- a/gdb/valprint.h +++ b/gdb/valprint.h @@ -260,7 +260,7 @@ extern void print_command_completer (struct cmd_list_element *ignore, symbols won't be used except in instances where no symbol was found; otherwise, a minsym might be used in some instances (mostly involving function with non-contiguous address ranges). Return - 0 in case of success, when all the info in the OUT paramters is + 0 in case of success, when all the info in the OUT parameters is valid. Return 1 otherwise. */ extern int build_address_symbolic (struct gdbarch *, diff --git a/gdb/value.c b/gdb/value.c index cbdb5f7..47e647a 100644 --- a/gdb/value.c +++ b/gdb/value.c @@ -2153,7 +2153,7 @@ value_of_internalvar (struct gdbarch *gdbarch, struct internalvar *var) on this value go back to affect the original internal variable. Do not do this for INTERNALVAR_MAKE_VALUE variables, as those have - no underlying modifyable state in the internal variable. + no underlying modifiable state in the internal variable. Likewise, if the variable's value is a computed lvalue, we want references to it to produce another computed lvalue, where diff --git a/gdb/value.h b/gdb/value.h index 0d75eaa..2b5d784 100644 --- a/gdb/value.h +++ b/gdb/value.h @@ -568,7 +568,7 @@ extern void mark_value_bits_unavailable (struct value *value, Optimized-out contents are equal to optimized-out contents, and are not equal to non-optimized-out contents. - Unavailable contente are equal to unavailable contents, and are not + Unavailable contents are equal to unavailable contents, and are not equal to non-unavailable contents. For example, if 'x's represent an unavailable byte, and 'V' and 'Z' diff --git a/gdb/varobj.c b/gdb/varobj.c index 10d5a67..b7ee619 100644 --- a/gdb/varobj.c +++ b/gdb/varobj.c @@ -1299,7 +1299,7 @@ install_new_value (struct varobj *var, struct value *value, bool initial) { /* For variables that are frozen, or are children of frozen variables, we don't do fetch on initial assignment. - For non-initial assignemnt we do the fetch, since it means we're + For non-initial assignment we do the fetch, since it means we're explicitly asked to compare the new value with the old one. */ intentionally_not_fetched = true; } diff --git a/gdb/x86-nat.h b/gdb/x86-nat.h index baa4218..36b061b 100644 --- a/gdb/x86-nat.h +++ b/gdb/x86-nat.h @@ -1,6 +1,6 @@ /* Native-dependent code for x86 (i386 and x86-64). - Low level functions to implement Oeprating System specific + Low level functions to implement Operating System specific code to manipulate x86 debug registers. Copyright (C) 2009-2019 Free Software Foundation, Inc. diff --git a/gdb/xtensa-tdep.c b/gdb/xtensa-tdep.c index 4af2ccd..2a430bf 100644 --- a/gdb/xtensa-tdep.c +++ b/gdb/xtensa-tdep.c @@ -638,7 +638,7 @@ xtensa_pseudo_register_write (struct gdbarch *gdbarch, DEBUGTRACE ("xtensa_pseudo_register_write (... regnum = %d (%s) ...)\n", regnum, xtensa_register_name (gdbarch, regnum)); - /* Renumber register, if aliase a0..a15 on Windowed ABI. */ + /* Renumber register, if aliases a0..a15 on Windowed ABI. */ if (gdbarch_tdep (gdbarch)->isa_use_windowed_registers && (regnum >= gdbarch_tdep (gdbarch)->a0_base) && (regnum <= gdbarch_tdep (gdbarch)->a0_base + 15)) |