ASF Bugzilla – Attachment 19538 Details for
Bug 41558
Unnecessary synchronization caused by JspFactory.getDefaultFactory while servicing JSPs
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
Difference in the generated code for hello_jsp.java before and after the patch
hello_jsp.diff (text/plain), 1.80 KB, created by
Arvind Srinivasan
on 2007-02-07 06:08:14 UTC
(
hide
)
Description:
Difference in the generated code for hello_jsp.java before and after the patch
Filename:
MIME Type:
Creator:
Arvind Srinivasan
Created:
2007-02-07 06:08:14 UTC
Size:
1.80 KB
patch
obsolete
>--- /tmp/hello_jsp.java Wed Feb 7 18:06:35 2007 >+++ output/build/work/Catalina/localhost/_/org/apache/jsp/hello_jsp.java Wed Feb 7 19:26:13 2007 >@@ -7,6 +7,8 @@ > public final class hello_jsp extends org.apache.jasper.runtime.HttpJspBase > implements org.apache.jasper.runtime.JspSourceDependent { > >+ private static final JspFactory _jspxFactory = JspFactory.getDefaultFactory(); >+ > private static java.util.List _jspx_dependants; > > private javax.el.ExpressionFactory _el_expressionfactory; >@@ -17,7 +19,7 @@ > } > > public void _jspInit() { >- _el_expressionfactory = JspFactory.getDefaultFactory().getJspApplicationContext(getServletConfig().getServletContext()).getExpressionFactory(); >+ _el_expressionfactory = _jspxFactory.getJspApplicationContext(getServletConfig().getServletContext()).getExpressionFactory(); > _jsp_annotationprocessor = (org.apache.AnnotationProcessor) getServletConfig().getServletContext().getAttribute(org.apache.AnnotationProcessor.class.getName()); > } > >@@ -27,7 +29,6 @@ > public void _jspService(HttpServletRequest request, HttpServletResponse response) > throws java.io.IOException, ServletException { > >- JspFactory _jspxFactory = null; > PageContext pageContext = null; > HttpSession session = null; > ServletContext application = null; >@@ -39,7 +40,6 @@ > > > try { >- _jspxFactory = JspFactory.getDefaultFactory(); > response.setContentType("text/html"); > pageContext = _jspxFactory.getPageContext(this, request, response, > null, true, 8192, true); >@@ -59,7 +59,7 @@ > if (_jspx_page_context != null) _jspx_page_context.handlePageException(t); > } > } finally { >- if (_jspxFactory != null) _jspxFactory.releasePageContext(_jspx_page_context); >+ _jspxFactory.releasePageContext(_jspx_page_context); > } > } > }
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Raw
Actions:
View
Attachments on
bug 41558
:
19537
| 19538