Red Hat Bugzilla – Bug 176574
OutOfMemory Exception during th refresh for Eclipse's Bugzilla
Last modified: 2007-11-30 17:11:19 EST
From Bugzilla Helper: User-Agent: Mozilla/5.0 (X11; U; Linux ppc; en-US; rv:1.8) Gecko/20051202 Ubuntu/1.4.99+1.5rc3.dfsg-1ubuntu4 Epiphany/1.9.3.1 Firefox/1.5 Description of problem: Hi, the plugin doesn't work with Eclipse's Bugzilla. I got a OutOfMemoryException during the refresh. I created a patch against the CVS HEAD to fix various problems I got. Problems, I solved: 1. OOME during the parsing of Eclipse's Bugzilla query page. 2. NPE if I deselect all tree items. 3. Error in the creation of the status object for exceptions. 4. Read content type for attachments, before I got problems because the id of the attachment wasn't read correct. 5. Make borders around lists. 6. UTF8(whatever)Exception because a java.io.Reader was used to read the binary content. 7. Use bugzillaAttachment.getDataAsString() only for text/* types. 8. Add Ubuntu's bugzilla Version-Release number of selected component (if applicable): How reproducible: Always Steps to Reproduce: 1. 2. 3. Additional info:
Created attachment 122587 [details] Patch against org.eclipse.team.bugs (CVS HEAD)
Created attachment 122588 [details] Patch against org.eclipse.team.bugzilla (CVS HEAD)
Created attachment 122845 [details] Revised patch Hi Stephan, Thanks for such an extensive patch to the plugin :). I've tested your patch and it seems to work well. I've made some small syntactic changes, such as leaving the explicit imports instead of .*, as well as making the instantiation of the Status object in BugzillaPlugin a bit more readable. If nothing else comes up I'll commit this.
Created attachment 122896 [details] Revised patch2 I am committing the patch, with some other minor changes, this is the final form. Stephan, thanks again for the work.
The patch that you attached is a little messed up, Igor. Lots of the files are completely changed according to the patch ... maybe you re-indented? It'd be cool if the contextual differences were more apparent. Thanks.
Opening the patch up in vi shows that it's most likely that the files that were changed without real changes had windows line endings. Since I already comitted to CVS and I don't see converting win line endings to unix line endings as a big transgression, I'll leave it in CVS as is, but I'll repost a patch without these files for easier reading. :) Igor
Created attachment 122953 [details] Cleaned up patch Here's a patch without the line-ending changes.