Re: [PATCH] Warn about MPI not supporting cross-context particles

[Jeffrey D. Oldham]

Richard Guenther wrote:
> Hi!
>
> This patch aborts on use of cross-context particles with MPI, which is not
> supported.
>
> Ok?
>
> Richard.
>
>
> 2004Jan18  Richard Guenther <address@hidden>
>
>         * src/Particles/PatchSwapLayout.cpp: abort on cross-context
>         particles with MPI.
>
> Index: src/Particles/PatchSwapLayout.cpp
> ===================================================================
> RCS file: /home/pooma/Repository/r2/src/Particles/PatchSwapLayout.cpp,v
> retrieving revision 1.15
> diff -u -u -r1.15 PatchSwapLayout.cpp
> --- src/Particles/PatchSwapLayout.cpp   8 Jun 2000 22:16:23 -0000       1.15
> +++ src/Particles/PatchSwapLayout.cpp   18 Jan 2004 13:38:28 -0000
> @@ -545,6 +545,8 @@
>      Pooma::particleSwapHandler()->send(toContext, tag, buf);
>    }
>
> +#elif POOMA_MPI
> +  PInsist(false, "Cross-context particles not supported for MPI");
>  #endif // POOMA_CHEETAH

Is the POOMA_CHEETAH comment correct?  I think it is probably correct 
since the first #if is probably for Cheetah, but is it misleading?  Is 
there a better comment?

>  }
>
> @@ -621,6 +623,8 @@
>    while (layout_m.patchInfo(lid).msgReceived() < remotePatches)
>      Pooma::poll();
>
> +#elif POOMA_MPI
> +  PInsist(false, "Cross-context particles not supported for MPI");
>  #endif // POOMA_CHEETAH

Likewise here.

>  }

Thanks for tightening the code.  OK to commit.

--
Jeffrey D. Oldham
address@hidden