coupling is something I noticed. if you call *GetMonthName* and pass an arg for the month name, than another arg to abbreviate. The same effect could be obtained writing a formatting function, Format(GetMonthName(arg), "mmm") or even better as you suggest calling Date. having the formatting within the function may be a short cut for now but I bet at sometime changes could be made introducing bugs.

its also pretty common trap to reimplement (usually poorly) functions that pre-exist.

Janitored by Arunbear - retitled from 'coupling', as per Monastery guidelines