Author: peterreilly
Date: Tue Sep 26 14:10:32 2006
New Revision: 450207

URL: http://svn.apache.org/viewvc?view=rev&rev=450207
Log:
eol-style

Modified:
    ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Provider.java   
(contents, props changed)
    ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Service.java   
(contents, props changed)
    ant/core/trunk/src/tests/antunit/taskdefs/jar-spi-test.xml   (contents, 
props changed)

Modified: ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Provider.java
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Provider.java?view=diff&rev=450207&r1=450206&r2=450207
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Provider.java 
(original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Provider.java Tue 
Sep 26 14:10:32 2006
@@ -1,62 +1,62 @@
-/*

- *  Licensed to the Apache Software Foundation (ASF) under one or more

- *  contributor license agreements.  See the NOTICE file distributed with

- *  this work for additional information regarding copyright ownership.

- *  The ASF licenses this file to You under the Apache License, Version 2.0

- *  (the "License"); you may not use this file except in compliance with

- *  the License.  You may obtain a copy of the License at

- *

- *      http://www.apache.org/licenses/LICENSE-2.0

- *

- *  Unless required by applicable law or agreed to in writing, software

- *  distributed under the License is distributed on an "AS IS" BASIS,

- *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

- *  See the License for the specific language governing permissions and

- *  limitations under the License.

- *

- */

-package org.apache.tools.ant.types.spi;

-

-import org.apache.tools.ant.ProjectComponent;

-import org.apache.tools.ant.BuildException;

-

-/**

- * ANT Jar-Task SPI extension

- * This class corresponds to the nested element

- * <provider type="type"> in the <service type="">

- * nested element of the jar task.

- * @see http://issues.apache.org/bugzilla/show_bug.cgi?id=31520

- */

-public class Provider extends ProjectComponent {

-    private String type;

-

-    /**

-     * @return the class name for

-     */

-    public String getClassName(){

-        return type;

-    }

-

-    /**

-     * Set the provider classname.

-     * @param type the value to set.

-     */

-    public void setClassName(String type){

-        this.type = type;

-    }

-

-    /**

-     * Check if the component has been configured correctly.

-     */

-    public void check() {

-        if (type == null) {

-            throw new BuildException(

-                "classname attribute must be set for provider element",

-                getLocation());

-        }

-        if (type.length() == 0) {

-            throw new BuildException(

-                "Invalid empty classname", getLocation());

-        }

-    }

-}

+/*
+ *  Licensed to the Apache Software Foundation (ASF) under one or more
+ *  contributor license agreements.  See the NOTICE file distributed with
+ *  this work for additional information regarding copyright ownership.
+ *  The ASF licenses this file to You under the Apache License, Version 2.0
+ *  (the "License"); you may not use this file except in compliance with
+ *  the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ *
+ */
+package org.apache.tools.ant.types.spi;
+
+import org.apache.tools.ant.ProjectComponent;
+import org.apache.tools.ant.BuildException;
+
+/**
+ * ANT Jar-Task SPI extension
+ * This class corresponds to the nested element
+ * <provider type="type"> in the <service type="">
+ * nested element of the jar task.
+ * @see http://issues.apache.org/bugzilla/show_bug.cgi?id=31520
+ */
+public class Provider extends ProjectComponent {
+    private String type;
+
+    /**
+     * @return the class name for
+     */
+    public String getClassName(){
+        return type;
+    }
+
+    /**
+     * Set the provider classname.
+     * @param type the value to set.
+     */
+    public void setClassName(String type){
+        this.type = type;
+    }
+
+    /**
+     * Check if the component has been configured correctly.
+     */
+    public void check() {
+        if (type == null) {
+            throw new BuildException(
+                "classname attribute must be set for provider element",
+                getLocation());
+        }
+        if (type.length() == 0) {
+            throw new BuildException(
+                "Invalid empty classname", getLocation());
+        }
+    }
+}

Propchange: ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Provider.java
------------------------------------------------------------------------------
    svn:eol-style = native

Modified: ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Service.java
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Service.java?view=diff&rev=450207&r1=450206&r2=450207
==============================================================================
--- ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Service.java 
(original)
+++ ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Service.java Tue Sep 
26 14:10:32 2006
@@ -1,122 +1,122 @@
-/*

- *  Licensed to the Apache Software Foundation (ASF) under one or more

- *  contributor license agreements.  See the NOTICE file distributed with

- *  this work for additional information regarding copyright ownership.

- *  The ASF licenses this file to You under the Apache License, Version 2.0

- *  (the "License"); you may not use this file except in compliance with

- *  the License.  You may obtain a copy of the License at

- *

- *      http://www.apache.org/licenses/LICENSE-2.0

- *

- *  Unless required by applicable law or agreed to in writing, software

- *  distributed under the License is distributed on an "AS IS" BASIS,

- *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

- *  See the License for the specific language governing permissions and

- *  limitations under the License.

- *

- */

-package org.apache.tools.ant.types.spi;

-

-import java.io.ByteArrayInputStream;

-import java.io.ByteArrayOutputStream;

-import java.io.IOException;

-import java.io.InputStream;

-import java.io.OutputStreamWriter;

-import java.io.Writer;

-import java.util.ArrayList;

-import java.util.Iterator;

-import java.util.List;

-

-import org.apache.tools.ant.ProjectComponent;

-import org.apache.tools.ant.BuildException;

-

-/**

- * ANT Jar-Task SPI extension

- * 

- * @see http://issues.apache.org/bugzilla/show_bug.cgi?id=31520

- */

-public class Service extends ProjectComponent {

-    private List providerList = new ArrayList();

-    private String type;

-

-    /**

-     * Set the provider classname.

-     * @param className the classname of a provider of this service.

-     */

-    public void setProvider(String className) {

-        Provider provider = new Provider();

-        provider.setClassName(className);

-        providerList.add(provider);

-    }

-

-    /**

-     * Add a nested provider element.

-     * @param provider a provider element.

-     */

-    public void addConfiguredProvider(Provider provider) {

-        provider.check();

-        providerList.add(provider);

-    }

-

-    /**

-     * @return the service type.

-     */

-    public String getType(){

-        return type;

-    }

-

-    /**

-     * Set the service type.

-     * @param type the service type, a classname of

-     *             an interface or a class (normally

-     *             abstract).

-     */

-    public void setType(String type){

-        this.type = type;

-    }

-

-    /**

-     * Return the implementations of this

-     * services as an inputstream.

-     * @return an inputstream of the classname names

-     *         encoded as UTF-8.

-     */

-    public InputStream getAsStream() throws IOException {

-        ByteArrayOutputStream arrayOut;

-        Writer writer;

-        Iterator providerIterator;

-        Provider provider;

-        

-        arrayOut = new ByteArrayOutputStream();

-        writer = new OutputStreamWriter(arrayOut, "UTF-8");

-        providerIterator = providerList.iterator();

-        while (providerIterator.hasNext()){

-            provider = (Provider) providerIterator.next();

-            writer.write(provider.getClassName());

-            writer.write("\n");

-        }

-        writer.close();

-        return new ByteArrayInputStream(arrayOut.toByteArray());

-    }

-

-    /**

-     * Check if this object is configured correctly as a nested

-     * element.

-     */

-    public void check() {

-        if (type == null) {

-            throw new BuildException(

-                "type attribute must be set for service element",

-                getLocation());

-        }

-        if (type.length() == 0) {

-            throw new BuildException(

-                "Invalid empty type classname", getLocation());

-        }

-        if (providerList.size() == 0) {

-            throw new BuildException(

-                "provider attribute or nested provider element must be set!",

-                getLocation());

-        }

-    }

-}

+/*
+ *  Licensed to the Apache Software Foundation (ASF) under one or more
+ *  contributor license agreements.  See the NOTICE file distributed with
+ *  this work for additional information regarding copyright ownership.
+ *  The ASF licenses this file to You under the Apache License, Version 2.0
+ *  (the "License"); you may not use this file except in compliance with
+ *  the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ *
+ */
+package org.apache.tools.ant.types.spi;
+
+import java.io.ByteArrayInputStream;
+import java.io.ByteArrayOutputStream;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.OutputStreamWriter;
+import java.io.Writer;
+import java.util.ArrayList;
+import java.util.Iterator;
+import java.util.List;
+
+import org.apache.tools.ant.ProjectComponent;
+import org.apache.tools.ant.BuildException;
+
+/**
+ * ANT Jar-Task SPI extension
+ * 
+ * @see http://issues.apache.org/bugzilla/show_bug.cgi?id=31520
+ */
+public class Service extends ProjectComponent {
+    private List providerList = new ArrayList();
+    private String type;
+
+    /**
+     * Set the provider classname.
+     * @param className the classname of a provider of this service.
+     */
+    public void setProvider(String className) {
+        Provider provider = new Provider();
+        provider.setClassName(className);
+        providerList.add(provider);
+    }
+
+    /**
+     * Add a nested provider element.
+     * @param provider a provider element.
+     */
+    public void addConfiguredProvider(Provider provider) {
+        provider.check();
+        providerList.add(provider);
+    }
+
+    /**
+     * @return the service type.
+     */
+    public String getType(){
+        return type;
+    }
+
+    /**
+     * Set the service type.
+     * @param type the service type, a classname of
+     *             an interface or a class (normally
+     *             abstract).
+     */
+    public void setType(String type){
+        this.type = type;
+    }
+
+    /**
+     * Return the implementations of this
+     * services as an inputstream.
+     * @return an inputstream of the classname names
+     *         encoded as UTF-8.
+     */
+    public InputStream getAsStream() throws IOException {
+        ByteArrayOutputStream arrayOut;
+        Writer writer;
+        Iterator providerIterator;
+        Provider provider;
+        
+        arrayOut = new ByteArrayOutputStream();
+        writer = new OutputStreamWriter(arrayOut, "UTF-8");
+        providerIterator = providerList.iterator();
+        while (providerIterator.hasNext()){
+            provider = (Provider) providerIterator.next();
+            writer.write(provider.getClassName());
+            writer.write("\n");
+        }
+        writer.close();
+        return new ByteArrayInputStream(arrayOut.toByteArray());
+    }
+
+    /**
+     * Check if this object is configured correctly as a nested
+     * element.
+     */
+    public void check() {
+        if (type == null) {
+            throw new BuildException(
+                "type attribute must be set for service element",
+                getLocation());
+        }
+        if (type.length() == 0) {
+            throw new BuildException(
+                "Invalid empty type classname", getLocation());
+        }
+        if (providerList.size() == 0) {
+            throw new BuildException(
+                "provider attribute or nested provider element must be set!",
+                getLocation());
+        }
+    }
+}

Propchange: ant/core/trunk/src/main/org/apache/tools/ant/types/spi/Service.java
------------------------------------------------------------------------------
    svn:eol-style = native

Modified: ant/core/trunk/src/tests/antunit/taskdefs/jar-spi-test.xml
URL: 
http://svn.apache.org/viewvc/ant/core/trunk/src/tests/antunit/taskdefs/jar-spi-test.xml?view=diff&rev=450207&r1=450206&r2=450207
==============================================================================
--- ant/core/trunk/src/tests/antunit/taskdefs/jar-spi-test.xml (original)
+++ ant/core/trunk/src/tests/antunit/taskdefs/jar-spi-test.xml Tue Sep 26 
14:10:32 2006
@@ -1,124 +1,124 @@
-<?xml version="1.0" encoding="utf-8"?>

-<project xmlns:au="antlib:org.apache.ant.antunit">

-  <property name="jar.dir" location="jar_spi_dir"/>

-  <property name="jar.src.dir" location="jar_spi_dir/src"/>

-  <property name="jar.src.file" location="jar_spi_dir/src/a_file"/>

-

-  <target name="init">

-    <mkdir dir="${jar.src.dir}"/>

-    <delete quiet="yes" file="${jar.src.file}"/>

-    <touch file="${jar.src.file}"/>

-    <delete quiet="yes" file="${jar.dir}/file.jar"/>

-    <delete quiet="yes" dir="${jar.dir}/output"/>

-  </target>

-

-  <target name="test-simple" depends="init">

-

-    <jar jarfile="${jar.dir}/file.jar">

-      <fileset dir="${jar.src.dir}"/>

-      <service type="a.b.c" provider="a.b.c.d"/>

-    </jar>

-

-    <unjar src="${jar.dir}/file.jar"

-           dest="${jar.dir}/output"/>

-

-    <loadfile property="simple"

-              srcfile="${jar.dir}/output/META-INF/service/a.b.c"

-              encoding="UTF-8"/>

-

-    <au:assertTrue>

-      <matches string="${simple}" pattern="^a\.b\.c\.d\n$"/>

-    </au:assertTrue>

-

-  </target>

-

-  <target name="test-providers" depends="init">

-

-    <jar jarfile="${jar.dir}/file.jar">

-      <fileset dir="${jar.src.dir}"/>

-      <service type="a.b.c">

-        <provider classname="a.X"/>

-        <provider classname="a.D"/>

-      </service>

-    </jar>

-

-    <unjar src="${jar.dir}/file.jar"

-           dest="${jar.dir}/output"/>

-

-    <loadfile property="providers"

-              srcfile="${jar.dir}/output/META-INF/service/a.b.c"

-              encoding="UTF-8"/>

-    <au:assertTrue>

-      <matches string="${providers}" pattern="^a\.X\na\.D$"/>

-    </au:assertTrue>

-

-  </target>

-

-  <target name="test-multi" depends="init">

-

-    <jar jarfile="${jar.dir}/file.jar">

-      <fileset dir="${jar.src.dir}"/>

-      <service type="a.b.c">

-        <provider classname="a.X"/>

-        <provider classname="a.D"/>

-      </service>

-      <service type="javax.a.service">

-        <provider classname="a.O.T"/>

-        <provider classname="a.B"/>

-      </service>

-    </jar>

-

-    <unjar src="${jar.dir}/file.jar"

-           dest="${jar.dir}/output"/>

-

-    <loadfile property="multi-a"

-              srcfile="${jar.dir}/output/META-INF/service/a.b.c"

-              encoding="UTF-8"/>

-

-    <au:assertTrue>

-      <matches string="${multi-a}" pattern="^a\.X\na\.D$"/>

-    </au:assertTrue>

-

-    <loadfile property="multi-b"

-              srcfile="${jar.dir}/output/META-INF/service/javax.a.service"

-              encoding="UTF-8"/>

-

-    <au:assertTrue>

-      <matches string="${multi-b}" pattern="^a\.O\.T\na\.B$"/>

-    </au:assertTrue>

-

-  </target>

-

-  <target name="test-reject-no-type" depends="init">

-    <au:expectfailure>

-      <jar jarfile="${jar.dir}/file.jar">

-        <fileset dir="${jar.src.dir}"/>

-        <service provider="a.X"/>

-      </jar>

-    </au:expectfailure>

-  </target>

-

-  <target name="test-reject-no-provider" depends="init">

-    <au:expectfailure>

-      <jar jarfile="${jar.dir}/file.jar">

-        <fileset dir="${jar.src.dir}"/>

-        <service type="a.X"/>

-      </jar>

-    </au:expectfailure>

-  </target>

-

-  <target name="test-reject-no-classname" depends="init">

-    <au:expectfailure>

-      <jar jarfile="${jar.dir}/file.jar">

-        <fileset dir="${jar.src.dir}"/>

-        <service type="a.X">

-          <provider/>

-        </service>

-      </jar>

-    </au:expectfailure>

-  </target>

-

-  <target name="tearDown">

-    <delete quiet="yes" dir="${jar.dir}"/>

-  </target>

-</project>

+<?xml version="1.0" encoding="utf-8"?>
+<project xmlns:au="antlib:org.apache.ant.antunit">
+  <property name="jar.dir" location="jar_spi_dir"/>
+  <property name="jar.src.dir" location="jar_spi_dir/src"/>
+  <property name="jar.src.file" location="jar_spi_dir/src/a_file"/>
+
+  <target name="init">
+    <mkdir dir="${jar.src.dir}"/>
+    <delete quiet="yes" file="${jar.src.file}"/>
+    <touch file="${jar.src.file}"/>
+    <delete quiet="yes" file="${jar.dir}/file.jar"/>
+    <delete quiet="yes" dir="${jar.dir}/output"/>
+  </target>
+
+  <target name="test-simple" depends="init">
+
+    <jar jarfile="${jar.dir}/file.jar">
+      <fileset dir="${jar.src.dir}"/>
+      <service type="a.b.c" provider="a.b.c.d"/>
+    </jar>
+
+    <unjar src="${jar.dir}/file.jar"
+           dest="${jar.dir}/output"/>
+
+    <loadfile property="simple"
+              srcfile="${jar.dir}/output/META-INF/service/a.b.c"
+              encoding="UTF-8"/>
+
+    <au:assertTrue>
+      <matches string="${simple}" pattern="^a\.b\.c\.d\n$"/>
+    </au:assertTrue>
+
+  </target>
+
+  <target name="test-providers" depends="init">
+
+    <jar jarfile="${jar.dir}/file.jar">
+      <fileset dir="${jar.src.dir}"/>
+      <service type="a.b.c">
+        <provider classname="a.X"/>
+        <provider classname="a.D"/>
+      </service>
+    </jar>
+
+    <unjar src="${jar.dir}/file.jar"
+           dest="${jar.dir}/output"/>
+
+    <loadfile property="providers"
+              srcfile="${jar.dir}/output/META-INF/service/a.b.c"
+              encoding="UTF-8"/>
+    <au:assertTrue>
+      <matches string="${providers}" pattern="^a\.X\na\.D$"/>
+    </au:assertTrue>
+
+  </target>
+
+  <target name="test-multi" depends="init">
+
+    <jar jarfile="${jar.dir}/file.jar">
+      <fileset dir="${jar.src.dir}"/>
+      <service type="a.b.c">
+        <provider classname="a.X"/>
+        <provider classname="a.D"/>
+      </service>
+      <service type="javax.a.service">
+        <provider classname="a.O.T"/>
+        <provider classname="a.B"/>
+      </service>
+    </jar>
+
+    <unjar src="${jar.dir}/file.jar"
+           dest="${jar.dir}/output"/>
+
+    <loadfile property="multi-a"
+              srcfile="${jar.dir}/output/META-INF/service/a.b.c"
+              encoding="UTF-8"/>
+
+    <au:assertTrue>
+      <matches string="${multi-a}" pattern="^a\.X\na\.D$"/>
+    </au:assertTrue>
+
+    <loadfile property="multi-b"
+              srcfile="${jar.dir}/output/META-INF/service/javax.a.service"
+              encoding="UTF-8"/>
+
+    <au:assertTrue>
+      <matches string="${multi-b}" pattern="^a\.O\.T\na\.B$"/>
+    </au:assertTrue>
+
+  </target>
+
+  <target name="test-reject-no-type" depends="init">
+    <au:expectfailure>
+      <jar jarfile="${jar.dir}/file.jar">
+        <fileset dir="${jar.src.dir}"/>
+        <service provider="a.X"/>
+      </jar>
+    </au:expectfailure>
+  </target>
+
+  <target name="test-reject-no-provider" depends="init">
+    <au:expectfailure>
+      <jar jarfile="${jar.dir}/file.jar">
+        <fileset dir="${jar.src.dir}"/>
+        <service type="a.X"/>
+      </jar>
+    </au:expectfailure>
+  </target>
+
+  <target name="test-reject-no-classname" depends="init">
+    <au:expectfailure>
+      <jar jarfile="${jar.dir}/file.jar">
+        <fileset dir="${jar.src.dir}"/>
+        <service type="a.X">
+          <provider/>
+        </service>
+      </jar>
+    </au:expectfailure>
+  </target>
+
+  <target name="tearDown">
+    <delete quiet="yes" dir="${jar.dir}"/>
+  </target>
+</project>

Propchange: ant/core/trunk/src/tests/antunit/taskdefs/jar-spi-test.xml
------------------------------------------------------------------------------
    svn:eol-style = native



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to