[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 Bug 56955 depends on bug 56953, which changed state. Bug 56953 Summary: A improvement for DataInputStream https://issues.apache.org/bugzilla/show_bug.cgi?id=56953 What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 --- Comment #6 from Mark Thomas ma...@apache.org --- If you continue to ignore the comments you are given then this issue is going to get closed as WONTFIX. Your first patch was 117k and full of irrelevant changes. The second attempt is worse at 130k. Starting at the beginning of the patch: 1. The first chunk changes restores an svn keyword the Tomcat team previously removed. This change has nothing to do with this issue and should not be in this patch. 2. The second chunk reverts a fix to a constant name the Tomcat team previously fixed and removes some code necessary for Java 8 support. This change has nothing to do with this issue and should not be in this patch. Further this change breaks Java 8 support. 3. The third chunk makes further changes that have nothing to do with this issue and further breaks Java 8 support. And so on. To repeat: - Patches should be against http://svn.apache.org/viewvc/tomcat/trunk/ - If you'd like to propose changes to back port form there to tc7.0.x/trunk then provide some revision numbers. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 --- Comment #7 from hzha...@ebay.com --- Sorry for any inconvenience. I am now reviewing my code. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 Mark Thomas ma...@apache.org changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #8 from Mark Thomas ma...@apache.org --- Numerous BCEL changes have been back-ported from trunk for the next 7.0.x release including ones that cover this report. The new code will be in 7.0.56 onwards. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 --- Comment #1 from hzha...@ebay.com --- Created attachment 31997 -- https://issues.apache.org/bugzilla/attachment.cgi?id=31997action=edit test case -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 hzha...@ebay.com changed: What|Removed |Added OS||All --- Comment #2 from hzha...@ebay.com --- Benefit (got by test case in the attachment): =lots of jar files= Normal: 551 Skip: 453 =few jar files= Normal: 88 Skip: 65 notice: 1.Skip version calls before original version in the test case, so the performance should be better. 2.It works more efficient in large projects. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 --- Comment #3 from Mark Thomas ma...@apache.org --- Several comments. A number of these changes have already been made in 8.0.x and can be back-ported. svn revisions to back-port are therefore preferred to new patches. There is no need to make skipping optional. The information is either required (in which case we keep it) or not (in which case we dump it). Unless a bug is specific to a major release, patches should always be against tomcat/trunk in svn and will be back-ported to the version the bug was opened against. This bug report prompted me to review (again) what information we keep in BCEL and I have removed a largish chunk that isn't covered by this patch. I'll take another look at the class parser next. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 --- Comment #4 from Mark Thomas ma...@apache.org --- Some further comments: The patch should address a single concern. Most of the provided patch has nothing to do with this bug report. There are a lot of changes that appear to undo previous clean-up. I recommend testing with tomcat/trunk (currently the 8.0.x development branch). A lot more changes have been made to that branch that haven't yet been backported to 7.0.x. -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
[Bug 56955] Skip some useless information in the class when parsing
https://issues.apache.org/bugzilla/show_bug.cgi?id=56955 hzha...@ebay.com changed: What|Removed |Added Attachment #31996|0 |1 is obsolete|| Attachment #31997|0 |1 is obsolete|| --- Comment #5 from hzha...@ebay.com --- Created attachment 32009 -- https://issues.apache.org/bugzilla/attachment.cgi?id=32009action=edit patch -- You are receiving this mail because: You are the assignee for the bug. - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
