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

Uwe Schindler commented on LUCENE-4362:
---------------------------------------

But Steven fixed JFlex? Maybe you must update your JFlex checkout and rebuild 
with maven?

Otherwise +1 to commit (the generated files are checked in, yes, and need to be 
regenerated). But when regenerating, check that the output looks almost the 
same, just to be sure that you are not using the wrong version. JavaCC is hard 
linked to a IVY-Download with specific version, but JFlex is trunk, so take 
care of unwanted changes, e.g. introduced by too old versions!
                
> 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_4x.patch, LUCENE-4362_core.patch, 
> LUCENE-4362.patch, LUCENE-4362.patch, LUCENE-4362_trunk.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