Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 335756 - AbstractDeclarativeScopeProvider.NAMED_DELEGATE not used consistently
Summary: AbstractDeclarativeScopeProvider.NAMED_DELEGATE not used consistently
Status: CLOSED FIXED
Alias: None
Product: TMF
Classification: Modeling
Component: Xtext (show other bugs)
Version: 1.0.1   Edit
Hardware: All All
: P3 trivial (vote)
Target Milestone: M5   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-29 05:31 EST by yirco CLA
Modified: 2017-09-19 18:04 EDT (History)
1 user (show)

See Also:
sven.efftinge: indigo+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description yirco CLA 2011-01-29 05:31:52 EST 
Hello,

class org.eclipse.xtext.scoping.impl.AbstractDeclarativeScopeProvider defines:

public final static String NAMED_DELEGATE = "org.eclipse.xtext.scoping.impl.AbstractDeclarativeScopeProvider.delegate";

But generated module e.g.
public abstract class AbstractMyDslRuntimeModule extends DefaultRuntimeModule {

uses a raw string

	// contributed by org.eclipse.xtext.generator.scoping.AbstractScopingFragment
	public void configureIScopeProviderDelegate(com.google.inject.Binder binder) {
		binder.bind(org.eclipse.xtext.scoping.IScopeProvider.class).annotatedWith(com.google.inject.name.Names.named("org.eclipse.xtext.scoping.impl.AbstractDeclarativeScopeProvider.delegate")).to(org.eclipse.xtext.scoping.impl.ImportedNamespaceAwareLocalScopeProvider.class);
	}
Comment 1 Sven Efftinge CLA 2011-01-29 13:39:44 EST 
Thanks. This has already been fixed in the current development stream.
Comment 2 yirco CLA 2011-01-30 02:04:35 EST 
(In reply to comment #1)
> Thanks. This has already been fixed in the current development stream.

OK, no problem.
Comment 3 Karsten Thoms CLA 2017-09-19 17:53:24 EDT 
Closing all bugs that were set to RESOLVED before Neon.0
Comment 4 Karsten Thoms CLA 2017-09-19 18:04:07 EDT 
Closing all bugs that were set to RESOLVED before Neon.0