Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing CSharpCodeGeneratorHelper to be a service which is passed in. #1646

Closed
wants to merge 1 commit into from

Conversation

lajones
Copy link
Contributor

@lajones lajones commented Feb 18, 2015

See issue #830. Updating CSharpCodeGeneratorHelper to no longer be a singleton and instead be a service which is passed in.

@ghost ghost added the cla-not-required label Feb 18, 2015
if (CSharpCodeGeneratorHelper == null)
{
throw new ArgumentException(typeof(ReverseEngineeringGenerator).Name + " cannot find a service of type " + typeof(CSharpCodeGeneratorHelper).Name);
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an extension method for doing this--GetRequiredService<T>().

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will do. Thanks.

@bricelam
Copy link
Contributor

:shipit:

@lajones
Copy link
Contributor Author

lajones commented Feb 20, 2015

Updated and checked in - commit d5f5629.

@lajones lajones closed this Feb 20, 2015
@lajones lajones deleted the lajones-150129_ReverseEngineering_12 branch February 20, 2015 19:23
@bricelam bricelam mentioned this pull request Aug 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants