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 39281

Summary:	IDE freezes when docking some windows.
Product:	obsolete	Reporter:	Marian Mirilovic <mmirilovic>
Component:	vcscore	Assignee:	Martin Entlicher <mentlicher>
Status:	VERIFIED FIXED
Severity:	blocker	CC:	mslama
Priority:	P1
Version:	3.x
Hardware:	All
OS:	All
Issue Type:	DEFECT	Exception Reporter:
Attachments:	Thread-dump Diff of fix

Description Marian Mirilovic 2004-01-27 15:13:50 UTC

Issue transfered from BugTraq
http://developer.java.sun.com/developer/bugParade/bugs/4983987.html

=========================

Steps to reproduce:

1. Select "Mount|Version control|Generic VCS"
2. Select CVS profile
3. Type existent path in "Working Directory" field.
4. Press "Finish"
5. On "Runtime" tab select mounted filesystem and
open "Output of VCS
   Commands" window by double-clicking any VCS
command in history tree.
6. In main IDE window select "Window|Dock View
Into|Output Window|Left"
7. In main IDE window select "Window|Dock View
Into|Output
   Window|Right"

IDE hangs, 100% CPU is used.

=====================

Reproducible on NB 3.5.1 (attached is thread-dump)

Comment 1 Marian Mirilovic 2004-01-27 15:14:25 UTC

Created attachment 13097 [details]
Thread-dump

Comment 2 mslama 2004-01-27 15:31:05 UTC

As I investigated problem I found it is caused by call of close() from
CommandOutputVisualizer.writeReplace(). Fix is to remove this call. I
have no idea why close() is called there.

Comment 3 mslama 2004-01-27 15:32:35 UTC

Created attachment 13098 [details]
Diff of fix

Comment 4 Martin Entlicher 2004-01-28 18:17:14 UTC

Thanks for the investigation!
I'm testing the patch. close() really does not have much sense there.

Comment 5 Martin Entlicher 2004-01-28 18:23:58 UTC

The patch seems to work O.K. Fixed in trunk:

/cvs/vcscore/src/org/netbeans/modules/vcscore/commands/CommandOutputVisualizer.java,v
 <--  CommandOutputVisualizer.java
new revision: 1.32; previous revision: 1.31

Comment 6 Marian Mirilovic 2004-02-24 10:13:20 UTC

It works for me in the trunk build - verified.