[ https://issues.apache.org/jira/browse/MYFACES-4239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16520828#comment-16520828 ]
ASF GitHub Bot commented on MYFACES-4239: ----------------------------------------- tandraschko commented on issue #8: MYFACES-4239: Multiple performance improvements URL: https://github.com/apache/myfaces/pull/8#issuecomment-399597447 I also thought a bit about it, i think it would be better if we discuss it first on the mailing list. And details about it would be great. You know, we already have a buffer. So could you maybe split the sb/out commits from the other improvements? ---------------------------------------------------------------- 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: us...@infra.apache.org > [perf] Additional performance improvements > ------------------------------------------ > > Key: MYFACES-4239 > URL: https://issues.apache.org/jira/browse/MYFACES-4239 > Project: MyFaces Core > Issue Type: Improvement > Components: JSR-372 > Affects Versions: 2.2.12, 2.3.1 > Reporter: Paul Nicolucci > Assignee: Paul Nicolucci > Priority: Minor > Fix For: 2.2.13, 2.3.2 > > > Some performance improvements in : > 1) ApplicationImpl.java > 2) ServletExternalContextImpl.java > 3) HtmlResponseWriterImpl.java > 4) HTMLEncoder.java, we also discussed on the mailing list changing > encodeURIAtributte to encodeUriAttribute to fix the typo in the method name > so I'll do that here as well. > 5) CDataEndEscapeFilterWriter.java > 6) ResourceValidationUtils.java > The following changes were made: > - Skip calling ConcurrentHashMap.containsKey since we will call get > afterward if containsKey == true. > - Change to use StringBuilders instead of calling Writer.write so often. > - Stop using Boolean for variables that don't have a null meaning. If > null == false, then just use boolean with a default of false. > - Don't call String.length() constantly for String variables that aren't > re-assigned. > - Change conditional order to avoid calling validateResourceName unless the > other conditions are true -- This message was sent by Atlassian JIRA (v7.6.3#76005)