This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
Summary: | ArrayIndexOutOfBoundsException: 8 | ||
---|---|---|---|
Product: | editor | Reporter: | Petr Blaha <blaha> |
Component: | Formatting & Indentation | Assignee: | issues@editor <issues> |
Status: | VERIFIED FIXED | ||
Severity: | blocker | CC: | gord, jsedek, mschovanek, natie, nicolo, pslechta, schnappi, sustaining, the_lucius |
Priority: | P2 | ||
Version: | 6.x | ||
Hardware: | All | ||
OS: | All | ||
URL: | http://statistics.netbeans.org/exceptions/detail.do?id=11928 | ||
Issue Type: | DEFECT | Exception Reporter: | 11928 |
Attachments: |
sample file from project that generates AIOOBE on reformat
tested project release61_fixes version of editor.indent with the fix. Backup the original file in ide9/modules and replace it with the new one. Mercurial bundle with the fix. |
Description
Petr Blaha
2008-07-09 08:10:12 UTC
Has anybody seen this in a recent builds (*)? If so, please attach a stack trace from a recent build and reopen. Otherwise, this seems to be already fixed as part of issue #134217: http://hg.netbeans.org/main/rev/979ca7bbfe0a (*): on statistics, there are reports that claim to be from "080702" - but the line numbers do not match recent sources (last change to this file as June 12). latest stacktraces are comming from 61 fixes branch - that the reason of incorrect line numbers against trunk I think we should port AIOOBE fix in relase61 clone and release the fix in patch3. I'm not able to reproduce on latest buils neither on 6.1. I would like to kindly ask reporters, who reported the problem using IDE if they could give a sample file where they are able to reproduce the problem. This issue has lot's of duplicates and it would be good to be sure it's fixed. I'm using NB61, patch2 where this occurs in EVERY JSP page (I will attach one) where the file had been opened with NB60 and forced the formatting to 8 characters per indent (ignoring my 3 characters/indent setting). I have tried both deleting the .netbeans\6.1\var\cache and a new user dir (changed the user dir in netbeans.conf file). Nothing I've tried will make the reformatting work. I had downloaded an NBM (org-netbeans-modules-web-project.nbm - I don't remember the issue number where this file was linked) on 5/15/2008 that seemed to correct this (and hence my email to the nbusers list about a REGRESSION). If I remember correctly, that patch seemed to indicate that it was scheduled for inclusion into patch2 (obviously not). I would argue that this issue should be re-opened and the fix DEFINITELY be included in patch 3. Created attachment 64207 [details]
sample file from project that generates AIOOBE on reformat
jsedek: I am able to reproduce in vanilla NB6.1 using the attached file with the default indentation settings (Expand tabs to spaces=on, Indentation Size=4, Tab Size=8). gord: the process of getting fixes into a patch release requires (among other things) that the bug is fixed and verified in the trunk (the bug needs to be in "resolved verified" state). finally I'm able to reproduce the problem I use 6.1 fixes build from 2008-07-02_03-01-08 : smetiste.czech.sun.com/builds/netbeans/6.1/fixes/2008-07-02_03-01-08/zip/ start NB with fresh userdir open attached project use tools/options/editor/indentation and disable expand tabs to spaces close option open index.jsp invoke reformat AIOOBE is thrown with these line numbers: java.lang.ArrayIndexOutOfBoundsException: 72 at org.netbeans.modules.editor.indent.api.IndentUtils.cachedOrCreatedIndentString(IndentUtils.java:228) at org.netbeans.modules.editor.indent.api.IndentUtils.createIndentString(IndentUtils.java:195) at org.netbeans.modules.editor.indent.spi.Context.modifyIndent(Context.java:177) at org.netbeans.modules.editor.structure.formatting.TagBasedLexerFormatter.reformat(TagBasedLexerFormatter.java:358) at org.netbeans.modules.editor.structure.formatting.TagBasedLexerFormatter.reformat(TagBasedLexerFormatter.java:160) at org.netbeans.modules.editor.structure.formatting.TagBasedLexerFormatter.process(TagBasedLexerFormatter.java:155) I'll reopen this issue since a patch for 6.1 is needed I'm not able to reproduce in trunk - it's blocked by issue #139831 Created attachment 64932 [details]
tested project
Not sure why the bug was reopened - the bug should be reopened only if the problem is not actually fixed in the trunk. Please note that the bug needs to be resolved verified in order to be included in a patch release. I am attaching editor.indent module built from the release61_fixes branch with the fix for this problem and a mercurial bundle that contains the fix. Someone please verify the fix. Created attachment 65006 [details]
release61_fixes version of editor.indent with the fix. Backup the original file in ide9/modules and replace it with the new one.
Created attachment 65007 [details]
Mercurial bundle with the fix.
verified on 6.1 builds Correct me if I'm wrong, but based on the attached bundled changegroup and stacktraces it looks like this issue 139350 is pretty clear duplicate of the issue 134217. Please re-evaluate nomination of this duplicate issue, because the fix was in fact done for issue 134217 and the changegroup bundle attached by jlahoda contains reference to issue 134217 (not 139350) in commit log. This would make a fix for issue 134217 appear in NB 6.1 Patch3 w/o any reason, while the patch for issue 139350 would not be easily identifiable by commit log text. I suggest to do following: #1 remove 61fixes3-candidate keyword from status whiteboard of this issue 139350 #2 put the keyword 61fixes3-candidate to status whiteboard of the issue 134217 #3 reopen&close this issue 139350 as a duplicate of issue 134217 #4 (formally) verify issue 134217 Any thoughts/suggestions are welcome, -R I've discussed with Petr Blaha (blaha) and we agreed on not changing the keywords as it turned out the changeset http://hg.netbeans.org/main/rev/979ca7bbfe0a is actually a subset of solution for issue 134217, which also depends on resolution of issue 90403, which is not acceptable for NB 6.1 fixes. In this the solution would be to use the changeset http://hg.netbeans.org/main/rev/979ca7bbfe0a and change just commit log to match this particular issue. I've transplanted the changeset http://hg.netbeans.org/main/rev/979ca7bbfe0a into release61_fixes repository as http://hg.netbeans.org/release61_fixes/rev/bdeaaf094a95 I've changed commit message, but kept authorship and timestamp. changeset: 77854:bdeaaf094a95 tag: tip user: Vita Stejskal <vstejskal@netbeans.org> date: Mon May 05 12:46:22 2008 +0200 summary: #139350: ArrayIndexOutOfBoundsException: 8 |