If no d3voide1.hog or d3voice2.hog in search path was found, Descent3 fails with segmentation fault after pressing cancel on "Please insert Descent 3 CD" dialogue on Linux. Added guard checks which gracefully handle this issue.
Refactored GetMultiCDPath() function which greatly simplifies it's logic and remove rare and little of use cmd-line options:
Removed -moviedir option as is only available on DEBUG build and useless in development environment (we can define movies location in many other ways).
Removed -intro option as is there a little use to redefine intro.mve location (again, we can control movies location in other ways).
Related Issues
Screenshots (if applicable)
Checklist
[x] I have tested my changes locally and verified that they work as intended.
[x] I have documented any new or modified functionality.
[x] I have reviewed the changes to ensure they do not introduce any unnecessary complexity or duplicate code.
[x] I understand that by submitting this pull request, I am agreeing to license my contributions under the project's license.
Pull Request Type
Description
If no d3voide1.hog or d3voice2.hog in search path was found, Descent3 fails with segmentation fault after pressing cancel on "Please insert Descent 3 CD" dialogue on Linux. Added guard checks which gracefully handle this issue.
Refactored GetMultiCDPath() function which greatly simplifies it's logic and remove rare and little of use cmd-line options:
-moviedir
option as is only available on DEBUG build and useless in development environment (we can define movies location in many other ways).-intro
option as is there a little use to redefine intro.mve location (again, we can control movies location in other ways).Related Issues
Screenshots (if applicable)
Checklist
Additional Comments