On 2016-06-08 07:14, Julian Reschke wrote:
On 2016-06-07 16:45, [email protected] wrote:
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStore.java

URL:
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStore.java?rev=1747246&r1=1747245&r2=1747246&view=diff

==============================================================================

---
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStore.java
(original)
+++
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/rdb/RDBDocumentStore.java
Tue Jun  7 14:45:31 2016
@@ -119,7 +119,7 @@ import com.google.common.collect.Sets;
  * derived from an Oak path, and the value is a serialization of a
  * {@link Document} (or a part of one). Additional fields are used
for queries,
  * debugging, and concurrency control:
- * <table style="text-align: left;">
+ * <table style="text-align: left;" summary="">
  * <thead>
  * <tr>
  * <th>Column</th>
...

And this is better exactly how?

Best regards, Julian

OK, I get it: it's better in that JDK8's docLint is happy, despite there being no actual improvement. (Note that the summary attribute is marked "obsolete" in HTML5).

My bigger concern is that we don't enforce docLine compliance, so Javadoc problems will creep back in, until somebody fixes all of them and does a large commit. However, fixing things this way is bad for change tracking and back ports.

Best regards, Julian











Reply via email to