Skip to content

Conversation

@jsquyres
Copy link
Member

@jsquyres jsquyres commented Nov 25, 2021

Static analysis found a few typos in 7b177ce: "&&" should have
been "&".

Fixes CIDs 1494439 and 1494440.

Signed-off-by: Jeff Squyres [email protected]

Follow-on to #9695, and should be included in cherry-picks to any other relevant branches.

Static analysis found a few typos in 7b177ce: "&&" should have
been "&".

Fixes CIDs 1494439 and 1494440.

Signed-off-by: Jeff Squyres <[email protected]>
Copy link
Contributor

@wzamazon wzamazon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@jsquyres jsquyres merged commit ed12420 into open-mpi:master Nov 25, 2021
@jsquyres jsquyres deleted the pr/fix-cids branch November 25, 2021 18:09
@jsquyres
Copy link
Member Author

@wzamazon I looked and it doesn't appear that you've PR'ed any of these OSC-fixes over to v5.0.x yet, right? If that's correct, can you (or whoever) be sure to include this commit when they get PR'ed to v5.0.x? Thanks!

@wzamazon
Copy link
Contributor

Sure, I will include this one when back port to v5.0.x.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants