I noticed an arbitrary "4:" on one of the lines and I asked him why it was there.
He replied, "I don't know, but it doesn't seem to hurt anything".
I literally facepalmed.
That "4:" is a label. You can actually use old BASIC syntax like "GOTO 4" to jump to that line. It wasn't really "best practices" even for VB6.