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

Allow keeping bool return values #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

petrutlucian94
Copy link
Member

PyMI contains a workaround to avoid returning boolean values for void MI functions, mostly in an attempt to remain compatible with other WMI libraries.

The problem is that we're losing all boolean return values. We could just drop the workaround and let PyMI return boolean values even for void functions, however that might break some applications.

Instead, we'll add an optional argument called "keep_bool_ret_vals", which can explicitly request the boolean return values to be preserved.

PyMI contains a workaround to avoid returning boolean values for
void MI functions, mostly in an attempt to remain compatible with
other WMI libraries.

The problem is that we're losing boolean return values. We could
just drop the workaround and let PyMI return boolean values even
for void functions, however that might break some applications.

Instead, we'll add an optional argument called
"keep_bool_ret_vals", which can explicitly request the boolean
return values to be preserved.
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

Successfully merging this pull request may close these issues.

1 participant