reta commented on a change in pull request #387: [CXF-7640] Create a form to 
set the use of Spring in the classHelper …
URL: https://github.com/apache/cxf/pull/387#discussion_r175261686
 
 

 ##########
 File path: core/src/main/java/org/apache/cxf/common/util/ClassHelper.java
 ##########
 @@ -69,19 +76,44 @@ protected Object getRealObjectInternal(Object o) {
     }
 
     public static Class<?> getRealClassFromClass(Class<?> cls) {
-        return HELPER.getRealClassFromClassInternal(cls);
+        Bus bus = getBus(null);
+        if (checkUseDefaultClassHelper(bus)) {
 
 Review comment:
   We could get rid of some duplication here:
   
   ```
   private static getContextClassHelper(Bus bus) {
       // We could check if DEFAULT_HELPER and HELPER  are referencing the same 
instance and than the bus property
       return (DEFAULT_HELPER == HELPER || checkUseDefaultClassHelper(bus)) ? 
DEFAULT_HELPER : HELPER;
   }
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to