Community
Participate
Working Groups
org.eclipse.rse.subsystems.terminals.core/Activator.java currently contains some code to force load the UI part. This is not a good idea since (a) the subsystem might be used in a headless non-UI envrionment eventually, and (b) the UI plugin should be loaded lazily. Consider bug 218304 which got rid of the force loading of adapters as a blueprint for how it should be done (i.e. when the UI plugin should be loaded lazily).
Note that I consider this API since clients might expect the UI plugin to be available at a given time, so the fact of loading it lazily is - to some extent - an API change.
Created attachment 98468 [details] patch that removes forced load of UI plug-in from core one. I, Anna Dushistova, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. I am authorized by my employer to make this contribution under the EPL.
Patch applied, 2 added and 16 removed lines. Anna, did you test everything, e.g. does the terminal work as expected for you? Or did you find any problems such as missing icons or NullPointerException in special cases?
> Anna, did you test everything, e.g. does the terminal work as expected for you? Yes, I tested it and it worked fine for me. > Or did you find any problems such as missing icons or NullPointerException in > special cases? No, I haven't seen any.