Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix naming of mrpc functions #1290

Closed
chu11 opened this issue Nov 16, 2017 · 0 comments
Closed

fix naming of mrpc functions #1290

chu11 opened this issue Nov 16, 2017 · 0 comments
Assignees
@chu11

Comments

@chu11
Copy link
Member

chu11 commented Nov 16, 2017

#1104 renamed a number jansson pack & unpack functions. It seems a few functions were missed in the mrpc library. flux_mrpcf() should be flux_mrpc_pack() and flux_mrpc_getf() should be flux_mrpc_unpack()
chu11 added a commit to chu11/flux-core that referenced this issue Feb 14, 2018
@chu11
libflux/rpc: Rename flux_mrpcf() and flux_mrpc_getf()
a2445cd 
For consistency, rename flux_mrpcf() to flux_mrpc_pack()
and rename flux_mrpc_getf() to flux_mrpc_get_unpack() just
as the changes were done in these prior commits.

libflux/rpc: rename rpcf->rpc_pack

9a5c28b

and

libflux/rpc: rename rpc_getf->rpc_get_unpack

1a3626b

Fixes flux-framework#1290
@chu11 chu11 mentioned this issue Feb 14, 2018
Merged
@chu11 chu11 self-assigned this Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chu11 chu11

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@chu11