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

Fixes, mostly RigParser #509

Merged
merged 6 commits into from Jan 4, 2016

Conversation

Projects
None yet
1 participant
@only-a-ptr
Member

only-a-ptr commented Dec 30, 2015

RigParser improvements, mostly loosening the super-strict syntax rules.

only-a-ptr added some commits Dec 29, 2015

[fix] RigParser/TractionControl: invalid mode print warning only
Previously, invalid 'mode' attribute would cause the whole line to be ignored.
[fix] Parser/rotators[2]: accept comma/space separators.
Previously comma-only.
NOTE: It appears there was a bug in rotator[2] processing - the _ParseOptionalInertia() function expected E_CAPTURE(E_DELIMITER)s, but was also used with E_ROTATORS_R1_R2_COMMON_INERTIA which used E_DELIMITER_COMMAs
[fix] Parser/nodes: Added data print for verification.
If the parser falls back to legacy unsafe method while parsing 'nodes[2]', it will print a WARNING log message containing all parsed data + source input text, including invalid trailing text.

only-a-ptr added a commit that referenced this pull request Jan 4, 2016

@only-a-ptr only-a-ptr merged commit aa756e7 into RigsOfRods:master Jan 4, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment