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

Update Aparapi package name #51

Merged
merged 2 commits into from Jul 26, 2017

Conversation

InonS
Copy link
Contributor

@InonS InonS commented Jul 11, 2017

Changed com.amd.aparapi to com.aparapi as per version 1.3.4 of the dependency (downloaded from maven central, and designated the recent release version on the Aparapi webpage).
Corrected some code analysis warnings: Malformed JavaDocs, more Java 8 usage, simplification of conditionals, unnecessary boxing, and an inefficient array copy.

Inon Sharony added 2 commits July 11, 2017 19:55
…8 usage, simplification of conditionals, unnecessary boxing, and an inefficient array copy.
@InonS
Copy link
Contributor Author

InonS commented Jul 20, 2017

Hi, @ivan-vasilev !

Could you please take a look at my PR and comment? I'm pretty sure the AParApi package name as it appears in your master branch is not up to date with the recent version of AParApi. Am I wrong?

@ivan-vasilev ivan-vasilev merged commit 656f27d into ivan-vasilev:master Jul 26, 2017
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.

None yet

2 participants