svn commit: r1580613 - in /lucene/dev/branches/lucene_solr_4_7: ./ dev-tools/ dev-tools/scripts/checkJavaDocs.py

[us...@apache.org]

Author: uschindler
Date: Sun Mar 23 21:04:58 2014
New Revision: 1580613

URL: http://svn.apache.org/r1580613
Log:
Merged revision(s) 1575337 from lucene/dev/branches/branch_4x:
unescape %20 in urls so we don't get false failures with 1.7.0_60

Modified:
    lucene/dev/branches/lucene_solr_4_7/   (props changed)
    lucene/dev/branches/lucene_solr_4_7/dev-tools/   (props changed)
    lucene/dev/branches/lucene_solr_4_7/dev-tools/scripts/checkJavaDocs.py

Modified: lucene/dev/branches/lucene_solr_4_7/dev-tools/scripts/checkJavaDocs.py
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene_solr_4_7/dev-tools/scripts/checkJavaDocs.py?rev=1580613&r1=1580612&r2=1580613&view=diff
==============================================================================
--- lucene/dev/branches/lucene_solr_4_7/dev-tools/scripts/checkJavaDocs.py (original)
+++ lucene/dev/branches/lucene_solr_4_7/dev-tools/scripts/checkJavaDocs.py Sun Mar 23 21:04:58 2014
@@ -212,7 +212,7 @@ def checkClassSummaries(fullPath):
     if inThing:
       if lineLower.find('</tr>') != -1:
         if not hasDesc:
-          missing.append((lastCaption, lastItem))
+          missing.append((lastCaption, unEscapeURL(lastItem)))
         inThing = False
         continue
       else:
@@ -298,6 +298,11 @@ def checkSummary(fullPath):
   f.close()
   return anyMissing
 
+def unEscapeURL(s):
+  # Not exhaustive!!
+  s = s.replace('%20', ' ')
+  return s
+
 def unescapeHTML(s):
   s = s.replace('&lt;', '<')
   s = s.replace('&gt;', '>')