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.

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

floppy.cpp: Update class variables to the current encouraged naming convention. #11851

Merged
merged 1 commit into from Jan 20, 2024

Conversation

mgarlanger
Copy link
Contributor

Updated all member variables to be prefixed with m_, the existing code had a few with the prefix but most without. This made it more difficult to follow the code.

@mgarlanger
Copy link
Contributor Author

@galibert Can you review this? The file has a mix of class members with and without the m_ prefix. The floppy_image_device mainly has it without it, but has m_fs, m_fs_managers, m_make_sound, m_sound_out, m_flux_per_pixel_infos, m_flux_per_combined_track_infos, and m_flux_screen. Also most of the rest of the file is using the prefix.

@galibert galibert merged commit 5c0398c into mamedev:master Jan 20, 2024
5 checks passed
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.

None yet

2 participants