On Apr 19, 2006, at 4:27 AM, Christian Siebert wrote:
> although I can't see a difference between your
>
> for (p = linebuf; *p; ++p) {
> if (*p == '#') {
> *p = '\0';
> - return;
> + continue;
> }
> }
>
> modification and my first attempt (simply delete the "return;" and
> without
> the "continue;"), it seems to be working (tested on 3 different
> clusters)
> and I don't know why (should compile to identical object code).
> Until now I
> did not discover the mistakes I made. Maybe it was just too late ...
>
> Thanks you for your help and please forgive me my disability.
I've spent years learning the LAM code base, so don't feel bad ;).
I've committed the fix above to our source code repository. If we do
a LAM 7.1.3 release, it will be part of that release. Thanks for the
bug report!
Brian
--
Brian Barrett
LAM/MPI developer and all around nice guy
Have a LAM/MPI day: http://www.lam-mpi.org/
|
