WIP - a branch with multiple commits for MF-AIX#261
Draft
lefessan wants to merge 12 commits intoOCamlPro:gitside-gnucobol-3.xfrom
Draft
WIP - a branch with multiple commits for MF-AIX#261lefessan wants to merge 12 commits intoOCamlPro:gitside-gnucobol-3.xfrom
lefessan wants to merge 12 commits intoOCamlPro:gitside-gnucobol-3.xfrom
Conversation
…25-11-25-gnucobol-3.x-mf-aix
Collaborator
|
Just a note: Instead of eprintf_token you want to use the (oboslete) %token-table. You may also want to have a look at https://www.gnu.org/software/bison/manual/html_node/Enabling-Traces.html (though this is not related to your scanner change), |
Collaborator
|
I highly encourage you to add test cases for what you want to achieve up-front. This does not only help to validate fast and have the tests "nearly ready" [you may want to adjust them during development}, but also is extra useful if someone has a look at the WIP changes to see its purpose. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This branch started as a merge of some other branches for MF-AIX, and then some other commits were added:
All these commits should probably be submitted through separate pull requests.