Cleanup and privatise commit comparator

README.md

@@ -20,13 +20,14 @@ repositories {
 }
 
 dependencies {
-    implementation("net.koval.teamcity-gitea-test-task:tinyvm:0.1.0")
+    // other dependencies
+    implementation("net.koval.teamcity-gitea-test-task:tinyvm:0.1.1")
 }
 ```
 
 ### Documentation
 Use autocompletion and hover menus in your IDE, or download the
-[generated HTML documentation](https://git.koval.net/cyclane/teamcity-gitea-test-task/releases/download/v0.1.0/tinyvm-0.1.0-javadoc.zip)
+[generated HTML documentation](https://git.koval.net/cyclane/teamcity-gitea-test-task/releases/download/v0.1.1/tinyvm-0.1.1-javadoc.zip)
 from the [latest release](https://git.koval.net/cyclane/teamcity-gitea-test-task/releases).
 
 ## Instructions

src/main/kotlin/tinyvm/Commit.kt

@@ -1,22 +1,6 @@
 package tinyvm
 
-import java.security.MessageDigest
 import java.time.Instant
-import java.util.HexFormat
-
-/**
- * Represents an arbitrary version manager object.
- */
-abstract class Object(val type: String) {
-    abstract val data: String
-
-    fun hash(): String =
-        HexFormat.of().formatHex(
-            MessageDigest
-                .getInstance("SHA-1")
-                .digest("$type ${data.length}\u0000$data".toByteArray()),
-        )
-}
 
 /**
  * Commits are a pointer to a 'head' tree with some metadata.
@@ -28,7 +12,6 @@ class Commit(
     val timestamp: Instant,
 ) : Object("commit") {
     override val data: String
-        // Use \n\n for end of header in-case additional metadata is implemented in the future.
         get() = "tree ${tree.hash()}\nauthor $author\ntimestamp ${timestamp.epochSecond}\n\n$message"
 
     override fun toString(): String = "commit ${hash()}\n$data"

src/main/kotlin/tinyvm/Object.kt

@@ -0,0 +1,17 @@
+package tinyvm
+
+import java.security.MessageDigest
+import java.util.HexFormat
+
+/**
+ * Represents an arbitrary version manager object.
+ */
+abstract class Object(val type: String) {
+    abstract val data: String
+
+    fun hash(): String =
+        HexFormat.of().formatHex(
+            MessageDigest.getInstance("SHA-1")
+                .digest("$type ${data.length}\u0000$data".toByteArray()),
+        )
+}

src/main/kotlin/tinyvm/Repository.kt

@@ -2,7 +2,7 @@ package tinyvm
 
 class HashCollisionException(hash: String) : Exception("Different object types with identical hash '$hash'")
 
-class CommitTimeComparator : Comparator<Commit> {
+private class CommitTimeComparator : Comparator<Commit> {
     override fun compare(
         o1: Commit,
         o2: Commit,

src/main/kotlin/tinyvm/Tree.kt

@@ -22,7 +22,7 @@ class Tree(val nodes: Map<String, Node>) : Node("tree") {
                 when (node) {
                     is Tree -> "+$name/\n"
                     is Blob -> "+$name\n"
-                } + node.toString().leftMargin()
+                } + node.toString().insertLeftMargin()
             }.sorted().joinToString("\n")
 }
 
@@ -30,7 +30,7 @@ class Tree(val nodes: Map<String, Node>) : Node("tree") {
  * A blob is a data container.
  */
 class Blob(override val data: String) : Node("blob") {
-    override fun toString(): String = "blob ${hash()}\n${data.leftMargin()}"
+    override fun toString(): String = "blob ${hash()}\n${data.insertLeftMargin()}"
 }
 
-private fun String.leftMargin(): String = split('\n').joinToString("\n") { "| $it" }
+private fun String.insertLeftMargin(): String = split('\n').joinToString("\n") { "| $it" }