You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by pe...@apache.org on 2005/01/21 15:32:13 UTC

cvs commit: ant/src/main/org/apache/tools/ant/taskdefs/repository ScheduledUpdatePolicy.java

peterreilly    2005/01/21 06:32:13

  Modified:    src/main/org/apache/tools/ant/taskdefs/repository
                        ScheduledUpdatePolicy.java
  Log:
  checkstyle: 100 char line limit
  
  Revision  Changes    Path
  1.4       +12 -7     ant/src/main/org/apache/tools/ant/taskdefs/repository/ScheduledUpdatePolicy.java
  
  Index: ScheduledUpdatePolicy.java
  ===================================================================
  RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/repository/ScheduledUpdatePolicy.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- ScheduledUpdatePolicy.java	22 Nov 2004 09:23:35 -0000	1.3
  +++ ScheduledUpdatePolicy.java	21 Jan 2005 14:32:13 -0000	1.4
  @@ -33,8 +33,8 @@
   import java.util.Properties;
   
   /**
  - * This {@link org.apache.tools.ant.taskdefs.repository.LibraryPolicy} updates the files only when the schedule
  - * indicates that it should.
  + * This {@link org.apache.tools.ant.taskdefs.repository.LibraryPolicy} updates the files only
  + * when the schedule indicates that it should.
    * <p/>
    * The default interval is eleven hours; it's prime, it encourages
    * regular but not excessive days.
  @@ -53,10 +53,14 @@
        */
       private Properties markerFileToSave;
   
  -    public static final String ERROR_NO_MARKER_FILE = "No marker file";
  -    public static final String MARKER_MISMATCH = "No match between last update and current one";
  -    public static final String INTERVAL_TRIGGERS_UPDATE = "Interval between updates is long; updating";
  -    public static final String INTERVAL_SHORT_NO_UPDATE = "Interval between updates is short; no update";
  +    public static final String ERROR_NO_MARKER_FILE
  +        = "No marker file";
  +    public static final String MARKER_MISMATCH
  +        = "No match between last update and current one";
  +    public static final String INTERVAL_TRIGGERS_UPDATE
  +        = "Interval between updates is long; updating";
  +    public static final String INTERVAL_SHORT_NO_UPDATE
  +        = "Interval between updates is short; no update";
   
   
       public File getMarkerFile() {
  @@ -148,7 +152,8 @@
               markerFileToSave = now;
               return true;
           } catch (IOException e) {
  -            throw new BuildException("Marker file " + markerFile.getAbsolutePath() + " access failed", e);
  +            throw new BuildException(
  +                "Marker file " + markerFile.getAbsolutePath() + " access failed", e);
           }
       }
   
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Re: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs/repository ScheduledUpdatePolicy.java

Posted by Stefan Bodewig <bo...@apache.org>.
On Fri, 21 Jan 2005, Peter Reilly <pe...@apache.org> wrote:

> in src/etc/checkstyle/checkstyle, the linelength check is qualified
> to 100 chars from the default 80:

OK.  No idea who changed it (or even added that file), since I must
admit I've never used it so far.

Cheers

        Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Re: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs/repository ScheduledUpdatePolicy.java

Posted by Peter Reilly <pe...@apache.org>.
in src/etc/checkstyle/checkstyle, the linelength check is qualified to 
100 chars
from the default 80:

    <module name="LineLength">
      <property name="max" value="100"/>
      <property name="ignorePattern" value="^ *\* *[^ ]+$"/>
    </module>

Peter

Stefan Bodewig wrote:

>On 21 Jan 2005, <pe...@apache.org> wrote:
>
>  
>
>>  checkstyle: 100 char line limit
>>    
>>
>
>shouldn't that be 80?
>
>Stefan
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
>For additional commands, e-mail: dev-help@ant.apache.org
>
>
>
>  
>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Re: cvs commit: ant/src/main/org/apache/tools/ant/taskdefs/repository ScheduledUpdatePolicy.java

Posted by Stefan Bodewig <bo...@apache.org>.
On 21 Jan 2005, <pe...@apache.org> wrote:

>   checkstyle: 100 char line limit

shouldn't that be 80?

Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org