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