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

Use suffix _mut #129

Closed
4 tasks done
bvssvni opened this issue Nov 3, 2014 · 0 comments · Fixed by #130
Closed
4 tasks done

Use suffix _mut #129

bvssvni opened this issue Nov 3, 2014 · 0 comments · Fixed by #130
Assignees
@bvssvni
Labels
easy

Comments

@bvssvni
Copy link
Contributor

bvssvni commented Nov 3, 2014

See #113

See http://aturon.github.io/style/naming/ownership.html

  • image::SubImage::mut_inner => image::SubImage::inner_mut
  • image::MutableRefImage::mut_pixels => image::MutableRefImage::pixels_mut
  • image::ImageBuf::mut_pixelbuf => image::ImageBuf::pixelbuf_mut
  • image::ImageBuf::mut_rawbuf => image::ImageBuf::rawbuf_mut
@bvssvni bvssvni added the draft Add if this issue includes suggested code, compares interfaces, preparses wording,etc label Nov 3, 2014
@bvssvni bvssvni changed the title Use postfix _mut Use suffix _mut Nov 3, 2014
@bvssvni bvssvni self-assigned this Nov 3, 2014
@bvssvni bvssvni added easy and removed draft Add if this issue includes suggested code, compares interfaces, preparses wording,etc labels Nov 3, 2014
bvssvni added a commit to bvssvni/image that referenced this issue Nov 3, 2014
@bvssvni
Renamed to suffix _mut
5ce5805 
Closes image-rs#129
This was referenced Nov 3, 2014
Merged
Closed
fintelia pushed a commit that referenced this issue Dec 20, 2024
@martinlindhe
Merge pull request #129 from martinlindhe/fix-windows-tests
51f0338 
fix render_images() test to work on Windows, fixes #125
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bvssvni bvssvni

Labels
easy
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Renamed to suffix _mut bvssvni/image
1 participant
@bvssvni