Fix some comments (#940)

Signed-off-by: cuishuang <imcusg@gmail.com>
Co-authored-by: Alexey Alexandrov <aalexand@users.noreply.github.com>

Author: cuishuang

internal/driver/driver_test.go

@@ -1050,7 +1050,7 @@ var autoCompleteTests = []struct {
 	{"t", "t"},                            // many matches
 	{"top abc", "top abc"},                // no function name match
 	{"top mangledM", "top mangledMALLOC"}, // single function name match
-	{"top cmd cmd mangledM", "top cmd cmd mangledMALLOC"},
+	{"top cmd mangledM", "top cmd mangledMALLOC"},
 	{"top mangled", "top mangled"},                      // many function name matches
 	{"cmd mangledM", "cmd mangledM"},                    // invalid command
 	{"top mangledM cmd", "top mangledM cmd"},            // cursor misplaced

internal/driver/fetch_test.go

@@ -617,7 +617,7 @@ func TestHTTPSInsecure(t *testing.T) {
 	defer os.RemoveAll(tempdir)
 
 	// pprof writes to $HOME/pprof by default which is not necessarily
-	// writeable (e.g. on a Debian buildd) so set $HOME to something we
+	// writeable (e.g. on a Debian build) so set $HOME to something we
 	// know we can write to for the duration of the test.
 	os.Setenv(homeEnv(), tempdir)
 	defer os.Setenv(homeEnv(), saveHome)
@@ -690,7 +690,7 @@ func TestHTTPSWithServerCertFetch(t *testing.T) {
 	defer os.RemoveAll(tempdir)
 
 	// pprof writes to $HOME/pprof by default which is not necessarily
-	// writeable (e.g. on a Debian buildd) so set $HOME to something we
+	// writeable (e.g. on a Debian build) so set $HOME to something we
 	// know we can write to for the duration of the test.
 	os.Setenv(homeEnv(), tempdir)
 	defer os.Setenv(homeEnv(), saveHome)

internal/driver/html/stacks.js

@@ -579,7 +579,7 @@ function stackViewer(stacks, nodes) {
   }
 
   // percentText returns text that displays v in appropriate units alongside its
-  // percentange.
+  // percentage.
   function percentText(v) {
     function percent(v, total) {
       return Number(((100.0 * v) / total).toFixed(1)) + '%';

internal/elfexec/elfexec.go

@@ -230,7 +230,7 @@ func GetBase(fh *elf.FileHeader, loadSegment *elf.ProgHeader, stextOffset *uint6
 		}
 		if stextOffset == nil && start > 0 && start < 0x8000000000000000 {
 			// A regular user-mode executable. Compute the base offset using same
-			// arithmetics as in ET_DYN case below, see the explanation there.
+			// arithmetic as in ET_DYN case below, see the explanation there.
 			// Ideally, the condition would just be "stextOffset == nil" as that
 			// represents the address of _stext symbol in the vmlinux image. Alas,
 			// the caller may skip reading it from the binary (it's expensive to scan
@@ -313,7 +313,7 @@ func ProgramHeadersForMapping(phdrs []elf.ProgHeader, mapOff, mapSz uint64) []*e
 		// value is dependent on the memory management unit of the CPU. The page
 		// size is 4KB virtually on all the architectures that we care about, so we
 		// define this metric as a constant. If we encounter architectures where
-		// page sie is not 4KB, we must try to guess the page size on the system
+		// page size is not 4KB, we must try to guess the page size on the system
 		// where the profile was collected, possibly using the architecture
 		// specified in the ELF file header.
 		pageSize       = 4096

internal/elfexec/elfexec_test.go

@@ -146,7 +146,7 @@ func TestFindProgHeaderForMapping(t *testing.T) {
 		return
 	}
 
-	// Variuos ELF program headers for unit tests.
+	// Various ELF program headers for unit tests.
 	tinyHeaders := []elf.ProgHeader{
 		{Type: elf.PT_LOAD, Flags: elf.PF_R | elf.PF_X, Off: 0, Vaddr: 0, Paddr: 0, Filesz: 0xc80, Memsz: 0xc80, Align: 0x200000},
 		{Type: elf.PT_LOAD, Flags: elf.PF_R | elf.PF_W, Off: 0xc80, Vaddr: 0x200c80, Paddr: 0x200c80, Filesz: 0x1f0, Memsz: 0x1f0, Align: 0x200000},

internal/measurement/measurement.go

@@ -206,7 +206,7 @@ func (ut UnitType) findByAlias(alias string) *Unit {
 	return nil
 }
 
-// sniffUnit simpifies the input alias and returns the unit associated with the
+// sniffUnit simplifies the input alias and returns the unit associated with the
 // specified alias. It returns nil if the unit with such alias is not found.
 func (ut UnitType) sniffUnit(unit string) *Unit {
 	unit = strings.ToLower(unit)

internal/report/report.go

@@ -569,7 +569,7 @@ func symbolsFromBinaries(prof *profile.Profile, g *graph.Graph, rx *regexp.Regex
 	return objSyms
 }
 
-// objSym represents a symbol identified from a binary. It includes
+// objSymbol represents a symbol identified from a binary. It includes
 // the SymbolInfo from the disasm package and the base that must be
 // added to correspond to sample addresses
 type objSymbol struct {

internal/report/source.go

@@ -490,7 +490,7 @@ func (sp *sourcePrinter) addStack(addr uint64, frames []plugin.Frame) {
 		file.lines[f.Line] = append(file.lines[f.Line], sourceInst{addr, stack})
 
 		// Remember the first function name encountered per source line
-		// and assume that that line belongs to that function.
+		// and assume that line belongs to that function.
 		if _, ok := file.funcName[f.Line]; !ok {
 			file.funcName[f.Line] = f.Func
 		}

profile/legacy_profile_test.go

@@ -59,7 +59,7 @@ func TestLegacyProfileType(t *testing.T) {
 }
 
 func TestCpuParse(t *testing.T) {
-	// profileString is a legacy encoded profile, represnted by words separated by ":"
+	// profileString is a legacy encoded profile, represented by words separated by ":"
 	// Each sample has the form value : N : stack1..stackN
 	// EOF is represented as "0:1:0"
 	profileString := "1:3:100:999:100:"                                      // sample with bogus 999 and duplicate leaf

profile/profile_test.go

@@ -119,7 +119,7 @@ func TestParseError(t *testing.T) {
 
 func TestParseConcatentated(t *testing.T) {
 	prof := testProfile1.Copy()
-	// Write the profile twice to buffer to create concatented profile.
+	// Write the profile twice to buffer to create concatenated profile.
 	var buf bytes.Buffer
 	prof.Write(&buf)
 	prof.Write(&buf)