commit c136a16600cd4f72d7def1af7b4aa7592310c898 parent 4eda8d33675ba1e004792635b33342c04c5cbe32 Author: Schanzenbach, Martin <mschanzenbach@posteo.de> Date: Thu, 17 Oct 2019 17:07:05 +0200 some comments Diffstat:
| M | src/include/gnunet_reclaim_attribute_lib.h | | | 6 | ++++++ |
| M | 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 @@ -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 @@ -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; /**