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

Parser compatibility fixes #674

Merged
merged 6 commits into from Jan 31, 2016

Conversation

Projects
None yet
2 participants
@only-a-ptr
Member

only-a-ptr commented Jan 31, 2016

Fixes #502

only-a-ptr added some commits Jan 30, 2016

[fix] Section "ties": Fixed parsing options, more error tolerance.
Details:
* Both comma and space accepted as separators.
* Tolerating trailing separator
Updated copyright notice formatting + year.
[fix] Parser/section "flexbodies/forset": tolerance++
Now tolerating botched forset lines like "forset12,34,56". All 3 nodes (12,34,56) must be accepted for backwards compatibility.
[add] Parser: more tolerant E_DELIMITER
Now tolerates duplicate separators - quite a common occurence in old truckfiles.
@ulteq

This comment has been minimized.

Show comment
Hide comment
@ulteq

ulteq Jan 31, 2016

Contributor

👍

But the parser also fails at parsing the hooks and lockgroups of this truck ISOcontainers.

I think we must always allow trailing separators (concluding comments).

Contributor

ulteq commented Jan 31, 2016

👍

But the parser also fails at parsing the hooks and lockgroups of this truck ISOcontainers.

I think we must always allow trailing separators (concluding comments).

@only-a-ptr

This comment has been minimized.

Show comment
Hide comment
@only-a-ptr

only-a-ptr Jan 31, 2016

Member

I think we must always allow trailing separators (concluding comments).

Yeah. * sigh *
I made a ticket to myself: #677

Member

only-a-ptr commented Jan 31, 2016

I think we must always allow trailing separators (concluding comments).

Yeah. * sigh *
I made a ticket to myself: #677

ulteq added a commit that referenced this pull request Jan 31, 2016

@ulteq ulteq merged commit 4dc6575 into RigsOfRods:master Jan 31, 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