Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Declared variants can be strings #54

Open
RaymondWise opened this issue Apr 14, 2016 · 1 comment
Open

Declared variants can be strings #54

RaymondWise opened this issue Apr 14, 2016 · 1 comment

Comments

@RaymondWise
Copy link
Contributor

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
@byronwall
Copy link
Owner

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.

@byronwall byronwall reopened this Apr 15, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants