-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PW3]: padding a macro argument
shown twice
#642
Comments
Not able to create a MRE, will need to confirm this is still occuring on the next release. This currently only produces a single error for me #include "script_macros_common.hpp"
ERROR_1("Invalid triggers in Trigger Markers! Make sure all triggers are correctly set! (%1)", _name); |
I'll give it another test on latest. |
Still happens twice at https://github.com/Theseus-Aegis/Mods/blob/943906d18817a7f0829742f0c91cdad34456c594/addons/shootingrange/functions/fnc_create.sqf#L89 on 2fd248b You can see it in CI as well: https://github.com/BrettMayson/HEMTT/actions/runs/7953825250/job/21710390932#step:8:5080 |
Also ran into this and took a quick look around, and frankly getting lost in the code base (wont be making a PR 😅). HEMTT/libs/preprocessor/tests/warnings.rs Lines 40 to 46 in cf8d577
Changing that to something along the lines of: let warning = config
.warnings()
.iter()
.map(|w| w.diagnostic().unwrap().to_string(&WorkspaceFiles::new()))
.collect::<String>(); Makes the example posted by Brett also spit out this warning. |
The text was updated successfully, but these errors were encountered: