[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE : [Tsp-devel] Modif BB
From: |
eric.noulard |
Subject: |
RE : [Tsp-devel] Modif BB |
Date: |
Thu, 27 Jul 2006 07:55:29 +0100 |
Pas de régression chez moi, mais je n'ai pas
eu le temps de beaucoup secouer :))
Après une rapide revue du code, je dis que c'est du propre.
J'aime bien les bb_operations ca va nous faciliter le portage vers
d'autres façons de créer des SHM, notamment
y'en a un qui me tente bien qui est un "bête" fichier projetté
en mémoire (mmap), l'avantage de ça c'est qu'à peu près tous
les systèmes ont ce genre de fonctionnalités.
Au niveau du choix des noms y'a qd même un truc qui me chagrine
c'est "enum bb_type"
parce que du coup dans bb_core.h on a:
enum bb_type {
BB_SYSV,
BB_KERNEL,
};
mais aussi
typedef enum {E_BB_DISCOVER=0, /*!< Discover is used by @ref bb_subscribe when
discovering data type */
[...]
E_BB_UCHAR, /*!< An 8bit unsigned character */
E_BB_USER /*!< A user type of any size (should be
supplied) in @ref bb_publish */
} E_BB_TYPE_T;
donc
enum bb_type
et
E_BB_TYPE_T
le premier étant le 'type' de Blackboard (kernel, sysv, posix, mmapped file
etc...)
conditionnant les bb_ops utilisées
le deuxieme étant le 'type' des données dans le BB (E_BB_DOUBLE, E_BB_INT32
etc...)
même si je dois l'avoir E_BB_TYPE_T a été mal choisi (ça aurait du être
BB_DATATYPE_T,
voire bb_datatype_t) je préfèrerais éviter les homonimies troublantes.
Je suggèrerais donc bb_kind ou plutôt bb_kind_t (voir ci-après).
Sinon pour être homogène avec le reste des sources, je suggèrerais aussi la
definition
d'une enum de la façon suivante:
typedef enum bb_kind {
BB_SYSV,
BB_KERNEL,
} BB_KIND_T;
ou
typedef enum bb_kind {
BB_SYSV,
BB_KERNEL,
} bb_kind_t;
si ce n'est pas en contradiction avec les règles de codage 'linux kernel'
idem pour les structures:
typedef struct kernel_private {
[...]
} kernel_private_t;
Ces remarques n'enlèvent rien à la qualité et à l'intérêt que je porte
au BB Kernel :))
---
Eric Noulard - Consultant
BT Consulting & Systems Integration
tel: (+33) (0)534 604970
mob: (+33) (0)607 948100
web: www.bt.com/consulting
-------- Message d'origine--------
De: address@hidden de la part de Frederik Deweerdt
Date: sam. 22/07/2006 19:12
À: Transport Sample Protocol development list
Objet : [Tsp-devel] Modif BB
Salut liste,
J'ai commité dans le CVS le code permettant de séparer le code SysV du
code exposant l'API du black board. Si tout va bien, aucune nouvelle
fonctionnalité,
aucun changement ne devrait venir perturber le fonctionnement du BB.
A bientôt,
Frederik
_______________________________________________
Tsp-devel mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/tsp-devel
<<winmail.dat>>