reta commented on code in PR #1542:
URL: https://github.com/apache/cxf/pull/1542#discussion_r1421780498
##########
rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/utils/JAXBUtilsTest.java:
##########
@@ -45,7 +48,10 @@ public void extendedXmlJavaTypeAdapter() {
}
private void correctValueType(Class<?> clazz) {
- Field field = clazz.getDeclaredFields()[0];
+ Field[] fields = clazz.getDeclaredFields();
Review Comment:
Thanks @prathyushreddylpr , this is a good finding, I don't see any reasons
why this classes are not declared as static, could you please change the
declaration for those? thank you
##########
rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/utils/JAXBUtilsTest.java:
##########
@@ -45,7 +48,10 @@ public void extendedXmlJavaTypeAdapter() {
}
private void correctValueType(Class<?> clazz) {
- Field field = clazz.getDeclaredFields()[0];
+ Field[] fields = clazz.getDeclaredFields();
Review Comment:
Thanks @prathyushreddylpr , this is a good finding, I don't see any reasons
why these classes are not declared as static, could you please change the
declaration for those? thank you
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
