achow101 commented on this pull request.
> // Do not refer to data directory yet, this can be overridden by Intro::pickDataDirectory
— /// 2. Basic Qt initialization (not dependent on parameters or configuration)
+ /// 1. Basic Qt initialization (not dependent on parameters or configuration)
I’m not sure about unsafe, but it certainly is not good practice to check for whether the argument is set before actually parsing the arguments. This also makes it so that the `-uiplatform` option is effectively ignored. It appears that I have been misled by the comment on this line.
How would you recommend that an error be shown to the user if they don’t set the correct arguments?