Patchwork [review] Fix typos in comments

login
register
mail settings
Submitter Simon Marchi (Code Review)
Date Oct. 17, 2019, 8:17 a.m.
Message ID <gerrit.1571300237000.I6047967abd2d51c9000dea15184d19f4e952c3ff@gnutoolchain-gerrit.osci.io>
Download mbox | patch
Permalink /patch/35084/
State New
Headers show

Comments

Simon Marchi (Code Review) - Oct. 17, 2019, 8:17 a.m.
Tom de Vries has uploaded a new change for review.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130
......................................................................

[gdb] Fix typos in comments

Fix typos in comments.  NFC.

Tested on x86_64-linux.

gdb/ChangeLog:

2019-10-17  Tom de Vries  <tdevries@suse.de>

	* arm-nbsd-nat.c: Fix typos in comments.
	* arm-tdep.c: Same.
	* darwin-nat-info.c: Same.
	* dwarf2read.c: Same.
	* elfread.c: Same.
	* event-top.c: Same.
	* findvar.c: Same.
	* gdbtypes.c: Same.
	* hppa-tdep.c: Same.
	* i386-tdep.c: Same.
	* jit.c: Same.
	* main.c: Same.
	* mdebugread.c: Same.
	* moxie-tdep.c: Same.
	* nto-procfs.c: Same.
	* osabi.c: Same.
	* ppc-linux-tdep.c: Same.
	* remote.c: Same.
	* riscv-tdep.c: Same.
	* s390-tdep.c: Same.
	* sh-tdep.c: Same.
	* sparc-linux-tdep.c: Same.
	* sparc-nat.c: Same.
	* stack.c: Same.
	* target-descriptions.c: Same.
	* top.c: Same.
	* varobj.c: Same.

Change-Id: I6047967abd2d51c9000dea15184d19f4e952c3ff
---
M gdb/arm-nbsd-nat.c
M gdb/arm-tdep.c
M gdb/darwin-nat-info.c
M gdb/dwarf2read.c
M gdb/elfread.c
M gdb/event-top.c
M gdb/findvar.c
M gdb/gdbtypes.c
M gdb/hppa-tdep.c
M gdb/i386-tdep.c
M gdb/jit.c
M gdb/main.c
M gdb/mdebugread.c
M gdb/moxie-tdep.c
M gdb/nto-procfs.c
M gdb/osabi.c
M gdb/ppc-linux-tdep.c
M gdb/remote.c
M gdb/riscv-tdep.c
M gdb/s390-tdep.c
M gdb/sh-tdep.c
M gdb/sparc-linux-tdep.c
M gdb/sparc-nat.c
M gdb/stack.c
M gdb/target-descriptions.c
M gdb/top.c
M gdb/varobj.c
27 files changed, 51 insertions(+), 51 deletions(-)
Simon Marchi (Code Review) - Oct. 17, 2019, 10:04 a.m.
Konrad Kleine has posted comments on this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

LGTM except for a minor question.

https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130/1/gdb/darwin-nat-info.c 
File gdb/darwin-nat-info.c:

https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130/1/gdb/darwin-nat-info.c@813 
PS1, Line 813: 	  /* FIXME: This need a privileged host port!  */
Shouldn't it say "This needs a privileged host port!" ?
Simon Marchi (Code Review) - Oct. 17, 2019, 10:17 a.m.
Konrad Kleine has posted comments on this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130
......................................................................


Patch Set 2:

(1 comment)

Thanks for incorporating my suggestion.

https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130/1/gdb/darwin-nat-info.c 
File gdb/darwin-nat-info.c:

https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130/1/gdb/darwin-nat-info.c@813 
PS1, Line 813: 	  /* FIXME: This need a privileged host port!  */
> Shouldn't it say "This needs a privileged host port!" ?
Done
Simon Marchi (Code Review) - Oct. 17, 2019, 10:18 a.m.
Konrad Kleine has posted comments on this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130
......................................................................


Patch Set 2: Code-Review+1
Simon Marchi (Code Review) - Oct. 17, 2019, 10:20 a.m.
Tom de Vries has posted comments on this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130
......................................................................


Patch Set 2:

(1 comment)

> Patch Set 1: Code-Review+1
> 
> (1 comment)
> 
> LGTM except for a minor question.

Fixed.

https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130/1/gdb/darwin-nat-info.c 
File gdb/darwin-nat-info.c:

https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130/1/gdb/darwin-nat-info.c@813 
PS1, Line 813: 	  /* FIXME: This need a privileged host port!  */
> Done
It should, thanks for noticing.

I only did a check for invalid words, not grammar, so I didn't catch this.

Fixed in Patchset 2.
Simon Marchi (Code Review) - Oct. 17, 2019, 2:59 p.m.
Simon Marchi has posted comments on this change.

Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/130
......................................................................


Patch Set 2: Code-Review+2

LGTM, thanks!  If you ever spot other such typos, you can definitely consider them as obvious changes.

Patch

diff --git a/gdb/arm-nbsd-nat.c b/gdb/arm-nbsd-nat.c
index 5160bfd..82c8ad5 100644
--- a/gdb/arm-nbsd-nat.c
+++ b/gdb/arm-nbsd-nat.c
@@ -437,7 +437,7 @@ 
 
 static struct core_fns arm_netbsd_elfcore_fns =
 {
-  bfd_target_elf_flavour,		/* core_flovour.  */
+  bfd_target_elf_flavour,		/* core_flavour.  */
   default_check_format,			/* check_format.  */
   default_core_sniffer,			/* core_sniffer.  */
   fetch_elfcore_registers,		/* core_read_registers.  */
diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c
index 5c1476a..fe5605c 100644
--- a/gdb/arm-tdep.c
+++ b/gdb/arm-tdep.c
@@ -6809,7 +6809,7 @@ 
 	      if (bit_4 == 0) /* STC/STC2.  */
 		return thumb_copy_unmodified_32bit (gdbarch, insn1, insn2,
 						    "stc/stc2", dsc);
-	      else /* LDC/LDC2 {literal, immeidate}.  */
+	      else /* LDC/LDC2 {literal, immediate}.  */
 		return thumb2_copy_copro_load_store (gdbarch, insn1, insn2,
 						     regs, dsc);
 	    }
@@ -6954,7 +6954,7 @@ 
   return 0;
 }
 
-/* Copy Thumb cbnz/cbz insruction.  */
+/* Copy Thumb cbnz/cbz instruction.  */
 
 static int
 thumb_copy_cbnz_cbz (struct gdbarch *gdbarch, uint16_t insn1,
@@ -7329,7 +7329,7 @@ 
 	  case 0:
 	    if (bit (insn1, 6))
 	      {
-		/* Load/store {dual, execlusive}, table branch.  */
+		/* Load/store {dual, exclusive}, table branch.  */
 		if (bits (insn1, 7, 8) == 1 && bits (insn1, 4, 5) == 1
 		    && bits (insn2, 5, 7) == 0)
 		  err = thumb2_copy_table_branch (gdbarch, insn1, insn2, regs,
@@ -7390,7 +7390,7 @@ 
 		err = thumb_copy_unmodified_32bit (gdbarch, insn1, insn2,
 						   "dp/pb", dsc);
 	    }
-	  else /* Data processing (modified immeidate) */
+	  else /* Data processing (modified immediate) */
 	    err = thumb_copy_unmodified_32bit (gdbarch, insn1, insn2,
 					       "dp/mi", dsc);
 	}
diff --git a/gdb/darwin-nat-info.c b/gdb/darwin-nat-info.c
index 4469cd5..3ad0aff 100644
--- a/gdb/darwin-nat-info.c
+++ b/gdb/darwin-nat-info.c
@@ -810,7 +810,7 @@ 
 	}
       else if (strcmp (args, "host") == 0)
 	{
-	  /* FIXME: This need a privilegied host port!  */
+	  /* FIXME: This need a privileged host port!  */
 	  kret = host_get_exception_ports
 	    (darwin_host_self, EXC_MASK_ALL, info.masks,
 	     &info.count, info.ports, info.behaviors, info.flavors);
diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c
index a78f818..5c705b0 100644
--- a/gdb/dwarf2read.c
+++ b/gdb/dwarf2read.c
@@ -1486,7 +1486,7 @@ 
   cu_partial_die_info (struct dwarf2_cu *cu, struct partial_die_info *pdi)
     : cu (cu),
       pdi (pdi)
-  { /* Nothhing.  */ }
+  { /* Nothing.  */ }
 
 private:
   cu_partial_die_info () = delete;
@@ -14942,7 +14942,7 @@ 
   return cu->producer_is_codewarrior;
 }
 
-/* Return the default accessibility type if it is not overriden by
+/* Return the default accessibility type if it is not overridden by
    DW_AT_accessibility.  */
 
 static enum dwarf_access_attribute
diff --git a/gdb/elfread.c b/gdb/elfread.c
index 53bdd35..1b5b4e0 100644
--- a/gdb/elfread.c
+++ b/gdb/elfread.c
@@ -1433,7 +1433,7 @@ 
   elf_symfile_read,		/* read a symbol file into symtab */
   NULL,				/* sym_read_psymbols */
   elf_symfile_finish,		/* finished with file, cleanup */
-  default_symfile_offsets,	/* Translate ext. to int. relocatin */
+  default_symfile_offsets,	/* Translate ext. to int. relocation */
   elf_symfile_segments,		/* Get segment information from a file.  */
   NULL,
   default_symfile_relocate,	/* Relocate a debug section.  */
@@ -1450,7 +1450,7 @@ 
   elf_symfile_read,		/* read a symbol file into symtab */
   NULL,				/* sym_read_psymbols */
   elf_symfile_finish,		/* finished with file, cleanup */
-  default_symfile_offsets,	/* Translate ext. to int. relocatin */
+  default_symfile_offsets,	/* Translate ext. to int. relocation */
   elf_symfile_segments,		/* Get segment information from a file.  */
   NULL,
   default_symfile_relocate,	/* Relocate a debug section.  */
diff --git a/gdb/event-top.c b/gdb/event-top.c
index 0b05b2f..0396dbc 100644
--- a/gdb/event-top.c
+++ b/gdb/event-top.c
@@ -421,7 +421,7 @@ 
 }
 
 /* Return the top level prompt, as specified by "set prompt", possibly
-   overriden by the python gdb.prompt_hook hook, and then composed
+   overridden by the python gdb.prompt_hook hook, and then composed
    with the prompt prefix and suffix (annotations).  */
 
 static std::string
@@ -1152,7 +1152,7 @@ 
   signal (sig, handle_sigfpe);
 }
 
-/* Event loop will call this functin to process a SIGFPE.  */
+/* Event loop will call this function to process a SIGFPE.  */
 static void
 async_float_handler (gdb_client_data arg)
 {
diff --git a/gdb/findvar.c b/gdb/findvar.c
index e52ec8d..3c8e631 100644
--- a/gdb/findvar.c
+++ b/gdb/findvar.c
@@ -218,7 +218,7 @@ 
    bytes.  If SOURCE_SIZE is greater than DEST_SIZE, then truncate the most
    significant bytes.  If SOURCE_SIZE is less than DEST_SIZE then either sign
    or zero extended according to IS_SIGNED.  Values are stored in memory with
-   endianess BYTE_ORDER.  */
+   endianness BYTE_ORDER.  */
 
 void
 copy_integer_to_size (gdb_byte *dest, int dest_size, const gdb_byte *source,
diff --git a/gdb/gdbtypes.c b/gdb/gdbtypes.c
index a2b81c8..fd1c765 100644
--- a/gdb/gdbtypes.c
+++ b/gdb/gdbtypes.c
@@ -2733,7 +2733,7 @@ 
     }
 }
 
-/* Ensure it is in .rodata (if available) by workarounding GCC PR 44690.  */
+/* Ensure it is in .rodata (if available) by working around GCC PR 44690.  */
 const struct cplus_struct_type cplus_struct_default = { };
 
 void
diff --git a/gdb/hppa-tdep.c b/gdb/hppa-tdep.c
index 249d5c9..c307de8 100644
--- a/gdb/hppa-tdep.c
+++ b/gdb/hppa-tdep.c
@@ -88,7 +88,7 @@ 
 				gdb::noop_deleter<hppa_objfile_private>>
   hppa_objfile_priv_data;
 
-/* Get at various relevent fields of an instruction word.  */
+/* Get at various relevant fields of an instruction word.  */
 #define MASK_5 0x1f
 #define MASK_11 0x7ff
 #define MASK_14 0x3fff
diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c
index ccec6d1..3eb6fa4 100644
--- a/gdb/i386-tdep.c
+++ b/gdb/i386-tdep.c
@@ -2956,7 +2956,7 @@ 
 	|| code == TYPE_CODE_UNION
 	|| code == TYPE_CODE_ARRAY)
        && !i386_reg_struct_return_p (gdbarch, type))
-      /* Complex double and long double uses the struct return covention.  */
+      /* Complex double and long double uses the struct return convention.  */
       || (code == TYPE_CODE_COMPLEX && TYPE_LENGTH (type) == 16)
       || (code == TYPE_CODE_COMPLEX && TYPE_LENGTH (type) == 24)
       /* 128-bit decimal float uses the struct return convention.  */
diff --git a/gdb/jit.c b/gdb/jit.c
index af01b4d..08fd862 100644
--- a/gdb/jit.c
+++ b/gdb/jit.c
@@ -76,7 +76,7 @@ 
   ULONGEST size;
 };
 
-/* Openning the file is a no-op.  */
+/* Opening the file is a no-op.  */
 
 static void *
 mem_bfd_iovec_open (struct bfd *abfd, void *open_closure)
diff --git a/gdb/main.c b/gdb/main.c
index 7f87363..a77d6ec 100644
--- a/gdb/main.c
+++ b/gdb/main.c
@@ -911,7 +911,7 @@ 
     }
 
   /* Lookup gdbinit files.  Note that the gdbinit file name may be
-     overriden during file initialization, so get_init_files should be
+     overridden during file initialization, so get_init_files should be
      called after gdb_init.  */
   std::string system_gdbinit;
   std::string home_gdbinit;
diff --git a/gdb/mdebugread.c b/gdb/mdebugread.c
index eed7146..e76fe9a 100644
--- a/gdb/mdebugread.c
+++ b/gdb/mdebugread.c
@@ -187,7 +187,7 @@ 
 
 static struct ecoff_debug_info *debug_info;
 
-/* Pointer to current file decriptor record, and its index.  */
+/* Pointer to current file descriptor record, and its index.  */
 
 static FDR *cur_fdr;
 static int cur_fd;
@@ -3729,7 +3729,7 @@ 
 	  if (rh == f_idx)
 	    continue;
 
-	  /* Do not add to dependeny list if psymtab was empty.  */
+	  /* Do not add to dependency list if psymtab was empty.  */
 	  if (fdr_to_pst[rh].pst == NULL)
 	    continue;
 	  pst->dependencies[pst->number_of_dependencies++]
diff --git a/gdb/moxie-tdep.c b/gdb/moxie-tdep.c
index 251c267..81fdb71 100644
--- a/gdb/moxie-tdep.c
+++ b/gdb/moxie-tdep.c
@@ -271,7 +271,7 @@ 
 };
 
 /* Read an unsigned integer from the inferior, and adjust
-   endianess.  */
+   endianness.  */
 static ULONGEST
 moxie_process_readu (CORE_ADDR addr, gdb_byte *buf,
 		     int length, enum bfd_endian byte_order)
diff --git a/gdb/nto-procfs.c b/gdb/nto-procfs.c
index b9866c2..0a199e7 100644
--- a/gdb/nto-procfs.c
+++ b/gdb/nto-procfs.c
@@ -605,7 +605,7 @@ 
 	      if (strcmp (map.info.path, printme.name))
 		continue;
 
-	      /* Lower debug_vaddr is always text, if nessessary, swap.  */
+	      /* Lower debug_vaddr is always text, if necessary, swap.  */
 	      if ((int) map.info.vaddr < (int) printme.text.debug_vaddr)
 		{
 		  memcpy (&(printme.data), &(printme.text),
diff --git a/gdb/osabi.c b/gdb/osabi.c
index cdf72a7..cc1652c 100644
--- a/gdb/osabi.c
+++ b/gdb/osabi.c
@@ -415,7 +415,7 @@ 
   /* If this assertion triggers, increase MAX_NOTESZ.  */
   gdb_assert (notesz <= MAX_NOTESZ);
 
-  /* Check whether SECT is big enough to comtain the complete note.  */
+  /* Check whether SECT is big enough to contain the complete note.  */
   if (notesz > bfd_section_size (sect))
     return 0;
 
diff --git a/gdb/ppc-linux-tdep.c b/gdb/ppc-linux-tdep.c
index ce3c727..e692cff 100644
--- a/gdb/ppc-linux-tdep.c
+++ b/gdb/ppc-linux-tdep.c
@@ -657,7 +657,7 @@ 
    general-purpose regsets for 32-bit, 64-bit big-endian, and 64-bit
    little endian targets.  The ptrace and core file buffers for 64-bit
    targets use 8-byte fields for the 4-byte registers, and the
-   position of the register in the fields depends on the endianess.
+   position of the register in the fields depends on the endianness.
    The 32-bit regmap is the same for both endian types because the
    fields are all 4-byte long.
 
@@ -761,7 +761,7 @@ 
 
 /* Regmaps for the Hardware Transactional Memory checkpointed vector
    regsets, for big and little endian targets.  The position of the
-   4-byte VSCR in its 16-byte field depends on the endianess.  */
+   4-byte VSCR in its 16-byte field depends on the endianness.  */
 
 static const struct regcache_map_entry ppc32_le_regmap_cvmx[] =
   {
diff --git a/gdb/remote.c b/gdb/remote.c
index 73b510d..f616569 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -141,7 +141,7 @@ 
   bool S = false;
 };
 
-/* About this many threadisds fit in a packet.  */
+/* About this many threadids fit in a packet.  */
 
 #define MAXTHREADLISTRESULTS 32
 
@@ -3250,7 +3250,7 @@ 
     }
   copy_threadref (&info->threadid, &ref);
 
-  /* Loop on tagged fields , try to bail if somthing goes wrong.  */
+  /* Loop on tagged fields , try to bail if something goes wrong.  */
 
   /* Packets are terminated with nulls.  */
   while ((pkt < limit) && mask && *pkt)
@@ -3389,7 +3389,7 @@ 
   struct remote_state *rs = get_remote_state ();
   int result = 1;
 
-  /* Trancate result limit to be smaller than the packet size.  */
+  /* Truncate result limit to be smaller than the packet size.  */
   if ((((result_limit + 1) * BUF_THREAD_ID_SIZE) + 10)
       >= get_remote_packet_size ())
     result_limit = (get_remote_packet_size () / BUF_THREAD_ID_SIZE) - 2;
@@ -3411,8 +3411,8 @@ 
   if (!threadmatch (&rs->echo_nextthread, nextthread))
     {
       /* FIXME: This is a good reason to drop the packet.  */
-      /* Possably, there is a duplicate response.  */
-      /* Possabilities :
+      /* Possibly, there is a duplicate response.  */
+      /* Possibilities :
          retransmit immediatly - race conditions
          retransmit after timeout - yes
          exit
@@ -11283,7 +11283,7 @@ 
 {
   char hexid[20];
 
-  pack_threadid (&hexid[0], ref);	/* Convert threead id into hex.  */
+  pack_threadid (&hexid[0], ref);	/* Convert thread id into hex.  */
   hexid[16] = 0;
   printf_filtered ("%s  %s\n", title, (&hexid[0]));
 }
diff --git a/gdb/riscv-tdep.c b/gdb/riscv-tdep.c
index 031fc8f..810b834 100644
--- a/gdb/riscv-tdep.c
+++ b/gdb/riscv-tdep.c
@@ -1010,7 +1010,7 @@ 
       LUI,
       SD,
       SW,
-      /* These are needed for software breakopint support.  */
+      /* These are needed for software breakpoint support.  */
       JAL,
       JALR,
       BEQ,
diff --git a/gdb/s390-tdep.c b/gdb/s390-tdep.c
index 329a122..a4aae07 100644
--- a/gdb/s390-tdep.c
+++ b/gdb/s390-tdep.c
@@ -2116,7 +2116,7 @@ 
 
 /* Frame unwinding.  */
 
-/* Implmement the stack_frame_destroyed_p gdbarch method.  */
+/* Implement the stack_frame_destroyed_p gdbarch method.  */
 
 static int
 s390_stack_frame_destroyed_p (struct gdbarch *gdbarch, CORE_ADDR pc)
@@ -5311,7 +5311,7 @@ 
 	case 0xe383: /* MSGC - multiply single 64x64mem -> 64 */
 	case 0xe398: /* ALC - add logical with carry */
 	case 0xe399: /* SLB - subtract logical with borrow */
-	case 0xe727: /* LCBB - load count to block bounduary */
+	case 0xe727: /* LCBB - load count to block boundary */
 	case 0xeb81: /* ICMY - insert characters under mask */
 	case 0xebdc: /* SRAK - shift left single */
 	case 0xebdd: /* SLAK - shift left single */
@@ -5537,7 +5537,7 @@ 
 	case 0xe704: /* VLLEZ - vector load logical element and zero */
 	case 0xe705: /* VLREP - vector load and replicate */
 	case 0xe706: /* VL - vector load */
-	case 0xe707: /* VLBB - vector load to block bounduary */
+	case 0xe707: /* VLBB - vector load to block boundary */
 	case 0xe712: /* VGEG - vector gather element */
 	case 0xe713: /* VGEF - vector gather element */
 	case 0xe722: /* VLVG - vector load vr element from gr */
@@ -6238,7 +6238,7 @@ 
 	    return -1;
 	  break;
 
-	/* 0xed42-0xed47 undefind */
+	/* 0xed42-0xed47 undefined */
 
 	case 0xed48: /* SLXT - shift significand left */
 	case 0xed49: /* SRXT - shift significand right */
@@ -6253,10 +6253,10 @@ 
 	    return -1;
 	  break;
 
-	/* 0xed4a-0xed4f undefind */
-	/* 0xed52-0xed53 undefind */
-	/* 0xed56-0xed57 undefind */
-	/* 0xed5a-0xed63 undefind */
+	/* 0xed4a-0xed4f undefined */
+	/* 0xed52-0xed53 undefined */
+	/* 0xed56-0xed57 undefined */
+	/* 0xed5a-0xed63 undefined */
 	/* 0xed68-0xeda7 undefined */
 
 	case 0xeda8: /* CZDT - convert to zoned */
diff --git a/gdb/sh-tdep.c b/gdb/sh-tdep.c
index 8337a07..48a388b 100644
--- a/gdb/sh-tdep.c
+++ b/gdb/sh-tdep.c
@@ -912,7 +912,7 @@ 
    not displace any of the other arguments passed in via registers R4
    to R7.  */
 
-/* Helper function to justify value in register according to endianess.  */
+/* Helper function to justify value in register according to endianness.  */
 static const gdb_byte *
 sh_justify_value_in_reg (struct gdbarch *gdbarch, struct value *val, int len)
 {
@@ -942,7 +942,7 @@ 
 }
 
 /* Helper functions for getting the float arguments right.  Registers usage
-   depends on the ABI and the endianess.  The comments should enlighten how
+   depends on the ABI and the endianness.  The comments should enlighten how
    it's intended to work.  */
 
 /* This array stores which of the float arg registers are already in use.  */
@@ -1045,7 +1045,7 @@ 
   /* Otherwise non-struct types are not treated as float.  */
   if (TYPE_CODE (type) != TYPE_CODE_STRUCT)
     return 0;
-  /* Otherwise structs with more than one memeber are not treated as float.  */
+  /* Otherwise structs with more than one member are not treated as float.  */
   if (TYPE_NFIELDS (type) != 1)
     return 0;
   /* Otherwise if the type of that member is float, the whole type is
@@ -1669,7 +1669,7 @@ 
 					      2, base_regnum, temp_buffer);
       if (status == REG_VALID)
 	{
-	  /* We must pay attention to the endiannes. */
+	  /* We must pay attention to the endianness. */
 	  sh_register_convert_to_virtual (gdbarch, reg_nr,
 					  register_type (gdbarch, reg_nr),
 					  temp_buffer, buffer);
@@ -1712,7 +1712,7 @@ 
       gdb_byte temp_buffer[4 * 2];
       base_regnum = dr_reg_base_num (gdbarch, reg_nr);
 
-      /* We must pay attention to the endiannes.  */
+      /* We must pay attention to the endianness.  */
       sh_register_convert_to_raw (gdbarch, register_type (gdbarch, reg_nr),
 				  reg_nr, buffer, temp_buffer);
 
diff --git a/gdb/sparc-linux-tdep.c b/gdb/sparc-linux-tdep.c
index 35bffdd..be3ff8d 100644
--- a/gdb/sparc-linux-tdep.c
+++ b/gdb/sparc-linux-tdep.c
@@ -68,7 +68,7 @@ 
   SIGTRAMP_FRAME,
   4,
   {
-    { 0x821020d8, ULONGEST_MAX },		/* mov __NR_sugreturn, %g1 */
+    { 0x821020d8, ULONGEST_MAX },		/* mov __NR_sigreturn, %g1 */
     { 0x91d02010, ULONGEST_MAX },		/* ta  0x10 */
     { TRAMP_SENTINEL_INSN, ULONGEST_MAX }
   },
diff --git a/gdb/sparc-nat.c b/gdb/sparc-nat.c
index 3685e5a..8a12487 100644
--- a/gdb/sparc-nat.c
+++ b/gdb/sparc-nat.c
@@ -308,7 +308,7 @@ 
 void
 _initialize_sparc_nat (void)
 {
-  /* Deafult to using SunOS 4 register sets.  */
+  /* Default to using SunOS 4 register sets.  */
   if (sparc_gregmap == NULL)
     sparc_gregmap = &sparc32_sunos4_gregmap;
   if (sparc_fpregmap == NULL)
diff --git a/gdb/stack.c b/gdb/stack.c
index c7c67fc..32c9c73 100644
--- a/gdb/stack.c
+++ b/gdb/stack.c
@@ -959,7 +959,7 @@ 
 /* Print information about frame FRAME.  The output is format according
    to PRINT_LEVEL and PRINT_WHAT and PRINT_ARGS.  For the meaning of
    PRINT_WHAT, see enum print_what comments in frame.h.
-   Note that PRINT_WHAT is overriden if FP_OPTS.print_frame_info
+   Note that PRINT_WHAT is overridden if FP_OPTS.print_frame_info
    != print_frame_info_auto.
 
    Used in "where" output, and to emit breakpoint or step
diff --git a/gdb/target-descriptions.c b/gdb/target-descriptions.c
index f4368bb..f4213bf 100644
--- a/gdb/target-descriptions.c
+++ b/gdb/target-descriptions.c
@@ -1747,7 +1747,7 @@ 
 
 #if GDB_SELF_TEST
 
-/* See target-descritpions.h.  */
+/* See target-descriptions.h.  */
 
 void
 record_xml_tdesc (const char *xml_file, const struct target_desc *tdesc)
diff --git a/gdb/top.c b/gdb/top.c
index 78355a0..15d4fab 100644
--- a/gdb/top.c
+++ b/gdb/top.c
@@ -86,7 +86,7 @@ 
 #define PREFIX(X) the_prompts.prompt_stack[the_prompts.top + X].prefix
 #define SUFFIX(X) the_prompts.prompt_stack[the_prompts.top + X].suffix
 
-/* Default command line prompt.  This is overriden in some configs.  */
+/* Default command line prompt.  This is overridden in some configs.  */
 
 #ifndef DEFAULT_PROMPT
 #define DEFAULT_PROMPT	"(gdb) "
diff --git a/gdb/varobj.c b/gdb/varobj.c
index 37a522b..09c42a3 100644
--- a/gdb/varobj.c
+++ b/gdb/varobj.c
@@ -1831,7 +1831,7 @@ 
   return true;			/* OK */
 }
 
-/* Unistall the object VAR.  */
+/* Uninstall the object VAR.  */
 static void
 uninstall_variable (struct varobj *var)
 {