HistorySize.Contents(): new method

Extract a new method from HistorySize.TableString().

Author: mhagger

sizes/output.go

@@ -323,78 +323,8 @@ type table struct {
 }
 
 func (s HistorySize) TableString(threshold Threshold, nameStyle NameStyle) string {
-	S := newSection
-	I := newItem
-	metric := counts.MetricPrefixes
-	binary := counts.BinaryPrefixes
-	t := &table{
-		contents: S(
-			"",
-			S(
-				"Overall repository size",
-				S(
-					"Commits",
-					I("Count", nil, s.UniqueCommitCount, metric, " ", 500e3),
-					I("Total size", nil, s.UniqueCommitSize, binary, "B", 250e6),
-				),
-
-				S(
-					"Trees",
-					I("Count", nil, s.UniqueTreeCount, metric, " ", 1.5e6),
-					I("Total size", nil, s.UniqueTreeSize, binary, "B", 2e9),
-					I("Total tree entries", nil, s.UniqueTreeEntries, metric, " ", 50e6),
-				),
-
-				S(
-					"Blobs",
-					I("Count", nil, s.UniqueBlobCount, metric, " ", 1.5e6),
-					I("Total size", nil, s.UniqueBlobSize, binary, "B", 10e9),
-				),
-
-				S(
-					"Annotated tags",
-					I("Count", nil, s.UniqueTagCount, metric, " ", 25e3),
-				),
-
-				S(
-					"References",
-					I("Count", nil, s.ReferenceCount, metric, " ", 25e3),
-				),
-			),
-
-			S("Biggest objects",
-				S("Commits",
-					I("Maximum size", s.MaxCommitSizeCommit, s.MaxCommitSize, binary, "B", 50e3),
-					I("Maximum parents", s.MaxParentCountCommit, s.MaxParentCount, metric, " ", 10),
-				),
-
-				S("Trees",
-					I("Maximum entries", s.MaxTreeEntriesTree, s.MaxTreeEntries, metric, " ", 1000),
-				),
-
-				S("Blobs",
-					I("Maximum size", s.MaxBlobSizeBlob, s.MaxBlobSize, binary, "B", 10e6),
-				),
-			),
-
-			S("History structure",
-				I("Maximum history depth", nil, s.MaxHistoryDepth, metric, " ", 500e3),
-				I("Maximum tag depth", s.MaxTagDepthTag, s.MaxTagDepth, metric, " ", 1.001),
-			),
-
-			S("Biggest checkouts",
-				I("Number of directories", s.MaxExpandedTreeCountTree, s.MaxExpandedTreeCount, metric, " ", 2000),
-				I("Maximum path depth", s.MaxPathDepthTree, s.MaxPathDepth, metric, " ", 10),
-				I("Maximum path length", s.MaxPathLengthTree, s.MaxPathLength, binary, "B", 100),
-
-				I("Number of files", s.MaxExpandedBlobCountTree, s.MaxExpandedBlobCount, metric, " ", 50e3),
-				I("Total size of files", s.MaxExpandedBlobSizeTree, s.MaxExpandedBlobSize, binary, "B", 1e9),
-
-				I("Number of symlinks", s.MaxExpandedLinkCountTree, s.MaxExpandedLinkCount, metric, " ", 25e3),
-
-				I("Number of submodules", s.MaxExpandedSubmoduleCountTree, s.MaxExpandedSubmoduleCount, metric, " ", 100),
-			),
-		),
+	t := table{
+		contents:  s.Contents(),
 		threshold: threshold,
 		nameStyle: nameStyle,
 		footnotes: NewFootnotes(),
@@ -447,3 +377,77 @@ func (t *table) formatRow(
 		prefix, name, spacer, citation, valueString, unitString, levelOfConcern,
 	)
 }
+
+func (s HistorySize) Contents() tableContents {
+	S := newSection
+	I := newItem
+	metric := counts.MetricPrefixes
+	binary := counts.BinaryPrefixes
+	return S(
+		"",
+		S(
+			"Overall repository size",
+			S(
+				"Commits",
+				I("Count", nil, s.UniqueCommitCount, metric, " ", 500e3),
+				I("Total size", nil, s.UniqueCommitSize, binary, "B", 250e6),
+			),
+
+			S(
+				"Trees",
+				I("Count", nil, s.UniqueTreeCount, metric, " ", 1.5e6),
+				I("Total size", nil, s.UniqueTreeSize, binary, "B", 2e9),
+				I("Total tree entries", nil, s.UniqueTreeEntries, metric, " ", 50e6),
+			),
+
+			S(
+				"Blobs",
+				I("Count", nil, s.UniqueBlobCount, metric, " ", 1.5e6),
+				I("Total size", nil, s.UniqueBlobSize, binary, "B", 10e9),
+			),
+
+			S(
+				"Annotated tags",
+				I("Count", nil, s.UniqueTagCount, metric, " ", 25e3),
+			),
+
+			S(
+				"References",
+				I("Count", nil, s.ReferenceCount, metric, " ", 25e3),
+			),
+		),
+
+		S("Biggest objects",
+			S("Commits",
+				I("Maximum size", s.MaxCommitSizeCommit, s.MaxCommitSize, binary, "B", 50e3),
+				I("Maximum parents", s.MaxParentCountCommit, s.MaxParentCount, metric, " ", 10),
+			),
+
+			S("Trees",
+				I("Maximum entries", s.MaxTreeEntriesTree, s.MaxTreeEntries, metric, " ", 1000),
+			),
+
+			S("Blobs",
+				I("Maximum size", s.MaxBlobSizeBlob, s.MaxBlobSize, binary, "B", 10e6),
+			),
+		),
+
+		S("History structure",
+			I("Maximum history depth", nil, s.MaxHistoryDepth, metric, " ", 500e3),
+			I("Maximum tag depth", s.MaxTagDepthTag, s.MaxTagDepth, metric, " ", 1.001),
+		),
+
+		S("Biggest checkouts",
+			I("Number of directories", s.MaxExpandedTreeCountTree, s.MaxExpandedTreeCount, metric, " ", 2000),
+			I("Maximum path depth", s.MaxPathDepthTree, s.MaxPathDepth, metric, " ", 10),
+			I("Maximum path length", s.MaxPathLengthTree, s.MaxPathLength, binary, "B", 100),
+
+			I("Number of files", s.MaxExpandedBlobCountTree, s.MaxExpandedBlobCount, metric, " ", 50e3),
+			I("Total size of files", s.MaxExpandedBlobSizeTree, s.MaxExpandedBlobSize, binary, "B", 1e9),
+
+			I("Number of symlinks", s.MaxExpandedLinkCountTree, s.MaxExpandedLinkCount, metric, " ", 25e3),
+
+			I("Number of submodules", s.MaxExpandedSubmoduleCountTree, s.MaxExpandedSubmoduleCount, metric, " ", 100),
+		),
+	)
+}