Re: Statistiques

Bonjour,

Je suis d'accord avec Scott pour les noms de paramètres.

Le 10/10/14 19:52, Scott Kostyshak a écrit :
>> Voilà ce que cela donne :
[... couic couic ...]
>>          Buffer buffer = cur.buffer();
>>          buffer.updateStatistics(from, to);

Alors là tu as un problème: quand tu récupère le buffer comme tu en fais 
une copie. Alors si c'est un grand document...

La bonne forme est:
   Buffer & buffer = cur.buffer();

Cela garde juste une référence au buffer.

>>          switch (cmd.getArg(0)) {
>>              case "words":
>>                  cur.message(buffer.wordCount());
>>                  break;

Comme l'a dit Scott, tu ne peux pas utiliser switch ici. Essaie plutot des
if (cmd.getArg(0) == "words") {
	...
} else if (cmd.getArg(0) == "signs") {
...

>>              default:
>>                  cur.message(buffer.wordCount() + buffer.charCount(false) +
>> buffer.charCount(true));

Alors là tu additionne les valeurs renvoyées par Buffer::xxxCount. Je 
pense que ce n'est pas ce que tu veux. Il faut faire comme dans 
LFUN_SERVER_GET_XY et utiliser convert<string>.

> Il me semble que tu fais beaucoup de progès. Il m'a fallu au moins
> cinquante "itérations Jean-Marc" pour mon premier patch.

:) C'est un jeu plutôt amusant quand la personne a l'autre bout a la 
patience de suivre toutes mes récriminations.

JMarc