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.

Sign up for GitHub

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

Jump to bottom

Consider changing RadarResults to contain instead of extend ArrayList<ObjectStatus> #124

Open
wortzmanb opened this issue May 16, 2016 · 1 comment
Open

Consider changing RadarResults to contain instead of extend ArrayList<ObjectStatus> #124

wortzmanb opened this issue May 16, 2016 · 1 comment
Assignees
@wortzmanb
Labels
client FUTURE idea question

Comments

@wortzmanb
Copy link

Extending is a little weird here. Is it better to be a standalone class with an ArrayList field, then write wrappers for the ArrayList methods we care about?
@wortzmanb wortzmanb self-assigned this May 16, 2016
@hawkerm
Copy link
Member

hawkerm commented May 17, 2016

I mean, it is nice that they can just iterated over the 'radar' without having to do much else, but I know that's not the main issue in their usage of it.

@hawkerm hawkerm added the FUTURE label May 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wortzmanb wortzmanb

Labels
client FUTURE idea question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wortzmanb @hawkerm