Skip to content

Commit 7194446

Browse files
committed
Simplify precompute_ecmult_print_*
1 parent dddb682 commit 7194446

File tree

1 file changed

+7
-11
lines changed

1 file changed

+7
-11
lines changed

src/precompute.c

+7-11
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
#include "ecmult_gen_prec_impl.h"
2424
#include "ecmult_prec_impl.h"
2525

26-
static void precompute_ecmult_print_table(FILE *fp, const char *name, int window_g, const secp256k1_ge_storage* table, int with_conditionals) {
26+
static void precompute_ecmult_print_table(FILE *fp, const char *name, int window_g, const secp256k1_ge_storage* table) {
2727
int j;
2828
int i;
2929

@@ -35,29 +35,25 @@ static void precompute_ecmult_print_table(FILE *fp, const char *name, int window
3535

3636
j = 1;
3737
for(i = 3; i <= window_g; ++i) {
38-
if (with_conditionals) {
39-
fprintf(fp, "#if ECMULT_TABLE_SIZE(WINDOW_G) > %ld\n", ECMULT_TABLE_SIZE(i-1));
40-
}
38+
fprintf(fp, "#if ECMULT_TABLE_SIZE(WINDOW_G) > %ld\n", ECMULT_TABLE_SIZE(i-1));
4139
for(;j < ECMULT_TABLE_SIZE(i); ++j) {
4240
fprintf(fp, ",S(%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32
4341
",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32",%"PRIx32")\n",
4442
SECP256K1_GE_STORAGE_CONST_GET(table[j]));
4543
}
46-
if (with_conditionals) {
47-
fprintf(fp, "#endif\n");
48-
}
44+
fprintf(fp, "#endif\n");
4945
}
5046
fprintf(fp, "};\n");
5147
}
5248

53-
static void precompute_ecmult_print_two_tables(FILE *fp, int window_g, const secp256k1_ge *g, int with_conditionals) {
49+
static void precompute_ecmult_print_two_tables(FILE *fp, int window_g, const secp256k1_ge *g) {
5450
secp256k1_ge_storage* table = malloc(ECMULT_TABLE_SIZE(window_g) * sizeof(secp256k1_ge_storage));
5551
secp256k1_ge_storage* table_128 = malloc(ECMULT_TABLE_SIZE(window_g) * sizeof(secp256k1_ge_storage));
5652

5753
secp256k1_ecmult_create_prec_two_tables(table, table_128, window_g, g);
5854

59-
precompute_ecmult_print_table(fp, "secp256k1_pre_g", window_g, table, with_conditionals);
60-
precompute_ecmult_print_table(fp, "secp256k1_pre_g_128", window_g, table_128, with_conditionals);
55+
precompute_ecmult_print_table(fp, "secp256k1_pre_g", window_g, table);
56+
precompute_ecmult_print_table(fp, "secp256k1_pre_g_128", window_g, table_128);
6157

6258
free(table);
6359
free(table_128);
@@ -104,7 +100,7 @@ static int precompute_ecmult(void) {
104100
fprintf(fp, "#else /* !defined(EXHAUSTIVE_TEST_ORDER) */\n");
105101
fprintf(fp, "#define WINDOW_G ECMULT_WINDOW_SIZE\n");
106102

107-
precompute_ecmult_print_two_tables(fp, ECMULT_WINDOW_SIZE, &g, 1);
103+
precompute_ecmult_print_two_tables(fp, ECMULT_WINDOW_SIZE, &g);
108104

109105
fprintf(fp, "#endif /* EXHAUSTIVE_TEST_ORDER */\n");
110106
fprintf(fp, "#undef S\n");

0 commit comments

Comments
 (0)