signalapp / curve25519-java

Pure Java and JNI backed Curve25519 implementation.
GNU General Public License v3.0
233 stars 95 forks source link

Renamed variables (fix issue #6) #8

Closed sebastianv89 closed 8 years ago

sebastianv89 commented 9 years ago

The result from ge_scalarmult_base is in projective coordinates, so the variable names should reflect this: I replaced one with ed_z. Variable ed_y was not used, so I removed it. (see also this answer and the curves mailing list)

moxie0 commented 8 years ago

Thanks but this isn't a functional change, so we're going to leave it as-is for now.