Opened 10 years ago

Closed 10 years ago

#834 closed defect (fixed)

mpi: Incorrect argument count for allgather-int and allgather-flonum procedures

Reported by: sjamaan Owned by: Ivan Raikov
Priority: minor Milestone:
Component: extensions Version: 4.7.x
Keywords: Cc:
Estimated difficulty:

Description

In Mario's scrutinizer run two problems were found in mpi, both related to wrong argument count: http://parenteses.org/mario/misc/specialize-report/install/mpi.html

I don't have openmpi installed, and I don't know much about mpi anyway, so I don't have a patch or a unit test for this. However, I have a hunch that the MPI:allgather-int and MPI:allgather-flonum should use make-allgather like MPI:allgather-bytevector is doing.

There are also two C compiler warnings, but I assume that's because Chicken doesn't allow array dimensionality specifications in its FFI, so those might be hard to "fix".

Change History (1)

comment:1 Changed 10 years ago by Ivan Raikov

Resolution: fixed
Status: newclosed

Fixed in release 1.12 of mpi, thanks for reporting.

Note: See TracTickets for help on using tickets.