Skip to content
This repository has been archived by the owner on Oct 10, 2024. It is now read-only.

Rely on inheritance for building referenced classes. #130

Merged
merged 1 commit into from
Dec 10, 2014

Conversation

JakeWharton
Copy link
Collaborator

This is better than forcing all subclasses to update when new types are referenced in super classes.

@JakeWharton
Copy link
Collaborator Author

I'll get to tests eventually... #131

@AlecKazakova
Copy link
Member

LGTM

@SuppressWarnings("unchecked")
Iterable<? extends HasClassReferences> concat =
Iterables.concat(super.referencedClasses(), implementedTypes, methodWriters,
nestedTypeWriters, fieldWriters.values(), explicitImports, ImmutableSet.of(name));
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think nested can be dropped here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Also name

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Keeping nested until I figure out it's behavior more. I think #77 might be related.

This is better than forcing all subclasses to update when new types are referenced in super classes.
JakeWharton added a commit that referenced this pull request Dec 10, 2014
Rely on inheritance for building referenced classes.
@JakeWharton JakeWharton merged commit aa28d0e into master Dec 10, 2014
@JakeWharton JakeWharton deleted the jw/super-referenced-classes branch December 10, 2014 00:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants