ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ste...@apache.org
Subject svn commit: r433795 - /ant/core/trunk/docs/ant_task_guidelines.html
Date Tue, 22 Aug 2006 23:12:23 GMT
Author: stevel
Date: Tue Aug 22 16:12:22 2006
New Revision: 433795

URL: http://svn.apache.org/viewvc?rev=433795&view=rev
Log:
s/cvs/svn/ and some other tweaks

Modified:
    ant/core/trunk/docs/ant_task_guidelines.html

Modified: ant/core/trunk/docs/ant_task_guidelines.html
URL: http://svn.apache.org/viewvc/ant/core/trunk/docs/ant_task_guidelines.html?rev=433795&r1=433794&r2=433795&view=diff
==============================================================================
--- ant/core/trunk/docs/ant_task_guidelines.html (original)
+++ ant/core/trunk/docs/ant_task_guidelines.html Tue Aug 22 16:12:22 2006
@@ -127,6 +127,7 @@
     destination directory for output
 </td>
 </tr>
+<tr>    
 <td>
     destfile
 </td>
@@ -167,7 +168,8 @@
 
 Keep member variables private. If read access by subclasses is required,
 add accessor methods rather than change the accessiblity of the member.
-This enables subclasses to access the contents, yet still be decoupled from the actual implementation.
+This enables subclasses to access the contents, yet still be decoupled 
+from the actual implementation.
 <p>
 
 The other common re-use mechanism in Ant is for one task to create and
@@ -223,14 +225,16 @@
 using the JUnit <tt>assert()</tt> method and call <tt>assertTrue()</tt>
 instead.
 <p>
-Java 1.5 adds the <tt>enum</tt>; again, this must not be used.
+Java 1.5 adds the <tt>enum</tt> type; again, this must not be used.
 
 <h2>Explicitly Expand properties in nested text</h2>
 
 For historical reasons, <tt>addText(String text)</tt> is called to
 set the task's nested text, without any property expansion taking place.
 Call <tt>Project.replaceProperties()</tt> to do this manually.
- 
+If you forget, you create a problem that is impossible to fix
+without breaking users' build files.
+
 <h2>Refactor</h2>
 
 If the changes made to a task are making it too unwieldy, split it up
@@ -396,7 +400,7 @@
 <li><h3>Approach 2 - The Manual Way</h3>
 <p>
 Patches to existing files should be generated with
-<code>cvs diff -u filename</code>
+<code>svn diff -u filename</code>
  and save the output to a file. If you want to get
 the changes made to multiple files in a directory , just use <code>cvs
 diff -u</code>.  Then, Tar and GZip the patch file as well as any new files
@@ -454,7 +458,7 @@
 <li>New test cases written and succeed.
 <li>Documentation page extended as appropriate.
 <li>Example task declarations in the documentation tested.
-<li>Diff files generated using cvs diff -u
+<li>Diff files generated using svn diff -u
 <li>Message to dev contains [PATCH], task name and patch reason in
 subject.
 <li>Message body contains a rationale for the patch.
@@ -473,7 +477,7 @@
 <li>New test cases written and succeed
 <li>Documentation page written
 <li>Example task declarations in the documentation tested.
-<li>Patch files generated using cvs diff -u
+<li>Patch files generated using svn diff -u
 <li>patch files include a patch to defaults.properties to register the
 tasks
 <li>patch files include a patch to coretasklist.html or
@@ -484,7 +488,7 @@
 test and patches zipped up to escape the HTML filter.
 </ul>
 <hr>
-<p align="center">Copyright &copy; 2001-2003, 2005 Apache Software Foundation.
All rights
+<p align="center">Copyright &copy; 2001-2003, 2005-2006 Apache Software Foundation.
All rights
 Reserved.</p>
 
 </body></html>



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


Mime
View raw message