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

fix setting of INDEX and POINTER data-items #316

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

GitMensch
Copy link
Contributor

needing SET, not MOVE

note: GnuCOBOL and ACUCOBOL-GT don't enforce this ISO rule, but this would fail to compile at least on IBM, so fixing it is

needing SET, not MOVE
@meyfa
Copy link
Owner

meyfa commented Jan 27, 2025

Yes, this should be done. Thanks for raising the issue!

There are many more instances of MOVE in place of SET. Is there an option to let GnuCOBOL enforce this rule specifically (besides going straight to full IBM compatibility)? I've had a look at the GC guide but couldn't find any option.

This would be helpful to prevent regressions, and also to find every occurrence initially.

@GitMensch
Copy link
Contributor Author

No, it is most likely reasonable to try compiling with a different compiler. gcobol may be possible after FOSDEM (COBOLworx wants to work on / push some necessary changes and can then likely be both installed per .deb and bring a "cobc driver" gcobc, where make COBC=gcobc GCVERSION=32 should do the trick.
I am likely to give that a go in 2-3 weeks and could work on this PR then (until then gnucobol will still compile that).

@meyfa meyfa marked this pull request as draft January 27, 2025 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants