cxf-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dk...@apache.org
Subject cxf git commit: Update -Psetup.eclipse to setup workspace based on changes needed for latest checkstyle
Date Thu, 21 Jul 2016 15:02:32 GMT
Repository: cxf
Updated Branches:
  refs/heads/master b3677b6a9 -> b8ab7eaf4


Update -Psetup.eclipse to setup workspace based on changes needed for latest checkstyle


Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/b8ab7eaf
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/b8ab7eaf
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/b8ab7eaf

Branch: refs/heads/master
Commit: b8ab7eaf4d070cf76c4f9088cefe1c0806cca1cd
Parents: b3677b6
Author: Daniel Kulp <dkulp@apache.org>
Authored: Thu Jul 21 10:58:37 2016 -0400
Committer: Daniel Kulp <dkulp@apache.org>
Committed: Thu Jul 21 10:58:37 2016 -0400

----------------------------------------------------------------------
 etc/eclipse/template.checkstyle-config.xml                  | 9 +++++++--
 pom.xml                                                     | 4 ++++
 .../java/org/apache/cxf/binding/corba/CorbaConduit.java     | 9 ++++-----
 3 files changed, 15 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf/blob/b8ab7eaf/etc/eclipse/template.checkstyle-config.xml
----------------------------------------------------------------------
diff --git a/etc/eclipse/template.checkstyle-config.xml b/etc/eclipse/template.checkstyle-config.xml
index 8f6dbe4..ac7151a 100644
--- a/etc/eclipse/template.checkstyle-config.xml
+++ b/etc/eclipse/template.checkstyle-config.xml
@@ -18,6 +18,11 @@
   under the License.
 -->
 <checkstyle-configurations file-format-version="5.0.0">
-    <check-configuration name="CXF Checks" location="@CHECKSTYLE_CONFIG_FILE@" type="external"
description=""/>
-    <check-configuration name="CXF CORBA Checks" location="@CHECKSTYLE_CORBA_CONFIG_FILE@"
type="external" description=""/>
+    <check-configuration name="CXF Checks" location="@CHECKSTYLE_CONFIG_FILE@" type="external"
description="">
+        <property name="checkstyle.suppressions.file" value="@CHECKSTYLE_SUPPRESSION_FILE@"
/>
+    </check-configuration>
+    <check-configuration name="CXF CORBA Checks" location="@CHECKSTYLE_CORBA_CONFIG_FILE@"
type="external" description="">
+        <property name="checkstyle.suppressions.file" value="@CHECKSTYLE_SUPPRESSION_FILE@"
/>
+    </check-configuration>
+
 </checkstyle-configurations>

http://git-wip-us.apache.org/repos/asf/cxf/blob/b8ab7eaf/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index f11303f..3a5225e 100644
--- a/pom.xml
+++ b/pom.xml
@@ -276,14 +276,17 @@
                                         <whichresource resource="/cxf-eclipse-checkstyle"
property="eclipse.checkstyle.url" />
                                         <whichresource resource="/cxf-checkstyle.xml"
property="checkstyle.url" />
                                         <whichresource resource="/cxf-checkstyle-corba.xml"
property="corba.checkstyle.url" />
+                                        <whichresource resource="/cxf-checkstyle-suppressions.xml"
property="checkstyle.suppression.url" />
                                         <mkdir dir="${full.eclipse.workspace}/.metadata/.plugins/org.eclipse.core.runtime/.settings"
/>
                                         <mkdir dir="${full.eclipse.workspace}/.metadata/.plugins/com.atlassw.tools.eclipse.checkstyle"
/>
                                         <mkdir dir="${full.eclipse.workspace}/.metadata/.plugins/net.sourceforge.pmd.eclipse"
/>
                                         <get src="${checkstyle.url}" dest="${full.eclipse.workspace}/cxf-checkstyle.xml"
/>
+                                        <get src="${checkstyle.suppression.url}" dest="${full.eclipse.workspace}/cxf-checkstyle-suppressions.xml"
/>
                                         <get src="${corba.checkstyle.url}" dest="${full.eclipse.workspace}/cxf-checkstyle-corba.xml"
/>
                                         <!-- Add checkstyle config -->
                                         <copy file="${basedir}/etc/eclipse/template.checkstyle-config.xml"
tofile="${full.eclipse.workspace}/.metadata/.plugins/com.atlassw.tools.eclipse.checkstyle/checkstyle-config.xml"
overwrite="no">
                                             <filterset>
+                                                <filter token="CHECKSTYLE_SUPPRESSION_FILE"
value="${full.eclipse.workspace}/cxf-checkstyle-suppressions.xml" />
                                                 <filter token="CHECKSTYLE_CONFIG_FILE"
value="${full.eclipse.workspace}/cxf-checkstyle.xml" />
                                                 <filter token="CHECKSTYLE_CORBA_CONFIG_FILE"
value="${full.eclipse.workspace}/cxf-checkstyle-corba.xml" />
                                                 <filter token="APACHE_HEADER_FILE" value="${full.eclipse.workspace}/apache-header.txt"
/>
@@ -297,6 +300,7 @@
                                         <!-- Add checkstyle config for 5.0 plugin -->
                                         <copy file="${basedir}/etc/eclipse/template.checkstyle-config.xml"
tofile="${full.eclipse.workspace}/.metadata/.plugins/net.sf.eclipsecs.core/checkstyle-config.xml"
overwrite="no">
                                             <filterset>
+                                                <filter token="CHECKSTYLE_SUPPRESSION_FILE"
value="${full.eclipse.workspace}/cxf-checkstyle-suppressions.xml" />
                                                 <filter token="CHECKSTYLE_CONFIG_FILE"
value="${full.eclipse.workspace}/cxf-checkstyle.xml" />
                                                 <filter token="CHECKSTYLE_CORBA_CONFIG_FILE"
value="${full.eclipse.workspace}/cxf-checkstyle-corba.xml" />
                                                 <filter token="APACHE_HEADER_FILE" value="${full.eclipse.workspace}/apache-header.txt"
/>

http://git-wip-us.apache.org/repos/asf/cxf/blob/b8ab7eaf/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/CorbaConduit.java
----------------------------------------------------------------------
diff --git a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/CorbaConduit.java
b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/CorbaConduit.java
index 7518ece..98218ca 100644
--- a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/CorbaConduit.java
+++ b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/CorbaConduit.java
@@ -189,14 +189,13 @@ public class CorbaConduit implements Conduit {
         if (request == null) {
             throw new CorbaBindingException("Couldn't build the corba request");
         }
+        Exception ex = null;
         try {
             request.invoke();
-        } catch (SystemException ex) {
-            message.setContent(Exception.class, new Fault(ex));
-            message.setSystemException(ex);
-            return;
+            ex = request.env().exception();
+        } catch (SystemException sysex) {
+            ex = sysex;
         }
-        Exception ex = request.env().exception();
         if (ex != null) {
             if (ex instanceof SystemException) {
                 message.setContent(Exception.class, new Fault(ex));


Mime
View raw message