x/tools: remove dead code

These unused functions and methods were discovered by a (work in
progress) dead function analyzer.

Change-Id: Iaa9c5fd4f06bdcd5d7c6e194bcc24d6b37df8f4c
Reviewed-on: https://go-review.googlesource.com/c/tools/+/639675
Auto-Submit: Alan Donovan <[email protected]>
Reviewed-by: Robert Findley <[email protected]>
LUCI-TryBot-Result: Go LUCI <[email protected]>

Author: adonovan

cmd/splitdwarf/splitdwarf.go

@@ -358,6 +358,7 @@ func CreateMmapFile(outDwarf string, size int64) (*os.File, []byte) {
 	return dwarfFile, buffer
 }
 
+// (dead code; retained for debugging)
 func describe(exem *macho.FileTOC) {
 	note("Type = %s, Flags=0x%x", exem.Type, uint32(exem.Flags))
 	for i, l := range exem.Loads {

cmd/ssadump/main.go

@@ -188,11 +188,6 @@ func doMain() error {
 // e.g. --flag=one --flag=two would produce []string{"one", "two"}.
 type stringListValue []string
 
-func newStringListValue(val []string, p *[]string) *stringListValue {
-	*p = val
-	return (*stringListValue)(p)
-}
-
 func (ss *stringListValue) Get() interface{} { return []string(*ss) }
 
 func (ss *stringListValue) String() string { return fmt.Sprintf("%q", *ss) }

cmd/stringer/endtoend_test.go

@@ -93,20 +93,6 @@ func typeName(fname string) string {
 	return fmt.Sprintf("%c%s", base[0]+'A'-'a', base[1:len(base)-len(".go")])
 }
 
-func moreTests(t *testing.T, dirname, prefix string) []string {
-	x, err := os.ReadDir(dirname)
-	if err != nil {
-		// error, but try the rest of the tests
-		t.Errorf("can't read type param tess from %s: %v", dirname, err)
-		return nil
-	}
-	names := make([]string, len(x))
-	for i, f := range x {
-		names[i] = prefix + "/" + f.Name()
-	}
-	return names
-}
-
 // TestTags verifies that the -tags flag works as advertised.
 func TestTags(t *testing.T) {
 	stringer := stringerPath(t)

container/intsets/sparse.go

@@ -287,14 +287,6 @@ func (s *Sparse) next(b *block) *block {
 	return b.next
 }
 
-// prev returns the previous block in the list, or end if b is the first block.
-func (s *Sparse) prev(b *block) *block {
-	if b.prev == &s.root {
-		return &none
-	}
-	return b.prev
-}
-
 // IsEmpty reports whether the set s is empty.
 func (s *Sparse) IsEmpty() bool {
 	return s.root.next == nil || s.root.offset == MaxInt
@@ -1077,6 +1069,7 @@ func (s *Sparse) AppendTo(slice []int) []int {
 // -- Testing/debugging ------------------------------------------------
 
 // check returns an error if the representation invariants of s are violated.
+// (unused; retained for debugging)
 func (s *Sparse) check() error {
 	s.init()
 	if s.root.empty() {

copyright/copyright.go

@@ -16,6 +16,7 @@ import (
 	"strings"
 )
 
+// (used only by tests)
 func checkCopyright(dir string) ([]string, error) {
 	var files []string
 	err := filepath.WalkDir(dir, func(path string, d fs.DirEntry, err error) error {

go/analysis/internal/analysisflags/flags.go

@@ -250,21 +250,12 @@ const (
 	setFalse
 )
 
-func triStateFlag(name string, value triState, usage string) *triState {
-	flag.Var(&value, name, usage)
-	return &value
-}
-
 // triState implements flag.Value, flag.Getter, and flag.boolFlag.
 // They work like boolean flags: we can say vet -printf as well as vet -printf=true
 func (ts *triState) Get() interface{} {
 	return *ts == setTrue
 }
 
-func (ts triState) isTrue() bool {
-	return ts == setTrue
-}
-
 func (ts *triState) Set(value string) error {
 	b, err := strconv.ParseBool(value)
 	if err != nil {

go/expect/expect.go

@@ -120,10 +120,3 @@ func MatchBefore(fset *token.FileSet, readFile ReadFile, end token.Pos, pattern
 	}
 	return f.Pos(startOffset + matchStart), f.Pos(startOffset + matchEnd), nil
 }
-
-func lineEnd(f *token.File, line int) token.Pos {
-	if line >= f.LineCount() {
-		return token.Pos(f.Base() + f.Size())
-	}
-	return f.LineStart(line + 1)
-}

go/packages/gopackages/main.go

@@ -248,11 +248,6 @@ func (app *application) print(lpkg *packages.Package) {
 // e.g. --flag=one --flag=two would produce []string{"one", "two"}.
 type stringListValue []string
 
-func newStringListValue(val []string, p *[]string) *stringListValue {
-	*p = val
-	return (*stringListValue)(p)
-}
-
 func (ss *stringListValue) Get() interface{} { return []string(*ss) }
 
 func (ss *stringListValue) String() string { return fmt.Sprintf("%q", *ss) }

go/packages/packages_test.go

@@ -2999,30 +2999,6 @@ func constant(p *packages.Package, name string) *types.Const {
 	return c.(*types.Const)
 }
 
-func copyAll(srcPath, dstPath string) error {
-	return filepath.Walk(srcPath, func(path string, info os.FileInfo, _ error) error {
-		if info.IsDir() {
-			return nil
-		}
-		contents, err := os.ReadFile(path)
-		if err != nil {
-			return err
-		}
-		rel, err := filepath.Rel(srcPath, path)
-		if err != nil {
-			return err
-		}
-		dstFilePath := strings.Replace(filepath.Join(dstPath, rel), "definitelynot_go.mod", "go.mod", -1)
-		if err := os.MkdirAll(filepath.Dir(dstFilePath), 0755); err != nil {
-			return err
-		}
-		if err := os.WriteFile(dstFilePath, contents, 0644); err != nil {
-			return err
-		}
-		return nil
-	})
-}
-
 func TestExportFile(t *testing.T) {
 	// This used to trigger the log.Fatal in loadFromExportData.
 	// See go.dev/issue/45584.

go/ssa/dom.go

@@ -318,6 +318,7 @@ func printDomTreeText(buf *bytes.Buffer, v *BasicBlock, indent int) {
 
 // printDomTreeDot prints the dominator tree of f in AT&T GraphViz
 // (.dot) format.
+// (unused; retained for debugging)
 func printDomTreeDot(buf *bytes.Buffer, f *Function) {
 	fmt.Fprintln(buf, "//", f)
 	fmt.Fprintln(buf, "digraph domtree {")