Allow NULL arrays for empty datatypes.

[jsquyres]

When building an empty datatype (aka. size = 0) because the count of
included datatypes is 0, be less strict on what the arguments are
(allow NULL pointers).

(cherry picked from commit open-mpi/ompi@6e6ed62)

@bosilca Please review. Straightforward cherry-pick from master.

@hppritcha I just PR'ed this to v1.10.3, and it apparently solves a user issue (reported by @wkliao in open-mpi/ompi#1611). I don't have strong feelings as to whether this is in v2.0.0 or v2.0.1, other than the fact that it fixes a user-reported error. Opinions?

[mellanox-github]

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1583/ for details.

[hppritcha]

2.0.1

Am Montag, 2. Mai 2016 schrieb Jeff Squyres :

When building an empty datatype (aka. size = 0) because the count of
included datatypes is 0, be less strict on what the arguments are
(allow NULL pointers).

(cherry picked from commit open-mpi/ompi@6e6ed62
open-mpi/ompi@6e6ed62
)

@bosilca https://github.com/bosilca Please review. Straightforward
cherry-pick from master.

@hppritcha https://github.com/hppritcha I just PR'ed this to v1.10.3,
and it apparently solves a user issue (reported by @wkliao
https://github.com/wkliao in open-mpi/ompi#1611
open-mpi/ompi#1611). I don't have strong
feelings as to whether this is in v2.0.0 or v2.0.1, other than the fact

that it fixes a user-reported error. Opinions?

You can view, comment on, or merge this pull request online at:

#1122
Commit Summary

• Allow NULL arrays for empty datatypes.

File Changes

• M ompi/datatype/ompi_datatype_create_subarray.c
  https://github.com/open-mpi/ompi-release/pull/1122/files#diff-0 (4)
• M ompi/mpi/c/type_create_darray.c
  https://github.com/open-mpi/ompi-release/pull/1122/files#diff-1 (5)
• M ompi/mpi/c/type_create_hindexed.c
  https://github.com/open-mpi/ompi-release/pull/1122/files#diff-2 (6)
• M ompi/mpi/c/type_create_subarray.c
  https://github.com/open-mpi/ompi-release/pull/1122/files#diff-3 (4)
• M ompi/mpi/c/type_hindexed.c
  https://github.com/open-mpi/ompi-release/pull/1122/files#diff-4 (6)
• M ompi/mpi/c/type_indexed.c
  https://github.com/open-mpi/ompi-release/pull/1122/files#diff-5 (6)

Patch Links:

• https://github.com/open-mpi/ompi-release/pull/1122.patch
• https://github.com/open-mpi/ompi-release/pull/1122.diff

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#1122

[bosilca]

👍

[jsquyres]

bot:retest

[jsquyres]

bot:retest