Andre Poenitz <[EMAIL PROTECTED]> writes:

| > Andre', could you have a quick look at it? You are much more
| > knowledgeable in STL than I am.
| 
| The code looks correct, but it slurps in the whole file while handling an
| item at a time should suffice. On low memory machines this might be a
| problem...
| 
| This is basically a tradeoff between speed and memory comsumption in
| favour of "high end" machines -- maybe going together with a poor stream
| implementation in this particular case.
| 
| I seem to remember a discussion on this topic. I think I suggested
| passing the two iterators to the summing function, but Lars had clear
| evidence that the version using the temporary string was faster.
| 
| I feel 45 s is clearly inacceptable so _I_ would not mind to replace the
| lyxsum with a "good ol' C implementation" which should be reasonably fast
| in all cases. It would not do much harm to our "new style C++ code" since
| it is restricted to a single file and it would probably remove what seems
| to have been identified as a bottle neck. 
| 
| But it's Lars' decision, isn't it?

Instead of a C version I'd rather have a C++ verison that does it in
the "obvious" way with out a temporary.

        Lgb

Reply via email to