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.

Bug 165960 - IllegalArgumentException appears after Ctrl-R
Summary: IllegalArgumentException appears after Ctrl-R
Status: VERIFIED FIXED
Alias: None
Product: cnd
Classification: Unclassified
Component: Code Model (show other bugs)
Version: 6.x
Hardware: All All
: P1 blocker (vote)
Assignee: Vladimir Voskresensky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-25 15:18 UTC by soldatov
Modified: 2009-05-28 14:45 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description soldatov 2009-05-25 15:18:35 UTC
Scenario:
- Create IO sample
- Open io.c file
- Find 'argv' string and call context menu. Select Refactor|Rename... menu item
- Type 'argv2' and press 'Refactor' button
- Press Ctrl-R
==> 
java.lang.IllegalArgumentException: No highlight contains the caret.
	at org.netbeans.modules.cnd.refactoring.actions.InstantRenamePerformer.<init>(InstantRenamePerformer.java:126)
	at
org.netbeans.modules.cnd.refactoring.actions.InstantRenamePerformer.performInstantRename(InstantRenamePerformer.java:243)
	at org.netbeans.modules.cnd.refactoring.actions.InstantRenamePerformer.doInstantRename(InstantRenamePerformer.java:226)
	at org.netbeans.modules.cnd.refactoring.actions.InstantRenamePerformer.invokeInstantRename(InstantRenamePerformer.java:178)
	at org.netbeans.modules.cnd.refactoring.actions.InstantRenameAction.actionPerformed(InstantRenameAction.java:62)
	at org.netbeans.editor.BaseAction.actionPerformed(BaseAction.java:325)
	at javax.swing.SwingUtilities.notifyAction(SwingUtilities.java:1636)
	at javax.swing.JComponent.processKeyBinding(JComponent.java:2849)
	at javax.swing.JComponent.processKeyBindings(JComponent.java:2884)
	at javax.swing.JComponent.processKeyEvent(JComponent.java:2812)
	at java.awt.Component.processEvent(Component.java:5993)
	at java.awt.Container.processEvent(Container.java:2041)
	at java.awt.Component.dispatchEventImpl(Component.java:4583)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Component.dispatchEvent(Component.java:4413)
	at java.awt.KeyboardFocusManager.redispatchEvent(KeyboardFocusManager.java:1848)
	at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(DefaultKeyboardFocusManager.java:704)
	at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(DefaultKeyboardFocusManager.java:969)
	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(DefaultKeyboardFocusManager.java:841)
	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(DefaultKeyboardFocusManager.java:668)
	at java.awt.Component.dispatchEventImpl(Component.java:4455)
	at java.awt.Container.dispatchEventImpl(Container.java:2099)
	at java.awt.Window.dispatchEventImpl(Window.java:2475)
	at java.awt.Component.dispatchEvent(Component.java:4413)
[catch] at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
	at org.netbeans.core.TimableEventQueue.dispatchEvent(TimableEventQueue.java:104)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)
Comment 1 Vladimir Voskresensky 2009-05-25 17:03:15 UTC
fixed
http://hg.netbeans.org/cnd-main/rev/7964d9795dbe

Please, verify.


Comment 2 Quality Engineering 2009-05-26 07:30:11 UTC
Integrated into 'main-golden', will be available in build *200905260201* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/7964d9795dbe
User: Vladimir Voskresensky <vv159170@netbeans.org>
Log: fixed IZ#165960: IllegalArgumentException appears after Ctrl-R
Comment 3 Alexander Simon 2009-05-26 08:27:07 UTC
I have reviewed the changes and they are safe and correct.
IMHO, we need to integrate the fix into 6.7.
One small note:
It is bad style to throw uncached exception in EDT thread.
Please, replace code:
	    throw new IllegalArgumentException("No highlight contains the caret."); // NOI18N
Possible variants:
- only log exception
- add throws in method definition
Comment 4 Vladimir Voskresensky 2009-05-26 09:32:01 UTC
reopen to integrate into release67 
Comment 5 soldatov 2009-05-26 09:53:54 UTC
verified in 200905260201 build from http://bits.netbeans.org/dev/nightly/
Comment 6 Alexander Pepin 2009-05-26 14:12:43 UTC
please integrate the fix into release67
Comment 7 Marian Mirilovic 2009-05-26 15:40:40 UTC
Would be nice to have this fixed in release67 today (ASAP), thanks in advance.
Comment 8 Vladimir Voskresensky 2009-05-26 15:50:16 UTC
fix was integrated in 67
http://hg.netbeans.org/release67?cmd=changeset;node=f69cbbef6f2e
Comment 9 soldatov 2009-05-28 14:45:35 UTC
verified in NetBeans 6.7rc1