bluegr/scummvm — comment on commit 0d70a51b91f191f5ac182696eb8b497d7b17a21e
File: engines/sci/detection.cpp line 715
By lordhoto:

If I got the old code correctly, we only tried to add options which are usable for the specified target here. Now we always add the KQ6 specific cursor setting for example. This seems like a step backwards from a usability point of view.

Maybe it would make more sense to only store the GUI options as a nested array, but still only return an Array of options which actually apply? Note that this could still be done with the suggestion to use IDs to account for this problem. The benefit of returning an Array with all available options is just that we can easily decouple the client code implementation from the details on how we store the available options internally.