kusalk commented on code in PR #741:
URL: https://github.com/apache/struts/pull/741#discussion_r1309606372


##########
core/src/main/java/com/opensymphony/xwork2/ognl/SecurityMemberAccess.java:
##########
@@ -84,16 +86,17 @@ public Object setup(Map context, Object target, Member 
member, String propertyNa
 
     @Override
     public void restore(Map context, Object target, Member member, String 
propertyName, Object state) {
-        if (state != null) {
-            final AccessibleObject accessible = (AccessibleObject) member;
-            final boolean stateBoolean = ((Boolean) state).booleanValue();  // 
Using twice (avoid unboxing)

Review Comment:
   Not sure what this comment meant



-- 
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: issues-unsubscr...@struts.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to