ASF Bugzilla – Attachment 29224 Details for
Bug 53713
Performance tuning solution for JspReader
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
[patch]
Patch for Mark.java
Mark.diff (text/plain), 1.36 KB, created by
Sheldon Shao
on 2012-08-14 06:29:38 UTC
(
hide
)
Description:
Patch for Mark.java
Filename:
MIME Type:
Creator:
Sheldon Shao
Created:
2012-08-14 06:29:38 UTC
Size:
1.36 KB
patch
obsolete
>Index: apache/jasper/compiler/Mark.java >=================================================================== >--- apache/jasper/compiler/Mark.java (revision 1370136) >+++ apache/jasper/compiler/Mark.java (working copy) >@@ -89,9 +89,23 @@ > * Constructor > */ > Mark(Mark other) { >- >- this.reader = other.reader; >- this.ctxt = other.reader.getJspCompilationContext(); >+ init(other); >+ } >+ >+ /** >+ * Simpler than {@link #init(Mark)}. To make good performance >+ * >+ * @param other >+ */ >+ void update(Mark other) { >+ this.cursor = other.cursor; >+ this.line = other.line; >+ this.col = other.col; >+ } >+ >+ void init(Mark other) { >+ this.reader = other.reader; >+ this.ctxt = other.ctxt; > this.stream = other.stream; > this.fileId = other.fileId; > this.fileName = other.fileName; >@@ -102,7 +116,12 @@ > this.encoding = other.encoding; > > // clone includeStack without cloning contents >- includeStack = new Stack<IncludeState>(); >+ if (includeStack != null) { >+ includeStack.clear(); >+ } >+ else { >+ includeStack = new Stack<IncludeState>(); >+ } > for ( int i=0; i < other.includeStack.size(); i++ ) { > includeStack.addElement( other.includeStack.elementAt(i) ); > }
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 Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 53713
:
29223
|
29224
|
29242
|
29243
|
29244
|
29245
|
29246