On Fri Aug 5 18:17:31 2022 +0000, Nikolay Sivov wrote:
We already have it as DllGetVersion() in current code and it compiles fine. If the only issue is that returned fields are "incorrect" in some way, you can simply fix them up. But then again, if nothing depends on it I wouldn't bother.
Yes, it compiles currently, as it doesn't follow the microsoft doc: it has HRETURN as return value instead of void and uses the DLLVERSIONINFO structure. This is both not compliant with the MS docs. As I had issues with the current implementation in early testing, I fixed both. Though, if that is not wanted, I can revert these changes. However, this seems to me more like kicking the can down the road to be honest: should an application really rely on it, it would have to be corrected again, even though this already is a working implementation (at least in my testing)
A second issue would be that I wouldn't know how to fix the return field if a wrong structure is handed over