While this is useful, it's very sloppy. I feel this would be much better received if it were cleaner. Functions should be something like_this instead ofThis. Words should be fully spelled out so it's readily apparent what it means and does. Just a suggestion.

Dear @Ming-Shu,

I landed on this repo hoping to get precisely what it announced: detection of USB plug-in. Your title says nothing of disks, but all it does is detect USB disks. I am trying to detect an Intel RealSense camera, which is clearly not storage and therefore doesn't get a drive letter.

Another highly misleading issue on the title is the detect part. Clearly English is not your native language, because when you announce detection of USB plug-in, you are saying that you detect the act of plugging in a USB device.

You do not do this, though. What you do is a polling spin scan on drives plugged. Polling is sooooo 1988 it hurts. I know, because I was already writing code then. Speaking of Jurassic Code, Python 2 was put to rest on New Year's Day 2020. print is now a function, in case no one told you. I swear Github should have a filter for Python 2 code, possibly brown.

And the argument that "Well, it works." falls through the cracks of polling interval, or CPU usage, among others. Threading would address this, if the GIL were not preventing real threading in pure Python.

All in all, perhaps a less alluring title, such as How-to-list-USB-plugged-in-disks-on-Windows would do this piece of code a bit more justice, while preventing people in search of true event notification from ending up in this dead end.

Thank you for wasting my time.