[Bug Fix] Changed shmem_team_destroy to return void

[avincigu]

No description provided.

[markbrown314]

@avincigu in line src/teams_c.c4:147 the code is attempting to return an int even though the function does not return a value. Please remove this return.

teams_c.c: In function ‘shmem_team_destroy’:
teams_c.c:147:12: warning: ISO C forbids ‘return’ with expression, in function returning void [-Wpedantic]
147 |     return shmem_internal_team_destroy((shmem_internal_team_t *)team);
|            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
teams_c.c:139:1: note: declared here
139 | shmem_team_destroy(shmem_team_t team)
| ^~~~~~~~~~~~~~~~~~

[davidozog]

Looks good - let's just loop in @abrooks98 to see if Intel SHMEM could be affected? (Maybe you checked already?) At a glance, it actually looks like it's already good to go, but I could be wrong.

[avincigu]

@avincigu in line src/teams_c.c4:147 the code is attempting to return an int even though the function does not return a value. Please remove this return.

teams_c.c: In function ‘shmem_team_destroy’:
teams_c.c:147:12: warning: ISO C forbids ‘return’ with expression, in function returning void [-Wpedantic]
147 |     return shmem_internal_team_destroy((shmem_internal_team_t *)team);
|            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
teams_c.c:139:1: note: declared here
139 | shmem_team_destroy(shmem_team_t team)
| ^~~~~~~~~~~~~~~~~~

Made the change

[abrooks98]

LGTM

[markbrown314]

@avincigu looks good. Please merge (select squash and merge)