Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Advanced Fatigue - Fix config macro #10334

Merged
merged 3 commits into from
Sep 24, 2024
Merged

Advanced Fatigue - Fix config macro #10334

merged 3 commits into from
Sep 24, 2024

Conversation

PabstMirror
Copy link
Contributor

Without arma crashes with

ErrorMessage: File z\ace\addons\advanced_fatigue\CfgEden.hpp, line 5: Config: End of line encountered after 
Application terminated intentionally

@PabstMirror PabstMirror added the ignore-changelog Release Notes: Excluded label Sep 24, 2024
@PabstMirror PabstMirror added this to the 3.18.0 milestone Sep 24, 2024
@johnb432
Copy link
Contributor

Without arma crashes with

ErrorMessage: File z\ace\addons\advanced_fatigue\CfgEden.hpp, line 5: Config: End of line encountered after 
Application terminated intentionally

What the hell - it's fine for me.

johnb432
johnb432 approved these changes Sep 24, 2024
View reviewed changes
Copy link
Contributor

@johnb432 johnb432 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressing the comment is optional, LGTM

@PabstMirror
Copy link
Contributor Author

it will work fine with hemtt as it can parse this macro and then output rapified configs
u can test with hemtt.exe launch --no-rap to have arma handle the configs natively and get the error

@PabstMirror @johnb432
Update addons/advanced_fatigue/CfgEden.hpp
14def74 
Co-authored-by: johnb432 <58661205+johnb432@users.noreply.github.com>
@PabstMirror PabstMirror merged commit ca3aa3f into master Sep 24, 2024
4 checks passed
@PabstMirror PabstMirror deleted the afmacro branch September 24, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johnb432 johnb432 johnb432 approved these changes

Assignees
No one assigned
Labels
ignore-changelog Release Notes: Excluded
Projects
None yet
Milestone
3.18.0
Development

Successfully merging this pull request may close these issues.
2 participants
@PabstMirror @johnb432