Remake of my MP3 Player using Windows Media Player

[stlaural]

Edit: Woops, I was typing at the same time I guess :P

I believe akhileshbc is talking about the 'goto' you are using.

Repeat:
x = MsgBox("Do you want to add more songs?", vbYesNo)
If x = vbYes Then
Music1.ShowOpen
frmSetup.Songopt(1).Value = True
frmSetup.Lstsongs.AddItem Music1.FileName
GoTo Repeat

Its just not considered a good practice anymore. Using loops makes your code more easily readable.