[tomcat] branch main updated: Fix possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new 83898d6471 Fix possible NPE 83898d6471 is described be

[tomcat] branch 10.1.x updated: Fix possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new 0fe267fa7d Fix possible NPE 0fe267fa7d is describe

[tomcat] branch 9.0.x updated: Fix possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new 8865396318 Fix possible NPE 8865396318 is described

[tomcat] branch main updated: Fix NPE and sync the stat

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new f7f1197459 Fix NPE and sync the stat f7f1197459 is des

[tomcat] branch 10.1.x updated: Fix NPE and sync the stat

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new daa4e108a1 Fix NPE and sync the stat daa4e108a1 is

[tomcat] branch 9.0.x updated: Sync the stat

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new b933447ecf Sync the stat b933447ecf is described bel

[tomcat] branch 8.5.x updated: Sync the stat

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 106f65aa68 Sync the stat 106f65aa68 is described bel

[tomcat] branch main updated: Default to Object for type, just in case

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new bde91d43f2 Default to Object for type, just in case bd

[tomcat] branch 10.1.x updated: Default to Object for type, just in case

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new 5d716263fa Default to Object for type, just in cas

[tomcat] branch 9.0.x updated: Default to Object for type, just in case

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new 833040c7c4 Default to Object for type, just in case

[tomcat] branch 8.5.x updated: Default to Object for type, just in case

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 8a5946414e Default to Object for type, just in case

[tomcat] branch main updated: Fix possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new 4270673737 Fix possible NPE 4270673737 is described be

[tomcat] branch 10.1.x updated: Fix possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new 2e8e3d33aa Fix possible NPE 2e8e3d33aa is describe

[tomcat] branch 9.0.x updated: Fix possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new 6a3fc9b1c5 Fix possible NPE 6a3fc9b1c5 is described

[tomcat] branch 8.5.x updated: Fix possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 6478ab72ac Fix possible NPE 6478ab72ac is described

[tomcat] branch main updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new e39e5e886f Avoid possible NPE e39e5e886f is described

[tomcat] branch 10.1.x updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new 7e6e7bcaa9 Avoid possible NPE 7e6e7bcaa9 is descri

[tomcat] branch 9.0.x updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new a23f69b46d Avoid possible NPE a23f69b46d is describe

[tomcat] branch 8.5.x updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new dcdf981ca9 Avoid possible NPE dcdf981ca9 is describe

Re: Potential performance improvement

2023-09-14 Thread Christopher Schultz
Mark, On 9/13/23 15:07, Mark Thomas wrote: So we are talking about a saving of around 35 nanoseconds per call. I just ran Tomcat 10.1.x through a profiler and requesting the Tomcat homepage triggered String.toLowerCase() just under 100 times (some of those calls may be from other JRE methods)

[tomcat] branch main updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new 3cf8b95731 Avoid possible NPE 3cf8b95731 is described

[tomcat] branch 10.1.x updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new 0fafa69214 Avoid possible NPE 0fafa69214 is descri

[tomcat] branch 9.0.x updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new a0a5fc2196 Avoid possible NPE a0a5fc2196 is describe

[tomcat] branch 8.5.x updated: Avoid possible NPE

2023-09-14 Thread remm
This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 6605420694 Avoid possible NPE 6605420694 is describe

[GitHub] [tomcat] anuragdy commented on pull request #666: Performance improvements for ImplicitObjectELResolver

2023-09-14 Thread via GitHub
anuragdy commented on PR #666: URL: https://github.com/apache/tomcat/pull/666#issuecomment-1719698843 Hi @michael-o I wonder if you have any further thoughts/comments on this change and if we could finish the Code Review process soon. -- This is an automated message from the Apache Git Se

[GitHub] [tomcat] michael-o commented on pull request #666: Performance improvements for ImplicitObjectELResolver

2023-09-14 Thread via GitHub
michael-o commented on PR #666: URL: https://github.com/apache/tomcat/pull/666#issuecomment-1719703370 I cannot make any judgement on any performance improvement. I'd prefer to leave this to other committers. -- This is an automated message from the Apache Git Service. To respond to the m

[GitHub] [tomcat] markt-asf commented on pull request #669: Relocate the useCompression check and apply parts back to back

2023-09-14 Thread via GitHub
markt-asf commented on PR #669: URL: https://github.com/apache/tomcat/pull/669#issuecomment-1719734655 Re-opening. Discussion on the dev list confirmed that this change was safe. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [tomcat] markt-asf merged pull request #669: Relocate the useCompression check and apply parts back to back

2023-09-14 Thread via GitHub
markt-asf merged PR #669: URL: https://github.com/apache/tomcat/pull/669 -- 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: dev-unsubscr...@tomcat.apache.o

[tomcat] branch main updated: Relocate the useCompression check and apply parts back to back.

2023-09-14 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new e47d0c46fe Relocate the useCompression check and appl

[tomcat] branch 10.1.x updated: Relocate the useCompression check and apply parts back to back.

2023-09-14 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 10.1.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/10.1.x by this push: new 97c58599b2 Relocate the useCompression check and

[tomcat] branch 9.0.x updated: Relocate the useCompression check and apply parts back to back.

2023-09-14 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new eca8f44c76 Relocate the useCompression check and ap

[tomcat] branch 8.5.x updated: Relocate the useCompression check and apply parts back to back.

2023-09-14 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 7abdf0b44c Relocate the useCompression check and ap

[GitHub] [tomcat] shin-mallang commented on pull request #669: Relocate the useCompression check and apply parts back to back

2023-09-14 Thread via GitHub
shin-mallang commented on PR #669: URL: https://github.com/apache/tomcat/pull/669#issuecomment-1719742560 @markt-asf I apologize for not sending a PR with an explanation after carefully checking that the change in the code was safe. I apologize for the inconvenience. Thank you. -