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
Minor changes #135
Minor changes #135
Conversation
arturobernalg
commented
Jun 16, 2021
•
edited
edited
- Replace with <>
- Simplify conditions and avoid extra checks.
- Use java array style
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good aside from the few comments I sprinkled throughout.
@@ -161,7 +161,7 @@ | |||
* @return 1 for success and 0 for failure. | |||
*/ | |||
public static native int EVP_CipherInit_ex(PointerByReference ctx, PointerByReference cipher, | |||
PointerByReference impl, byte key[], byte iv[], int enc); | |||
PointerByReference impl, byte[] key, byte[] iv, int enc); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No weird formatting please.
@@ -133,7 +133,7 @@ | |||
* @return 1 for success and 0 for failure. | |||
*/ | |||
public static native int EVP_CipherInit_ex(PointerByReference ctx, PointerByReference cipher, | |||
PointerByReference impl, byte key[], byte iv[], int enc); | |||
PointerByReference impl, byte[] key, byte[] iv, int enc); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No weird formatting please.
@@ -161,10 +161,6 @@ final protected int next(final int numBits) { | |||
public void close() { | |||
closeRdrandEngine(); | |||
OpenSslNativeJna.ENGINE_cleanup(); | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unless you've authored these comments or you know they are invalid, leave them alone please.
* Simplify conditions and avoid extra checks. * Use java array.
HI @garydgregory |