Re: svn commit: r1148432 - in /cxf/branches/2.4.x-fixes: ./ rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/ext/codegen/SourceGenerator.java

[Sergey Beryozkin]

Hi Glen
@@ -583,7 +583,7 @@ public class SourceGenerator {
                  for (int i = 0; i<  responseEls.size(); i++) {
                      String statusValue = 
responseEls.get(0).getAttribute("status");
                      try {
-                        int status = statusValue.isEmpty() ? 200 : 
Integer.valueOf(statusValue);
+                        int status = statusValue.length() == 0 ? 200 
: Integer.valueOf(statusValue);

Unsure of the need for this change, according to the JavaDoc, the 
above two statements are equivalent:
http://download.oracle.com/javase/6/docs/api/java/lang/String.html#isEmpty%28%29 



That was a fix for Java 5 compilation error, where String.length() does 
not exist

Cheers, Sergey

Glen


                          if (status == 200) {
                              okResponse = responseEls.get(i);
                              break;