On Wed Mar 15 17:28:17 2023 +0000, Rémi Bernon wrote:
I don't know if it's very useful either, but I believe (though it's been a long time since I checked this) that it is how MIDL behaves: when preprocessing is disabled it is disabled globally. It's mostly there to use `open_input_file` for the main file in the next change.
What's the midl option for disabling preprocessor? I couldn't find it.
Note that this change can potentially break working reasonable scenarios, like importing SDK IDLs in a IDL file that doesn't use preprocessor and then building it with -N. It's probably not a big deal as I don't expect -N to be popular, but if using the helper in main is the only reason for the change, I'd suggest to handle no_preprocess in main instead. You may still use the helper for the rest of the job.