byronwall/bUTL

Declared variants can be strings

RaymondWise opened this issue · 1 comments

Re: #51 (comment)

These variables may be able to be declared as strings rather than variants

Variable Module
Variable 'folder' bUTLAddIn.RandomCode
Variable 'path' bUTLAddIn.RandomCode
Variable 'str_folder' bUTLAddIn.RandomCode

I suspect they can be strings as well. The RandomCode section of the add-in is a little bit of the wild west. I think it should be properly buried if the code can't be justified in another module. That code came through from things that I assumed would be useful someday. I think that assumption may not prove out.

In the short term, we can clean this code up to make it consistent with best practices. In the long term, this code may not be useful or worth keeping in the add-in.