Core currently depends on Text API, which means almost any platform application will need to include the Text API even though they do not use it.
It is referenced from org.netbeans.beaninfo, in IndentEngine, IndentEngineBeanInfo and IndentEngineEditor (and the layer file, though I don't really understand
what NoIndentEngine.instance is for).
These could easily be moved either into the Text API module or possibly just deleted (the only two indentation engines registered in NB 6 which have any
custom properties are Java and JSP, and at least Java is handled differently in the Basic Options dialog in 6.0).
It should take very little work to break this dependency and make many platform apps smaller. Any reason not to do it for 6.0? At worst, we just move the
BeanInfo into o.n.m.text and register it in a static initializer for IndentationEngine.
Created attachment 43589 [details]
Diff for openide/text
Created attachment 43590 [details]
Diff for core
The attached patch almost works; I rewrote IndentEngineEditor to remove the dependency on ServiceTypeEditor (Lookups.forPath() is very handy). However,
for some reason the property sheet is resetting the property editor with the old value immediately after setAsText() is called with a new value. So that needs
to be diagnosed some more (I can't depend on ExPropertyEditor unless we want to create a bridge module just for this one property editor).
But basically it works.
reassigning back to tim to re-evaluate
Just leave openide.text alone and delete the customizers. IndentEngine is deprecated and not used for customizing
formatting any more, so there is no need for a prop editor / customizer. Vita can confirm I guess.
Also I am not sure what <friend>org.netbeans.modules.indentationbridge</friend> is all about.
BTW do not use the IDE's wacky diff tool to generate diff. Use 'hg diff --git'. And make sure that you use 'hg ren' when
renaming files (renames from the IDE should do it right); diff --git will show these files as renamed, not necessarily
Feel free to assign to me.
"IndentEngine is deprecated and not used ..." - correct
Reassigning to Jesse per his request
Will remove dep: core-main #0f94078d5899
Note that the IndentEngine editors were in fact already deleted. So the dep was just a relic.
Most platform applications will _still_ need to include the Text API because it is used by other modules, such as the
Loaders API. It would be possible to split this off with some further effort.
Integrated into 'main-golden', will be available in build *200902130336* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
User: Jesse Glick <firstname.lastname@example.org>
Log: #106423: removing unnecessary dep o.n.core -> openide.text.