[ 
https://issues.apache.org/jira/browse/LUCENE-4362?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13453374#comment-13453374
 ] 

Robert Muir commented on LUCENE-4362:
-------------------------------------

Steve, thanks for taking a look. you can see it in Erick's first patch:
{noformat}
Index: 
lucene/analysis/common/src/java/org/apache/lucene/analysis/standard/StandardTokenizerImpl.java
===================================================================
--- 
lucene/analysis/common/src/java/org/apache/lucene/analysis/standard/StandardTokenizerImpl.java
      (revision 1381159)
+++ 
lucene/analysis/common/src/java/org/apache/lucene/analysis/standard/StandardTokenizerImpl.java
      (working copy)
@@ -936,7 +936,7 @@
       }     
     }
 
-       // numRead < 0
+  // numRead < 0
     return true;
   }
{noformat}

So all the generated files seem to have that one lone tab.
                
> ban tab-indented source
> -----------------------
>
>                 Key: LUCENE-4362
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4362
>             Project: Lucene - Core
>          Issue Type: Task
>            Reporter: Robert Muir
>            Assignee: Erick Erickson
>         Attachments: LUCENE-4326_trunk.patch, LUCENE-4362_4x.patch, 
> LUCENE-4362_4x.patch, LUCENE-4362_core.patch, LUCENE-4362.patch, 
> LUCENE-4362.patch
>
>
> This makes code really difficult to read and work with.
> Its easy enough to prevent.
> {noformat}
> Index: build.xml
> ===================================================================
> --- build.xml (revision 1380979)
> +++ build.xml (working copy)
> @@ -77,11 +77,12 @@
>          <or>
>            <containsregexp expression="@author\b" casesensitive="yes"/>
>            <containsregexp expression="\bno(n|)commit\b" casesensitive="no"/>
> +          <containsregexp expression="\t" casesensitive="no"/>
>          </or>
>        </fileset>
>        <map from="${validate.currDir}${file.separator}" to="* "/>
>      </pathconvert>
> -    <fail if="validate.patternsFound">The following files contain @author 
> tags or nocommits:${line.separator}${validate.patternsFound}</fail>
> +    <fail if="validate.patternsFound">The following files contain @author 
> tags, tabs or nocommits:${line.separator}${validate.patternsFound}</fail>
>    </target>
> {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to