Mentions légales du service

Skip to content

nmad sendrecv: add API to trigger events RECV_COMPLETED and FINALIZED separately

Philippe SWARTVAGHER requested to merge split-unpack-finalized into master

Je me suis dit que j'allais tenter de le faire moi-même. Mais puisque c'est assez critique (ça touche à nm_core ! 😱 ), je veux bien une revue de code avant de mettre ça dans le master !

Quelques remarques:

  • je ne l'ai fait que pour la réception (pour mes besoins, et je pense plus généralement, ça n'a pas vraiment de sens pour l'envoi)
  • il faudra aussi faire les changements nécessaires dans les interfaces rpc et mcast (d'où le WIP), mais on commence déjà par valider cette première étape
  • est-ce vraiment nécessaire de prendre le lock dans nm_core_unpack_finalize() ? Il n'est pas content sinon, donc je suppose que oui...
  • est que le paramètre split de la fonction nm_core_unpack_set_split_unpack_finalized_status() est vraiment nécessaire ? Le débat est ouvert...
  • je te laisse vérifier qu'il n'y a pas d'endroits critiques où on suppose que NM_STATUS_UNPACK_COMPLETED et NM_STATUS_FINALIZED sont positionnés en même temps.
Edited by Philippe SWARTVAGHER

Merge request reports