[gnunet] 10/25: some comments

[gnunet]

This is an automated email from the git hooks/post-receive script.

martin-schanzenbach pushed a commit to branch master
in repository gnunet.

commit c136a16600cd4f72d7def1af7b4aa7592310c898
Author: Schanzenbach, Martin <address@hidden>
AuthorDate: Thu Oct 17 17:07:05 2019 +0200

    some comments
---
 src/include/gnunet_reclaim_attribute_lib.h    |  6 ++++++
 src/include/gnunet_reclaim_attribute_plugin.h | 10 ++++++++--
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/src/include/gnunet_reclaim_attribute_lib.h 
b/src/include/gnunet_reclaim_attribute_lib.h
index 34429da51..8476e77fc 100644
--- 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 bdd815dcf..e61cca5b2 100644
--- 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;
 
   /**

-- 
To stop receiving notification emails like this one, please contact
address@hidden.