Skip to content

spec: Rename handle to volume_handle in VolumeInfo. #114

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.

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

Conversation

gpaul
Copy link

@gpaul gpaul commented Sep 21, 2017

Throughout the spec the name of any field of type VolumeHandle is
volume_handle except for the single use in VolumeInfo. Rename
VolumeInfo.handle to VolumeInfo.volume_handle.

Throughout the spec the name of any field of type VolumeHandle is
`volume_handle` except for the single use in `VolumeInfo`. Rename
`VolumeInfo.handle` to `VolumeInfo.volume_handle`.
@gpaul
Copy link
Author

gpaul commented Sep 21, 2017

I'll rebase once #113 lands.

@jieyu
Copy link
Member

jieyu commented Oct 3, 2017

Discussed this in the WG meeting. The reason we use handle in VolumeInfo rather than volume_handle is because the context is clear. It reads more nicely:

VolumeInfo volume;
volume.handle = ...;

Comparing to

VolumeInfo volume;
volume.volume_handle = ...;

Closing this.

@jieyu jieyu closed this Oct 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants