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
uknc.cpp: hook up cassette and floppy #12254
Conversation
src/devices/bus/qbus/uknc_kmd.h
Outdated
protected: | ||
// device-level overrides | ||
virtual void device_start() override; | ||
virtual void device_add_mconfig(machine_config &config) override; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don’t copy/paste the “device-level overrides” comment – it doesn’t even make sense. Use “device_t implementation” or something if you want a comment for the overridden device_t member functions.
src/devices/bus/qbus/uknc_kmd.h
Outdated
private: | ||
static void floppy_formats(format_registration &fr); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is private and doesn’t need to access other class members, it doesn’t need to be a class member at all. You can throw it in an anonymous namespace and keep it out of the header.
src/devices/bus/qbus/uknc_kmd.h
Outdated
#include "machine/1801vp128.h" | ||
|
||
#include "qbus.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Local headers should come first. It helps avoid headers that don’t #include
everything they need.
src/devices/bus/qbus/uknc_kmd.cpp
Outdated
#include "emu.h" | ||
|
||
#include "machine/pdp11.h" | ||
#include "formats/bk0010_dsk.h" | ||
|
||
#include "uknc_kmd.h" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The translation unit’s header should be the first thing after the prefix header.
No description provided.