From 6c47edac9ee31ccf5e378f2fed8c010792424cdd Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Tue, 19 Feb 2013 09:53:13 +0900 Subject: [PATCH] Add clarification comment in SparseFileContent.Range.toString Change I34fb07f incorrectly "fixed" the brackets in the `toString` method, however the usage of [ and ) was intentional, to denote that the range is inclusive/exclusive. Add a short comment in the method to prevent the same mistake from being made again. Change-Id: Idfe9b560c0cc55bfa4a6ab06fba172d971c3089c --- .../com/google/gerrit/prettify/common/SparseFileContent.java | 1 + 1 file changed, 1 insertion(+) diff --git a/gerrit-prettify/src/main/java/com/google/gerrit/prettify/common/SparseFileContent.java b/gerrit-prettify/src/main/java/com/google/gerrit/prettify/common/SparseFileContent.java index 1a5468caa7..609f091c9f 100644 --- a/gerrit-prettify/src/main/java/com/google/gerrit/prettify/common/SparseFileContent.java +++ b/gerrit-prettify/src/main/java/com/google/gerrit/prettify/common/SparseFileContent.java @@ -296,6 +296,7 @@ public class SparseFileContent { @Override public String toString() { + // Usage of [ and ) is intentional to denote inclusive/exclusive range return "Range[" + base + "," + end() + ")"; } }