Have `ITaskFolder::GetTasks` use `Option<TASK_ENUM_FLAGS>` instead of `i32`

[Zerowalker]

Suggestion

Currently ITaskFolder::GetTasks has this definition GetTasks(&self, flags: i32).
But as It only allows for TASK_ENUM_FLAGS it should be GetTasks(&self, flags: Option<TASK_ENUM_FLAGS>),
if I'm not mistaken, or perhaps add a:
pub const TASK_ENUM_ALL: TASK_ENUM_FLAGS = TASK_ENUM_FLAGS(0i32);
Though I'm guessing that is a bad idea as you making up a flag that doesn't exist.
Having an Option allows for the flag to be set, and if None it's simply 0.

I tried finding it in win32metadata to no avail, but I am guessing the "fault" lies there?
I think it should be defined here, but it's not: https://github.com/microsoft/win32metadata/blob/main/generation/WinSDK/RecompiledIdlHeaders/um/MSTask.idl

Thanks:)!

[tim-weis]

The ITaskFolder interface is defined in taskschd.h with this signature for GetTasks():

HRESULT GetTasks(LONG flags, IRegisteredTaskCollection **ppTasks)

LONG is properly translated to i32. Changing the Rust mapping to use TASK_ENUM_FLAGS instead would require modifying the interface definition, but it's too late for that. And there isn't enough value to justify addressing this from the code generator either.

Unless I'm overlooking a manageable solution, I don't think there is anything that can be done here to improve the situation (and the situation isn't terrible to begin with).