Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Refactor "integer" types to "long" #178

Closed
hecon5 opened this issue Feb 23, 2021 · 3 comments
Closed

Refactor "integer" types to "long" #178

hecon5 opened this issue Feb 23, 2021 · 3 comments

Comments

@hecon5
Copy link
Contributor

hecon5 commented Feb 23, 2021

Slog:
Refactor integer types to long types, as with the current versions of VBA, integer uses the same memory space as long, but does not get the advantages of a long data type.

See StackOverflow: Why Use Integer Instead of Long?

@hecon5
Copy link
Contributor Author

hecon5 commented Mar 5, 2021

@joyfullservice; this probably should go to a discussion, as it's an over-arching concept, and might not affect anything, but could inform new development work ongoing.
(Putting this on the right issue this time........)

@hecon5 hecon5 closed this as completed Mar 5, 2021
@joyfullservice
Copy link
Owner

Perhaps we should make a page in the wiki where we describe the coding conventions and practices used in this project. Specific discussions like this could be added as discussion questions, and the wiki page updated per the outcome of the discussion.

Just so you know where I am at with some of these things, generally, I would lean towards the best practices promoted by RubberDuck VBA, with a few exceptions based on the purpose and target audience of this project.

@hecon5
Copy link
Contributor Author

hecon5 commented Mar 5, 2021

I agree with this; RD's coders are waaay more experienced and hooked into the current standards than I am, and also have the energy to build an auditor, so, I agree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants