Re: [Maitretarot-devel-fr] *.h

[philippe brochard]

"Yves Mettier" <address@hidden> writes:

> > Une petite suggestion : j'ai un peu de mal à voir la séparation logique
> > de maitretarot.h et libmt_client.h. libmt_client.h inclue maitretarot.h
> > (double include de glib.h d'ailleurs). Or les 2 sont nécessaires au
> > client et au serveur.
> 
> Moi aussi j'ai du mal a voir la separation logique entre ces deux fichiers
> d'en-tete.
> Par contre, non, libmt_client.h ne sert pas au serveur. C'est peut-etre
> cela, la separation logique. Philippe, est-ce juste cela? Philippe
> toujours: est-ce que ca a vraiment un interet de separer cela en deux
> fichiers? Un seul fichier, avec la separation en tant qu'un commentaire ne
> suffirait pas?
> 
La separation, c'est juste que le libmt_client.h rassemble tout
ce qui est commun aux clients (c'est pour ça que c'est dans la lib).
Le serveur ne se sert pas du tout de ce qu'il y a dans ce fichier.

Le seul interet que je vois a separer ces deux fichiers, c'est que
ça permet de maintenir le code suffisament separe au niveau de la
lib => code serveur/entete serveur et code client/entete client.

> >
> > Or je ne pense pas qu'il soit judicieux d'embéter les clients avec des
> > entètes utiles uniquement au serveur, non ? D'ailleurs, pourquoi
> > partager un *.h qui ne sert qu'au serveur ?
> 
> maitretarot.h sert au client comme au serveur. Donc la, non, t'as tout faux.
> C'est juste libmt_client.h qui ne sert qu'au client. Pour les questions,
> cf ci-dessus :)
> 
Oui, par contre maitretarot.h est indispensable au client s'il se sert
de la lib. D'ailleur, il suffit d'inclure libmt_client.h sans inclure
maitretarot.h puisque celui-ci est inclu pas defaut dans libmt_client.h.
Cela fais une autre separation : le client qui se sert de la lib inclu
juste libmt_client.h qui correspond a la partie client de la lib maitretarot.


[...]



Philippe

-- 
(    )
 ~oo~         Philippe Brochard    <address@hidden>
  .. Gnu!                           http://hocwp.free.fr
  / =\   \=
 -   -    -   -=-= http://www.fsf.org/home.fr.html =-=-