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

Répondre à