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

Consider using enums on OleAuto varformat functions #1086

Closed
elachlan opened this issue Aug 21, 2022 · 5 comments · Fixed by #1245
Closed

Consider using enums on OleAuto varformat functions #1086

elachlan opened this issue Aug 21, 2022 · 5 comments · Fixed by #1245
Assignees
Labels
missing enum An enum is missing for constant parameters WinForms

Comments

@AArnott AArnott added the missing enum An enum is missing for constant parameters label Aug 21, 2022
@chenss3 chenss3 self-assigned this Aug 29, 2022
@chenss3
Copy link
Collaborator

chenss3 commented Sep 1, 2022

Currently working on adding an enum for the flags.

I'm also looking at in VarFormat there are a set of values for iFirstDay and iFirstWeek. Has WinForms built a custom enum for this? If that's the case, it might make sense to do the same in the metadata since we don't currently have any constants defined in the headers for those values.

@elachlan
Copy link
Contributor Author

elachlan commented Sep 2, 2022

@chenss3 can you please also look into ivbformat?
https://docs.microsoft.com/en-us/windows/win32/api/vbinterf/nf-vbinterf-ivbformat-format

@chenss3
Copy link
Collaborator

chenss3 commented Sep 9, 2022

@elachlan I also see that VarFormatPercent and other functions use a similar set of values for iIncLead, iUseParens, and iGroup. Do you know if WinForms has also built custom enums for these values?

@elachlan
Copy link
Contributor Author

elachlan commented Sep 9, 2022

For that example I suggest creating an enum. Its limited to three separate values, which is perfect for an enum.

As far as what is used by winforms, its just varformat and ivbformat. I added the rest as a guide for myself to add enums to related methods/structs when I was going to go back through and complete this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
missing enum An enum is missing for constant parameters WinForms
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants