NumCols should be an Integer that you entered in an inputbox. It is defined as Long so conceivably you are not putting an Integer in as the input value.

What are you seeing if you hover over the word NumCols when that line is being debugged?

__________________
Andrew Lockton
Chrysalis Design, Melbourne Australia