Re: [Tsp-devel] TSP_STATUS, TRUE, FALSE etc...

[Erk]

2006/4/16, Erk <address@hidden>:
> Salut les amis,
>
> Avant de le faire côté consumer j'aimerais
> que ceux qui auraient des modifs non-commités
> côté consumer les remontent en conf
> afin de leur éviter un merge.

Je viens de commiter les modifs de gestion
des TSP_STATUS_xxx côté consumer.

Les fonctions de l'API TSP_consumer_XXX
qui renvoient un status (int32_t)  renvoient
donc:
   TSP_STATUS_OK en cas de succès
  TSP_STATUS_ERROR_xxx en cas d'erreur.

Merci de ne pas supposer une valeur quelconque pour OK
et de bien tester
if (TSP_STATUS_OK!=TSP_consumer_xxx) {
error case ...
}

Il y a ... hum... quelques fichiers modifiés.
A vos update!!!

>
> Je pense plus particulièrement à Fred D (tspfs)
> et Euskadi (gdisp+/targa).
>

Dans l'objectif ne ne pas laisser dans le CVS des choses
trop fausses j'ai mis à jour TOUS les consumers.
Donc pour ceux qui ont des modifs en cours,
il y aura des merges, soyez vigilant.

Eric
>
> Le 14/04/06, address@hidden<address@hidden> a écrit :
> > Cher amis TSPiens,
> >
> > J'aimerais rationaliser un peu les codes de retours
> > des diffférentes API TSP (intérieur du CORE et également API publique).
> >
> > Certaines fonctions renvoient TRUE/FALSE
> > d'autres 0 et  != 0
> > d'autres des TSP_STATUS_xxx
> > d'autres encore TSP_CONSUMER_STATUS_xxx
> >
> > J'aimerais unifier tout ça en ne renvoyant uniquement
> > des TSP_STATUS_xxx afin de pouvoir vérifer la valeur
> > de retour du genre
> >
> > if (TSP_STATUS_OK!=TSP_consumer_request_open) {
> >
> > ]
> >
> > Ces status étant définis par un enum dans tsp_rpc.x
> > (et plus tard [ bientôt :))] dans tsp.idl) on peut
> > dispoer des valeurs "symboliques" partout (ruby, perl, python, C, java ...)
> > de façon consistente.
> >
> > Les seules fonctions qui ne renverraient pas de "TSP_STATUS_xxx"
> > seraien celles renvoyant des pointeurs.
> > Auquel cas pour ces dernière on poeut tester NULL ou !=NULL
> >
> > Qu'en dîtes-vous?
> >
> > ---
> > Eric Noulard - Software Architect
> > BT Consulting & Systems Integration
> > tel: (+33) (0)534 604970
> > mob: (+33) (0)607 948100
> > web: www.bt.com/consulting
> >
> >
> >
> > _______________________________________________
> > Tsp-devel mailing list
> > address@hidden
> > http://lists.nongnu.org/mailman/listinfo/tsp-devel
> >
>
>
> --
> Erk
>


--
Erk