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

Added missing iOS devices (fixes #3426) #3497

Merged
merged 1 commit into from Oct 22, 2015
Merged

Conversation

Zomby2D
Copy link
Contributor

@Zomby2D Zomby2D commented Oct 19, 2015

I have added the missing devices to IOSDevice.java, following issue #3426

The change is untested since I do not have the required hardware to test it. It is, however, a trivial change.

Sources:
https://www.theiphonewiki.com/wiki/Models
http://www.everymac.com/systems/apple/index-apple-specs-applespec.html

@Zomby2D Zomby2D changed the title Added missing iOS devices (fixes #3246) Added missing iOS devices ( fixes #3246 ) Oct 19, 2015
@Zomby2D Zomby2D changed the title Added missing iOS devices ( fixes #3246 ) Added missing iOS devices (fixes #3246) Oct 19, 2015
@davebaol davebaol added the ios label Oct 19, 2015
@Zomby2D Zomby2D changed the title Added missing iOS devices (fixes #3246) Added missing iOS devices (fixes #3426) Oct 19, 2015
@mikemee
Copy link
Contributor

mikemee commented Oct 20, 2015

+1 for this asap.... both iOS 6S devices are dead in the water without this... thx for the PR!!

@mikemee
Copy link
Contributor

mikemee commented Oct 22, 2015

@MobiDevelop @xoppa Any chance of incorporating this soon please?

@xoppa
Copy link
Member

xoppa commented Oct 22, 2015

Can you explain why it is needed change the existing values as well? That would make this an api breaking change I'd guess.

@MobiDevelop
Copy link
Member

That was my concern as well, but I am not sure if these values are generally used by people or if they are mostly for use by the backend.

@mikemee
Copy link
Contributor

mikemee commented Oct 22, 2015

In case it helps, I'm happy to test a private build ( I've tried building everything and failed unfortunately :( ). Fwiw the libGDX code I care about just matches the string against the resolution value and doesn't reference the enum name at all. Certainly none of those enum names appear in my own code.

@mikemee
Copy link
Contributor

mikemee commented Oct 22, 2015

Also happy to do a new PR that leaves the names the same and just adds the new ones...

@xoppa
Copy link
Member

xoppa commented Oct 22, 2015

I guess the enum is intended for the user to identify the device (the values doesnt seem to be used anywhere else in the repo). Changing these public enum values might break existing code. That might be justified, but I can't judge on that.

If you send (and preferably test) a PR that only adds the new values and doesn't break any existing code, then we can pull it in more easily. If you like, you can then send an additional PR to change the names of the values (including some explanation as to why), so that can be reviewed separately.

@mikemee
Copy link
Contributor

mikemee commented Oct 22, 2015

Created new PR #3505 - apologies if there was a better way (let me know if so and I'll do that instead).

@Zomby2D
Copy link
Contributor Author

Zomby2D commented Oct 22, 2015

It's my understanding that these values are not used anywhere by the back end so I tried to sanitize them a bit, but it's true that someone could theoretically have used them in their app.

xoppa added a commit that referenced this pull request Oct 22, 2015
Added missing iOS devices (fixes #3426), closes #3505
@xoppa xoppa merged commit b53054d into libgdx:master Oct 22, 2015
@xoppa
Copy link
Member

xoppa commented Oct 22, 2015

Thanks!

@xoppa xoppa mentioned this pull request Oct 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants