some comments

diff --git a/src/include/gnunet_reclaim_attribute_lib.h b/src/include/gnunet_reclaim_attribute_lib.h
index c23b393830ee8af6e75bdda2282e2bc8d233c554..1fa97dcc42420bd28ed223fbd553c43aa0d8f30d 100644 (file)
--- a/src/include/gnunet_reclaim_attribute_lib.h
+++ b/src/include/gnunet_reclaim_attribute_lib.h
@@ -348,6 +348,12 @@ GNUNET_RECLAIM_ATTRIBUTE_number_to_typename (uint32_t type);
 
 /**
    * Get required size for serialization buffer
+   * FIXME:
+   * 1. The naming convention is violated here.
+   * It should GNUNET_RECLAIM_ATTRIBUTE_<lowercase from here>.
+   * It might make sense to refactor attestations into a separate folder.
+   * 2. The struct should be called GNUNET_RECLAIM_ATTESTATION_Data or
+   * GNUNET_RECLAIM_ATTRIBUTE_Attestation depending on location in source.
    *
    * @param attr the attestation to serialize
    * @return the required buffer size

diff --git a/src/include/gnunet_reclaim_attribute_plugin.h b/src/include/gnunet_reclaim_attribute_plugin.h
index bdd815dcf260f5d9f916d1bf4d7a08659efaff5b..e61cca5b22f420799669e521eea019732e34add2 100644 (file)
--- a/src/include/gnunet_reclaim_attribute_plugin.h
+++ b/src/include/gnunet_reclaim_attribute_plugin.h
@@ -136,8 +136,14 @@ struct GNUNET_RECLAIM_ATTRIBUTE_PluginFunctions
   GNUNET_RECLAIM_ATTRIBUTE_NumberToTypenameFunction number_to_typename;
 
   /**
-* Attestation Conversion to string.
-*/
+   * FIXME: It is odd that attestation functions are withing the attribute
+   * plugin. An attribute type may be backed by an attestation, but not
+   * necessarily.
+   * Maybe it would make more sense to refactor this into an attestation
+   * plugin?
+   *
+   * Attestation Conversion to string.
+   */
   GNUNET_RECLAIM_ATTRIBUTE_ValueToStringFunction value_to_string_attest;
 
   /**