Skip to main content


Eclipse Community Forums
Forum Search:

Search      Help    Register    Login    Home
Home » Archived » IMP » Proposed API change to IEditorService
Proposed API change to IEditorService [message #578014] Fri, 06 August 2010 22:25 Go to next message
Adam Taylor is currently offline Adam TaylorFriend
Messages: 12
Registered: August 2010
Junior Member
*** Problem: ***

https://bugs.eclipse.org/bugs/show_bug.cgi?id=322035

IMP does not have enough "knowledge" to open editors on all types of IEditorInput


*** Diagnosis: ***

EditorInputUtils can handle only certain types of IEditorInput. Need a way for language provider to handle other types


*** Proposed IMP API change: ***

Add the following methods:

public interface IEditorService{
// ...

public IFile getFile(IEditorInput editorInput);

public String getNameExtension(IEditorInput editorInput);

public IPath getPath(IEditorInput editorInput);

// ...
}
Re: Proposed API change to IEditorService [message #578033 is a reply to message #578014] Fri, 06 August 2010 22:47 Go to previous message
Adam Taylor is currently offline Adam TaylorFriend
Messages: 12
Registered: August 2010
Junior Member
Upon further review, LanguageServiceManager can return multiple instances of IEditorService. We need to add these methods to a service which has only one instance per language, or create a new service.

*** NEW Proposed IMP API change: ***

Create a new service with the following methods:

public interface IEditorInputService{

public IFile getFile(IEditorInput editorInput);

public String getNameExtension(IEditorInput editorInput);

public IPath getPath(IEditorInput editorInput);

}

and create the corresponding extension point
Previous Topic:Proposed API change to IEditorService
Next Topic:Proposed API change to IMessageHandler
Goto Forum:
  


Current Time: Tue Apr 16 13:58:50 GMT 2024

Powered by FUDForum. Page generated in 0.38810 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top