Re: Statistiques

Le 13/10/2014 23:01, Stéphane Mourey a écrit :
> Le 13/10/2014 22:49, Jean-Marc Lasgouttes a écrit :
>> Je vois plein de changements d'espaces.
> Mon éditeur de code est configuré pour supprimer les espaces
> surnuméraires... faudra peut-être que je change ça.
>> J'ai fait la même chose dans master, est-ce que tu pourrais rebaser
>> ton patch sur le nouveau master? Cela fera un patch plus clair.
> Le voilà donc.
>
> Bonne nuit.

Bonjour,

J'ai appliqué le patch et cela marche:

fantomas: src/client/lyxclient -c LYXCMD:server-get-statistics
lyxclient: Connected to /tmp/lyx_tmpdir.CBZRWVx18000/lyxsocket
INFO:server-get-statistics:3518 17028 20401

Le binaire lyxclient devrait être amélioré pour être plus utilisable, 
mais cela marche, sans aucune configuration de mon coté.

Bon en ce qui concerne le patch, quelques remarques futiles:


 > Subject: [PATCH] Add LFUN_SERVER_GET_STATISTICS command, which make 
possible
 > to get the LyX statistics on .lyxpipe.out

Mets commit en deux parties
1/ une explication d'une ligne: "Add LFUN_SERVER_GET_STATISTICS command"
2/ Après une ligne vide, l'explication.

Il ne s'agit pas ici d'obtenir de l'info dans .lyxpipe.out, mais par le 
serveur LyX. Selon la configuration ou la plateforme, cela peut passer 
par une pipe ou un socket (chez moi par défaut c'est un socket).

Je vois que la documentation n'est pas terrible à ce sujet, mais bon...

Bref, une bonne explication serait "This function can be used with the 
LyX server to obtain the word, characters or characters+spaces count in 
the current document or selection".


+  * \li Syntax: server-get-statistics <words|chars|chars-space>

Comme l'argument est optionnel, il faut le metter entre crochets et 
majuscules:
   * \li Syntax: server-get-statistics [<TYPE>]
   * Params: <TYPE>: words|chars|chars-space
      words: count words
etc. etc.

Reagarde d'autres exemples. Dans la description, dire aussi que cela 
peut compter la selection courante.


+			const string arg0 = cmd.getArg(0);

Nous préférons la forme "string const arg0".


+				cur.message(convert<docstring>(cur.buffer()->wordCount()) + " " + 
convert<docstring>(cur.buffer()->charCount(false)) + " " + 
convert<docstring>(cur.buffer()->charCount(true)));


Nous essayons d'avoir des ligne de moins de 80 caractères. Cette devrait 
être coupée en morceaux.


Moyennant ces petits changements, je pense que ce patch est prêt.

JMarc