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: | Deadlock After pressing Go back to button in editor toolbar | ||
|---|---|---|---|
| Product: | editor | Reporter: | Petr Cyhelsky <cyhelsky> |
| Component: | -- Other -- | Assignee: | Vitezslav Stejskal <vstejskal> |
| Status: | RESOLVED FIXED | ||
| Severity: | blocker | Keywords: | RANDOM, THREAD |
| Priority: | P2 | ||
| Version: | 6.x | ||
| Hardware: | PC | ||
| OS: | Windows Vista | ||
| Issue Type: | DEFECT | Exception Reporter: | |
| Attachments: |
ThreadDump
Removing extWriteLock/Unlock |
||
|
Description
Petr Cyhelsky
2009-09-23 11:08:48 UTC
Created attachment 88174 [details]
ThreadDump
"org.openide.text Document Processing" vs. "AWT-EventQueue-1" - I'll have a closer look. Created attachment 88347 [details]
Removing extWriteLock/Unlock
GuardedDocument uses extWriteLock/Unlock when adding DrawLayer for a style, which IMO is unnecessary. Even acquiring a readlock is not neccessary for that matter. See the attached patch. I'll discuss it with Mila next week. I'm obviously leaving alone the fact that both NbDocument, DocumentLine and GuardedDocument is in this part using long time deprecated stuff. Plus GuardedDocument implements StyleDocument is now a pure historical artifact. This implementation must have lost its meaning ages ago with invention of editor annotations... Since the addStyledLayer() leads to firing of a DocumentEvent we agreed with Vita that we will surround the addStyledLayer() with readLock(). |