opendev/gerrit
Code Issues Proposed changes

Allow to build api with Java 8

Browse Source 
Java 8's javadoc fails hard on unknown HTML tags. Due some
documentation code snippets that contain generics and lacked a
{@code...}, the code got picked up as html, which made the target fail
like:

  [...]/extensions/registration/DynamicSet.java:64: error: unknown tag: Foo
   *   DynamicSet.setOf(binder(), new TypeLiteral<Thing<Foo>>() {});

We insert the needed {@code...} to make the build pass for Java 8.

Change-Id: I430b8eb9f2f158c2c7a2a7a8d93446eff3253529
This commit is contained in:
Christian Aistleitner
2015-03-29 14:07:15 +02:00
parent 19547b5ac6
commit 0da441ef87
11 changed files with 48 additions and 37 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/util/RangeUtil.java
View File

@@ -46,7 +46,7 @@ public final class RangeUtil {
/**
* Determine the range of values being requested in the given query.
*
* @param rangeQuery the raw query, e.g. "added:>12345"
* @param rangeQuery the raw query, e.g. "{@code added:>12345}"
* @param minValue the minimum possible value for the field, inclusive
* @param maxValue the maximum possible value for the field, inclusive
* @return the calculated {@link Range}, or null if the query is invalid