Delete a few old FIXME comments

diff --git a/Makefile.in b/Makefile.in
index c2ce4cd6cfbca2a59f89f412b2f89ce9c84a67c1..e6fc25b89cc0f0fe15f95e8872454c691318e3da 100644 (file)
--- a/Makefile.in
+++ b/Makefile.in
@@ -26,7 +26,6 @@ include config.make
 PRE_CPPFLAGS = -I.
 EXTRA_CFLAGS = $(CCPIC)
 
-# FIXME: Add configuration of LIBEXT?
 LIBTARGETS = @IF_STATIC@ libnettle.a @IF_HOGWEED@ libhogweed.a
 SHLIBTARGETS = @IF_SHARED@ $(LIBNETTLE_FORLINK) @IF_HOGWEED@ $(LIBHOGWEED_FORLINK)
 
@@ -67,11 +66,6 @@ check-fat:
            NETTLE_FAT_OVERRIDE=$$f $(MAKE) check ; \
          done
 
-# FIXME: Remove. These targets aren't supported, but they are expected by the
-# automake generated Makefiles in the lsh build.
-dvi installcheck uninstallcheck:
-       true
-
 all-here: $(TARGETS) $(DOCTARGETS)
 
 nettle_SOURCES = aes-decrypt-internal.c aes-decrypt.c aes-decrypt-table.c \

diff --git a/ecc-curve25519.c b/ecc-curve25519.c
index 5d221ae352edbdec38364b472b8212241ba33890..7182b2ff87e80df7db57980812f9b8d916698a25 100644 (file)
--- a/ecc-curve25519.c
+++ b/ecc-curve25519.c
@@ -37,7 +37,6 @@
 
 #include <assert.h>
 
-#include "ecc.h"
 #include "ecc-internal.h"
 
 #define USE_REDC 0

diff --git a/ecc-curve448.c b/ecc-curve448.c
index 04b64905dd6a3c23b23e990df362863df30144d4..bd87c5b8f416683c20887ba0440f07fd6610eeec 100644 (file)
--- a/ecc-curve448.c
+++ b/ecc-curve448.c
@@ -38,7 +38,6 @@
 
 #include <assert.h>
 
-#include "ecc.h"
 #include "ecc-internal.h"
 
 #define USE_REDC 0
@@ -252,8 +251,8 @@ const struct ecc_curve _nettle_curve448 =
     NULL,
     ecc_qp1h,
 
-    ecc_mod,         /* FIXME: Implement optimized mod function */
-    ecc_mod,         /* FIXME: Implement optimized reduce function */
+    ecc_mod,
+    ecc_mod,
     ecc_mod_inv,
     NULL,
   },

diff --git a/ecc-internal.h b/ecc-internal.h
index c20078466723f305f1923317da43723275bff6fb..245a6517c5823369e248270aef0928d2534a5207 100644 (file)
--- a/ecc-internal.h
+++ b/ecc-internal.h
@@ -189,9 +189,8 @@ struct ecc_curve
 {
   /* The prime p. */
   struct ecc_modulo p;
-  /* Group order. FIXME: Currently, many functions rely on q.size ==
-     p.size. This has to change for radix-51 implementation of
-     curve25519 mod p arithmetic. */
+  /* Group order. Currently, many functions rely on q.size ==
+     p.size. */
   struct ecc_modulo q;
 
   unsigned short use_redc;

diff --git a/ecc-secp192r1.c b/ecc-secp192r1.c
index 285278a5951bcef0146fde131c040c850262720c..abdad0c9dc30f264ea7b5ddd9fcba7cbad914362 100644 (file)
--- a/ecc-secp192r1.c
+++ b/ecc-secp192r1.c
@@ -39,9 +39,6 @@
 
 #include <assert.h>
 
-/* FIXME: Remove ecc.h include, once prototypes of more internal
-   functions are moved to ecc-internal.h */
-#include "ecc.h"
 #include "ecc-internal.h"
 
 #define USE_REDC 0

diff --git a/ecc-secp224r1.c b/ecc-secp224r1.c
index 98260412c030c0433d2c70f304ff69a4de87219a..91c337d4415e8b9cd7c4ba439000b3c0e392c3f7 100644 (file)
--- a/ecc-secp224r1.c
+++ b/ecc-secp224r1.c
@@ -37,7 +37,6 @@
 # include "config.h"
 #endif
 
-#include "ecc.h"
 #include "ecc-internal.h"
 
 #if HAVE_NATIVE_ecc_secp224r1_modp

diff --git a/ecc-secp256r1.c b/ecc-secp256r1.c
index 9bdd0077e382d8711b13f6f5ef77890dbac943a8..57ecdd5f2bfae6e982cb89774a4284023ef16b99 100644 (file)
--- a/ecc-secp256r1.c
+++ b/ecc-secp256r1.c
@@ -39,7 +39,6 @@
 
 #include <assert.h>
 
-#include "ecc.h"
 #include "ecc-internal.h"
 
 #if HAVE_NATIVE_ecc_secp256r1_redc

diff --git a/ecc-secp384r1.c b/ecc-secp384r1.c
index fef47036742b45f85f165ee04f1da53652e04908..2c00523bd0c0bbf9323172caf6503adecbd223e6 100644 (file)
--- a/ecc-secp384r1.c
+++ b/ecc-secp384r1.c
@@ -39,7 +39,6 @@
 
 #include <assert.h>
 
-#include "ecc.h"
 #include "ecc-internal.h"
 
 #define USE_REDC 0

diff --git a/ecc-secp521r1.c b/ecc-secp521r1.c
index 42b7e1560b6458c0a0b65bf639b4901383dd056e..7eda37323012a8b23175dbf029cdfd17d351e4cb 100644 (file)
--- a/ecc-secp521r1.c
+++ b/ecc-secp521r1.c
@@ -37,7 +37,6 @@
 # include "config.h"
 #endif
 
-#include "ecc.h"
 #include "ecc-internal.h"
 
 #define USE_REDC 0

diff --git a/ecdsa.h b/ecdsa.h
index 693aca8b0e6b125450ef95f791c521381ca9b056..259efc10553543d7b5eefbf492453b657279a2ce 100644 (file)
--- a/ecdsa.h
+++ b/ecdsa.h
@@ -53,7 +53,7 @@ extern "C" {
 /* High level ECDSA functions.
  *
  * A public key is represented as a struct ecc_point, and a private
- * key as a struct ecc_scalar. FIXME: Introduce some aliases? */
+ * key as a struct ecc_scalar. */
 void
 ecdsa_sign (const struct ecc_scalar *key,
            void *random_ctx, nettle_random_func *random,

diff --git a/gostdsa.h b/gostdsa.h
index d479201e32951fab30795144a8fab21e03c3c2d9..bcad1f95f996655fa0d9672665651452027ca1c2 100644 (file)
--- a/gostdsa.h
+++ b/gostdsa.h
@@ -56,7 +56,7 @@ extern "C" {
 /* High level GOST DSA functions.
  *
  * A public key is represented as a struct ecc_point, and a private
- * key as a struct ecc_scalar. FIXME: Introduce some aliases? */
+ * key as a struct ecc_scalar. */
 void
 gostdsa_sign (const struct ecc_scalar *key,
              void *random_ctx, nettle_random_func *random,

diff --git a/rsa-sign-tr.c b/rsa-sign-tr.c
index 9e137c7ac3095de1a86b49e6fdf65c1779b43e57..f4c30e60f0863fa892036608a00240229e4d133c 100644 (file)
--- a/rsa-sign-tr.c
+++ b/rsa-sign-tr.c
@@ -244,7 +244,6 @@ sec_equal(const mp_limb_t *a, const mp_limb_t *b, size_t limbs)
       z |= (a[i] ^ b[i]);
     }
 
-  /* FIXME: Might compile to a branch instruction on some platforms. */
   return z == 0;
 }
 

diff --git a/testsuite/testutils.c b/testsuite/testutils.c
index 64dbcd0d9b667d819e9ba35e52176558967a4679..626c13fcc6b3ed3310a9e0adbf53e08c78964692 100644 (file)
--- a/testsuite/testutils.c
+++ b/testsuite/testutils.c
@@ -1382,7 +1382,6 @@ test_rsa_key(struct rsa_public_key *pub,
   
   if (verbose)
     {
-      /* FIXME: Use gmp_printf */
       fprintf(stderr, "Public key: n=");
       mpz_out_str(stderr, 16, pub->n);
       fprintf(stderr, "\n    e=");