Adding an option to force loading of d3dx9_43.dll? about d3d8to9

[crosire]

Mmh. The error should be made more noticable, I agree. Could display an error messagebox instead of just silently writing a log message ...

[HerMajestyDrMona]

I noticed that before commit: "Clean up code" ( 9eb8faa )
When I added d3dx9.lib to additional dependencies, and file "C:\Windows\SysWOW64\D3DX9_43.dll" was missing - it still displayed an error that it's unable to start the application because d3dx9_43.dll is missing. So I suppose that one of those cleaning changes "fixed" it. Do you maybe know what change could cause it? Maybe I could add some unused function to the code, something to force d3dx9_43.dll loading? I don't know anything about DirectX API, so I would very appreciate any recommendation. I think system errors are better than message boxes created by a .dll file (when a .dll file is missing then some Windows process displays the error, not the app itself),

[crosire]

This was changed in and because of #16.

[HerMajestyDrMona]

You're right. I double checked it. Seems like I was testing in a wrong folder and I thought it already had #16 :) I'll see if I can revert this change later, or at least add a workaround.

[elishacloud]

I would not recommend reverting change #16. Instead you could add two lines to d3d8to9.cpp after this line:

LOG << "Failed to load d3dx9_43.dll! Some features will not work correctly." << std::endl;

Add these lines:

MessageBox(nullptr, TEXT("Failed to load \"d3dx9_43.dll\"!"), nullptr, MB_ICONWARNING);
return nullptr;

That should do what you want.

[HerMajestyDrMona]

Thank you. Seems like this solution is acceptable too :) I made more detailed error message with MB_OKCANCEL, and also ShellExecute with: https://www.microsoft.com/en-us/download/details.aspx?id=8109 when user clicks "OK". It opens the site where the missing "End-User Runtime" can be downloaded.

[crosire]

Closed in #23.