in reply to
Re^5: The joys of bad code
in thread The joys of bad code
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