IEnumExplorerCommand.Next array marshaling SizeParameterIndex incorrect?

https://github.com/dahall/Vanara/blob/c88bf0acfd4e0b209b5ed6d2d688fd22a1aad37a/PInvoke/Shell32/ShObjIdl.IExplorerCommand.cs#L115

Here, should the celt be used as the size parameter, instead of pceltFetched? That is should the SizeParamIndex be set to 0, instead of 2?

The documentation says :

celt Specifies the number of elements to fetch. pceltFetched When this method returns, contains a pointer to the number of elements actually retrieved.

When this method gets called, the out parameter pceltFetched will always be the default value zero, so, seems pUICommand should be initialized as an array of size celt, instead of pceltFetched? otherwise, the initial value of pUICommand will always be an empty array and can’t be populated later.

Also see: https://github.com/wjk/MSIExtractApp/blob/ba59307319a97c8a1d0a49bdaaa5be54fbf479af/src/MSIExtract.ShellExtension/Interop/IEnumExplorerCommand.cs#L18

[dahall]

Thank you. That helps my understanding immensely. I think I need to expand my testing to including implementations and then adjust all the SizeParamIndex values accordingly.

[dahall]

I have updated all interfaces with similar patterns to using the input size parameter for SizeParamIndex.