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