Re: [OMPI devel] [patch] Bugs in mpi-f90-interfaces.h and its bridge implementation

2012-04-06 Thread Jeffrey Squyres
On Apr 6, 2012, at 7:09 AM, Kawashima wrote: > I've checked your code in bitbucket. Two types of error are found. > I've attached the patch. > > First one (ignore-tkr) seems to be an error by manual patching. > Second one (tkr) seems that patch command could not apply my fixes > because neighbori

Re: [OMPI devel] [patch] Bugs in mpi-f90-interfaces.h and its bridge implementation

2012-04-06 Thread Kawashima
Hi Jeff, I've checked your code in bitbucket. Two types of error are found. I've attached the patch. First one (ignore-tkr) seems to be an error by manual patching. Second one (tkr) seems that patch command could not apply my fixes because neighboring lines were modified in your code. Regards,

Re: [OMPI devel] [patch] Bugs in mpi-f90-interfaces.h and its bridge implementation

2012-04-04 Thread Kawashima
Hi Jeff, Jeffrey Squyres wrote: > > On Apr 3, 2012, at 10:56 PM, Kawashima wrote: > > > I and my coworkers checked mpi-f90-interfaces.h against MPI 2.2 standard > > and found many bugs in it. Attached patches fix them for trunk. > > Though some of them are trivial, others are not so trivial. > >

Re: [OMPI devel] [patch] Bugs in mpi-f90-interfaces.h and its bridge implementation

2012-04-04 Thread Jeffrey Squyres
On Apr 3, 2012, at 10:56 PM, Kawashima wrote: > I and my coworkers checked mpi-f90-interfaces.h against MPI 2.2 standard > and found many bugs in it. Attached patches fix them for trunk. > Though some of them are trivial, others are not so trivial. > So I'll explain them below. Excellent -- many

[OMPI devel] [patch] Bugs in mpi-f90-interfaces.h and its bridge implementation

2012-04-03 Thread Kawashima
Hi Open MPI developers, I and my coworkers checked mpi-f90-interfaces.h against MPI 2.2 standard and found many bugs in it. Attached patches fix them for trunk. Though some of them are trivial, others are not so trivial. So I'll explain them below. 1. incorrect parameter types Two trivial para