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.

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

sqf_validator doesn't support multi-line statements #5979

Closed
Dystopian opened this issue Dec 26, 2017 · 4 comments
Closed

sqf_validator doesn't support multi-line statements #5979

Dystopian opened this issue Dec 26, 2017 · 4 comments
Assignees
Milestone

Comments

@Dystopian
Copy link
Contributor

ACE3 Version: 3.12.0 (dev + 4cfa64d)

Description:
SQF preprocessor supports multi-line statements with \ at the line end, including inside strings and excluding comments. E.g. this code

a=1 \
 + 2;
b="q\
s";
c="z
x"; \
// c1
// c2 \
// c3
// c4 \
a

is preprocessed to

a=1  + 2;

b="qs";

c="z
x"; 




a

It looks like tools\sqf_validator.py doesn't support such multi-line statements and produces false errors in some cases.

Steps to reproduce:

  • create test.sqf in addons\ai folder with this correct sqf content:
[{hint "1"} \
] call CBA_fnc_waitAndExecute;
  • run sqf_validator.py -m ai;
  • see false error ERROR: Possible missing semi-column ';' detected at addons/ai\test.sqf Line number: 1

Where did the issue occur?

  • tools
@kymckay
Copy link
Member

kymckay commented Dec 28, 2017

That python script will become obsolete with the introduction of #5468

@stale
Copy link

stale bot commented Nov 18, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status/stale label Nov 18, 2018
@dedmen
Copy link
Contributor

dedmen commented Nov 18, 2018

Will be fixed by #5468

@PabstMirror
Copy link
Contributor

sorta fixed by #8566

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants