Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#113 PackageName documentation error

release_3.1
closed
5
2014-08-26
2003-06-03
pbloemendaal
No

Hi, for CheckStyle 3.1, the documentation for the
PackageName module says: default value of format =
^[a-z]+(\.[a-zA-Z_][a-zA-Z0-9_])$
Isn't there is a \ too much in there?

Discussion

  • pbloemendaal
    pbloemendaal
    2003-06-03

    Logged In: YES
    user_id=194131

    The \. is supposed to mean that . is entered in the regexp.
    For PackageName you specify the default with double \
    ^[a-z]+(\.[a-zA-Z_][a-zA-Z0-9_])$
    Then in the example you specify it with a single \ <property name="format" value="^[a-z]+(.[a-z][a-z0-9])
    $"/>
    It is simply inconsistent, CheckStyle wants the version
    with the single \ (It is confusing to include the escape codes)

     
  • Logged In: YES
    user_id=746148

    You are right. I missed double slash :(
    I've commited fix in CVS for 3.2.
    Thank you.