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 244744 - Get rid of openide.text dependencies
Summary: Get rid of openide.text dependencies
Alias: None
Product: java
Classification: Unclassified
Component: Source (show other bugs)
Version: 8.0
Hardware: PC Linux
: P3 normal (vote)
Assignee: Svata Dedic
Depends on:
Blocks: 247200
  Show dependency tree
Reported: 2014-05-27 13:25 UTC by Svata Dedic
Modified: 2014-10-18 05:12 UTC (History)
4 users (show)

See Also:
Issue Type: TASK
Exception Reporter:


Note You need to log in before you can comment on or make changes to this bug.
Description Svata Dedic 2014-05-27 13:25:06 UTC
A few references to openide.text API is still in java.source.base - NbDocument and PositionRef.
According to offline discussion, PositionRef can be removed from the API (with some provisions for backward binary compatibility), NbDocument services can be replaced by other APIs introduced recently in editor.document module.

The API change (compatible) is that org.openide.text.PositionRef will implement j.s.t.Position directly (discussed with Tomas Z.) - this allows compatibility bridges to extract necessary data for old clients.

Change is to be committed into server_split jet-main branch.
Comment 1 Svata Dedic 2014-05-28 07:27:20 UTC
Additional fixes were necessary in:

* j2ee.ejbverification/src/org/netbeans/modules/j2ee/ejbverification/fixes/
* websocket/src/org/netbeans/modules/websocket/editor/ 

Please review whether the fix is acceptable - saving the file (verification) may now be even unnecessary.

Implemented in jet-main#733fc267c72e (branch server_split)
Comment 2 Martin Fousek 2014-05-28 11:32:45 UTC
Thanks Svato for the code change, looks well, just one note ...

Shouldn't be there (2.26):
docToSave = target.getLookup().lookup(CloneableEditorSupport.class);
instead of:
Comment 3 Svata Dedic 2014-05-28 11:48:55 UTC
(In reply to Martin Fousek from comment #2)
> Thanks Svato for the code change, looks well, just one note ...
Very, very politely said :) Thanks for the review -- hopefully will be fixed in jet-main#9b968f378c77
Comment 4 Quality Engineering 2014-10-18 05:12:34 UTC
Integrated into 'main-silver', will be available in build *201410180001* on (upload may still be in progress)

User: Svata Dedic <>
Log: #244744: openide.text dependency removed from java.source.base, clients updated