Obsoleted SHA-1 option in time ranges.

Time ranges are now specified using either
Unix timestamps or 'first' or 'last'. The
option of specifying a SHA-1 value was
considered not to be particularly useful
in practice. This option has therefore
been obsoleted to simplify the code.

2 files changed, 22 insertions, 51 deletions

diff --git a/doc/bmclient.html b/doc/bmclient.html
index ac491bb..f6f2406 100644
--- a/doc/bmclient.html
+++ b/doc/bmclient.html
@@ -389,12 +389,7 @@ Content-type: text/json
     <test function> <data tag> \
     <metric> <platform> <host> \
     <git repo> <git branch> \
-    {-timestamp \
-     <first Unix timestamp> | \
-     -sha1 <first sha1>} \
-    {-timestamp \
-     <last Unix timestamp> | \
-     -sha1 <last sha1>} \
+    -timerange <from> <to> \
     <diff tolerance> \
     <stab tolerance> \
     <max size> \
@@ -414,16 +409,12 @@ the <i>input</i> time series. The command also lists the most recent fraction
 <br /><br />
 
 The range of the input time series is defined by the <span
-class="command">-timestamp</span>/<span class="command">-sha1</span> options.
-If the <span class="command">-timestamp</span> option is used, the values
-<b><code>first</code></b> and <b><code>last</code></b> specify the first and
-last available snapshot respectively. Otherwise, the first timestamp refers to
-the earliest snapshot that is not earlier than the specified value, while the
-second timestamp refers to the latest snapshot that is not later than the
-specified value.
-
-<b>Note:</b> Using the <span class="command">-sha1</span> option requires a
-snapshot with this exact value to exist for the command to succeed.
+class="command">-timerange</span> option. Each of the two arguments to this
+option may be either 1) <b><code>first</code></b> or <b><code>last</code></b> to
+specify the first and last available snapshot respectively, or 2) a Unix
+timestamp value (an integer) that refers to the earliest snapshot that is not
+earlier than, or the latest snapshot that is not later than the specified
+value respectively.
 
 <br /><br />
 For the sake of the following description, denote the input time series as
@@ -659,12 +650,7 @@ Content-type: text/html
     &lt;metric&gt; &lt;platform&gt; &lt;host&gt; \
     &lt;git repo 1&gt; &lt;git branch 1&gt; \
     &lt;git repo 2&gt; &lt;git branch 2&gt; \
-    {-timestamp \
-     &lt;first Unix timestamp&gt; | \
-     -sha1 &lt;first sha1&gt;} \
-    {-timestamp \
-     &lt;last Unix timestamp&gt; | \
-     -sha1 &lt;last sha1&gt;} \
+    -timerange &lt;from&gt; &lt;to&gt; \
     &lt;diff tolerance&gt; \
     &lt;stab tolerance&gt; \
     &lt;max size&gt; \
@@ -708,12 +694,7 @@ Content-type: text/html
 ./bmclient <b>get rankedbenchmarks</b> \
     &lt;metric&gt; &lt;platform&gt; &lt;host&gt; \
     &lt;git repo&gt; &lt;git branch&gt; \
-    {-timestamp \
-     &lt;first Unix timestamp&gt; | \
-     -sha1 &lt;first sha1&gt;} \
-    {-timestamp \
-     &lt;last Unix timestamp&gt; | \
-     -sha1 &lt;last sha1&gt;} \
+    -timerange &lt;from&gt; &lt;to&gt; \
     &lt;diff tolerance&gt; \
     &lt;stab tolerance&gt; \
     &lt;ranking&gt; &lt;scope&gt; \
@@ -920,12 +901,7 @@ Content-type: text/json
 ./bmclient <b>get stats</b> \
     &lt;metric&gt; &lt;platform&gt; &lt;host&gt; \
     &lt;git repo&gt; &lt;git branch&gt; \
-    {-timestamp \
-     &lt;first Unix timestamp&gt; | \
-     -sha1 &lt;first sha1&gt;} \
-    {-timestamp \
-     &lt;last Unix timestamp&gt; | \
-     -sha1 &lt;last sha1&gt;} \
+    -timerange &lt;from&gt; &lt;to&gt; \
     &lt;diff tolerance&gt; \
     &lt;stab tolerance&gt;
 </pre>
@@ -969,7 +945,7 @@ Content-type: text/json
 <tr>
 <td>
 <pre class="command">
-./bmclient <b>get context</b>
+./bmclient <b>get contexts</b>
 </pre>
 </td>
 <td class="commandDescr">
diff --git a/doc/bmproto.html b/doc/bmproto.html
index 70178b5..2cf9d80 100644
--- a/doc/bmproto.html
+++ b/doc/bmproto.html
@@ -267,13 +267,12 @@ Behaves similarly to GetBenchmarks (see this).
 Lists statistics and history for a specific benchmark.
 
 <br /><br />
-<b>Note:</b> For each snapshot (defining the beginning and end of the range
-respectively) only one of <i>timestamp</i> or <i>sha1</i> is required. If
-<i>timestamp</i> is used, the values <i>first</i> and <i>last</i> specify the
-first and last available snapshot respectively. Otherwise, the first timestamp
-refers to the earliest snapshot that is not earlier than the specified value,
-while the second timestamp refers to the latest snapshot that is not later than
-the specified value.
+<b>Note:</b> For the timestamp1 and timestamp2 attributes (defining the
+beginning and end of the snapshot range respectively), the values <i>first</i>
+and <i>last</i> specify the first and last available snapshot
+respectively. Otherwise, the first timestamp refers to the earliest snapshot
+that is not earlier than the specified value, while the second timestamp
+refers to the latest snapshot that is not later than the specified value.
 
 <br /><br />
 <b>Notes:</b>
@@ -299,8 +298,7 @@ the specified value.
         dataTag="..." metric="..."
         platform="..." host="..."
         gitRepo="..." gitBranch="..."
-        timestamp1="..." sha11="..."
-        timestamp2="..." sha12="..."
+        timestamp1="..." timestamp2="..."
         diffTolerance="..." stabTolerance="..."
         maxSize="..." /&gt;
 &lt;/request&gt;
@@ -345,8 +343,7 @@ Lists statistics and history for a specific benchmark in two branches.
         platform="..." host="..."
         gitRepo1="..." gitBranch1="..."
         gitRepo2="..." gitBranch2="..."
-        timestamp1="..." sha11="..."
-        timestamp2="..." sha12="..."
+        timestamp1="..." timestamp2="..."
         diffTolerance="..." stabTolerance="..."
         maxSize="..." /&gt;
 &lt;/request&gt;
@@ -389,7 +386,7 @@ Lists statistics and history for a specific benchmark in two branches.
 Lists the benchmarks with the most important <i>last difference</i> values.
 
 <br /><br />
-The snapshot range (timestamp{1..2} / sha1{1..2}) is defined as for the <span
+The snapshot range (timestamp1/2) is defined as for the <span
 class="requestName">GetHistory</span> request (see this).
 
 <br /><br />
@@ -405,8 +402,7 @@ exists (or can be computed in the case of the regression).
     &lt;context metric="..."
         platform="..." host="..."
         gitRepo="..." gitBranch="..."
-        timestamp1="..." sha11="..."
-        timestamp2="..." sha12="..."
+        timestamp1="..." timestamp2="..."
         diffTolerance="..." stabTolerance="..."
         ranking="..." scope="..."
         maxSize="..."
@@ -495,8 +491,7 @@ Computes basic statistics for the benchmarks within a certain context.
     &lt;context metric="..."
         platform="..." host="..."
         gitRepo="..." gitBranch="..."
-        timestamp1="..." sha11="..."
-        timestamp2="..." sha12="..."
+        timestamp1="..." timestamp2="..."
         diffTolerance="..." stabTolerance="..." /&gt;
 &lt;/request&gt;
 </pre>