You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@chemistry.apache.org by fm...@apache.org on 2012/04/13 13:47:40 UTC

svn commit: r1325723 - /chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java

Author: fmui
Date: Fri Apr 13 11:47:39 2012
New Revision: 1325723

URL: http://svn.apache.org/viewvc?rev=1325723&view=rev
Log:
Workbench: fixed possible NPE

Modified:
    chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java

Modified: chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java?rev=1325723&r1=1325722&r2=1325723&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java Fri Apr 13 11:47:39 2012
@@ -117,7 +117,7 @@ public class ClientHelper {
             exceptionName = ((CmisBaseException) ex).getExceptionName();
         }
 
-        StringBuilder sb = new StringBuilder(ex.getMessage());
+        StringBuilder sb = new StringBuilder(ex.getMessage() == null ? "null" : ex.getMessage());
 
         int width = 80;
         while (sb.length() > width) {



Re: svn commit: r1325723 - /chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java

Posted by Florent Guillaume <fg...@nuxeo.com>.
Or you could use String.valueOf() ;)

Florent

On Fri, Apr 13, 2012 at 1:47 PM,  <fm...@apache.org> wrote:
> Author: fmui
> Date: Fri Apr 13 11:47:39 2012
> New Revision: 1325723
>
> URL: http://svn.apache.org/viewvc?rev=1325723&view=rev
> Log:
> Workbench: fixed possible NPE
>
> Modified:
>    chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java
>
> Modified: chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java
> URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java?rev=1325723&r1=1325722&r2=1325723&view=diff
> ==============================================================================
> --- chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java (original)
> +++ chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java Fri Apr 13 11:47:39 2012
> @@ -117,7 +117,7 @@ public class ClientHelper {
>             exceptionName = ((CmisBaseException) ex).getExceptionName();
>         }
>
> -        StringBuilder sb = new StringBuilder(ex.getMessage());
> +        StringBuilder sb = new StringBuilder(ex.getMessage() == null ? "null" : ex.getMessage());
>
>         int width = 80;
>         while (sb.length() > width) {
>
>



-- 
Florent Guillaume, Director of R&D, Nuxeo
Open Source, Java EE based, Enterprise Content Management (ECM)
http://www.nuxeo.com   http://www.nuxeo.org   +33 1 40 33 79 87

Re: svn commit: r1325723 - /chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java

Posted by Florent Guillaume <fg...@nuxeo.com>.
Or you could use String.valueOf() ;)

Florent

On Fri, Apr 13, 2012 at 1:47 PM,  <fm...@apache.org> wrote:
> Author: fmui
> Date: Fri Apr 13 11:47:39 2012
> New Revision: 1325723
>
> URL: http://svn.apache.org/viewvc?rev=1325723&view=rev
> Log:
> Workbench: fixed possible NPE
>
> Modified:
>    chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java
>
> Modified: chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java
> URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java?rev=1325723&r1=1325722&r2=1325723&view=diff
> ==============================================================================
> --- chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java (original)
> +++ chemistry/opencmis/trunk/chemistry-opencmis-workbench/chemistry-opencmis-workbench/src/main/java/org/apache/chemistry/opencmis/workbench/ClientHelper.java Fri Apr 13 11:47:39 2012
> @@ -117,7 +117,7 @@ public class ClientHelper {
>             exceptionName = ((CmisBaseException) ex).getExceptionName();
>         }
>
> -        StringBuilder sb = new StringBuilder(ex.getMessage());
> +        StringBuilder sb = new StringBuilder(ex.getMessage() == null ? "null" : ex.getMessage());
>
>         int width = 80;
>         while (sb.length() > width) {
>
>



-- 
Florent Guillaume, Director of R&D, Nuxeo
Open Source, Java EE based, Enterprise Content Management (ECM)
http://www.nuxeo.com   http://www.nuxeo.org   +33 1 40 33 79 87