tree 0fca55b591682a51994609513a6f10671177f578
parent 1eddb4be29e49008524397bfdcdca2949e90f97d
author David Benjamin <davidben@google.com> 1514230533 -0500
committer Adam Langley <agl@google.com> 1515440917 +0000

Fold away ec_point_set_Jprojective_coordinates_GFp.

p224-64.c can just write straight into the EC_POINT, as the other files
do, which saves the mess around BN_CTX. It's also more correct.
ec_point_set_Jprojective_coordinates_GFp abstracts out field_encode, but
then we would want to abstract out field_decode too when reading.

That then allows us to inline ec_point_set_Jprojective_coordinates_GFp
into ec_GFp_simple_point_set_affine_coordinates and get rid of an
unnecessary tower of helper functions. Also we can use the precomputed
value of one rather than recompute it each time.

Change-Id: I8282dc66a4a437f5a3b6a1a59cc39be4cb71ccf9
Reviewed-on: https://boringssl-review.googlesource.com/24687
Reviewed-by: Adam Langley <agl@google.com>