Re: [Tsp-devel] Re: TSP Design and Programming Guide 1.0

[Erk]

2006/5/25, Frederik Deweerdt <address@hidden>:
> Salut Eric,
> Je vais tâcher d'intégrer les README de tspfs dans le doc, mais c'est
> assez bas dans la todo list :).

Très honnêtement maintenir la doc de tous les consumers dans
un seul doc me parait TRES TRES pénible.
On pourrait tout au plus maintenir une liste de consumer "à jour"
avec 2 exemples (genre AsciiWriter et Targa) et pour les autres
il suffit d'améliorer la doc Doxygen, donc... je serais même
d'avis de ne pas intégrer de doc "un peu complète" des consumers/providers
dans le TSP Prog Guide.

> Autrement, j'ai ré(d)oxygéné le code de tspfs.

Pourquoi tu dis re- ? J'avais dégommé ça lors d'un merge/commit?
Si c'est le cas je m'excuse.

> Ca devrait générer un petit tutorial

C'est désormais le cas j'ai vérifié.
D'ailleurs je t'ai commité un mini-patch vu que ton block
de commentaire doxygen commençait
    par /*   (1 étoile)
et pas /** (2 étoiles)
du coup doyxgen ne parsait pas tes commentaires et ta
doc n'apparraisait pas dans la doc générée,
tu peux vérifier maintenant (aprsè update)
en faisant:

cd src/doxy
make

puis ouvrir:

file://$DEVBASE/dist/doc/api/html/group__TSP__Consumers.html

> en attendant que je commente un peu plus le reste du code.

Pas de problème. Je pense que pour 80% des utilisateurs
c'est la doc d'utilisation qui est nécessaire.

> Autrement j'ai fait des modifs sur tspfs, en supprimant notamment
> l'inutile data_collector qui récoltait les valeurs des symboles en
> mode synchrone.

Va bene.

> Je pense qu'il faudrait intégrer la dernière version à TSP 0.8.0 si ce
> n'est pas trop tard.

Non il n'est pas trop tard vu que moi même je suis en retard :))
D'ailleurs j'attends encore 1 patch et je fais la version.

> Encore un petit truc, je regardais le code de tsp_data_receiver.c, et
> la modif suivante devrait faire taire un warning:
> Index: src/core/driver/tsp_data_receiver.c

Cool on va bientôt pouvoir activer -Werror globalement :))
J'ai intégré en ajoutant un STRACE_WARNING car à cet
endroit on ne devrait jamais devoir traiter
TSP_UNKNOWN et encore moins default.

> ===================================================================
> RCS file: /sources/tsp/tsp/src/core/driver/tsp_data_receiver.c,v
> retrieving revision 1.25
> diff -u -r1.25 tsp_data_receiver.c
> --- src/core/driver/tsp_data_receiver.c 5 May 2006 15:18:05 -0000       1.25
> +++ src/core/driver/tsp_data_receiver.c 25 May 2006 16:36:53 -0000
> @@ -312,7 +312,9 @@
>                                 break;
>
>                            case TSP_TYPE_RAW:
> -                               sample->uvalue.uint8_value= ((uint8_t
> *)groups[group_index].decode_buffer)[i];
> +                          case TSP_TYPE_UNKNOWN:
> +                          default:
> +                               sample->uvalue.raw_value= ((uint8_t
> *)groups[group_index].decode_buffer)[i];
>                                 break;


--
Erk