Sounds fine.

I was also thinking about having only one definition for `get_cdrom_drives()` and testing for the platform inside the function.
We would need a separate if-then for `AUTO_DETECT_DRIVES` then.

Having `get_cdrom_drives()` defined at 3 different places is a bit weird, I'd say.
That might be just me, though.