Optional not working in Excel 2007

[AlexanderBB]

Sub DoComment(r As Range, T, Optional Append = False, Optional theSize = 10)

After much debugging, I find theSize above is NOT 10 !!!
Just confirmed by stepping.

It works correctly in Excel 2003.

Can I fix it ? Thanks

 

[Jaafar Tribak]

Not sure what you are doing but are you changing the value of the theSize argument inside the DoComment Sub ?

Try passing the argument ByVal and see if it makes any difference :

Sub DoComment(r As Range, T, Optional Append = False, Optional ByVal theSize = 10)

 

[AlexanderBB]

This is really weird!
I wasn't changing theSize, for testing there was a Stop on the first line in DoComment, and I was running the mouse over the variable name. The balloon showed 'Empty'.
So I added ByVal and it worked. Then I removed ByVal out to recheck, and it still worked.

The size being empty brought up an Application Defined Error 'Unable to Set the Size of the Font Class". This was fixed by hard coding a size.
Thanks for suggesting ByVal, and all seems ok now... but a bit worrying !

 

[Jaafar Tribak]

This is really weird!
I wasn't changing theSize, for testing there was a Stop on the first line in DoComment, and I was running the mouse over the variable name. The balloon showed 'Empty'.
So I added ByVal and it worked. Then I removed ByVal out to recheck, and it still worked.

The size being empty brought up an Application Defined Error 'Unable to Set the Size of the Font Class". This was fixed by hard coding a size.
Thanks for suggesting ByVal, and all seems ok now... but a bit worrying !

That's weird.. It should have the value set by default in the sub definition.