Skip to content

Tracking PR for diffing oo4gnucobol-3.x against gitside-gnucobol-3.x - #290

Draft
nberth wants to merge 11 commits into
gitside-gnucobol-3.xfrom
oo4gnucobol-3.x
Draft

Tracking PR for diffing oo4gnucobol-3.x against gitside-gnucobol-3.x#290
nberth wants to merge 11 commits into
gitside-gnucobol-3.xfrom
oo4gnucobol-3.x

Conversation

@nberth

@nberth nberth commented May 12, 2026

Copy link
Copy Markdown
Contributor

⚠️ do not merge ⚠️

kmr-srbh added 2 commits May 5, 2026 14:35
…warning

* Add support for parsing an "empty" `CLASS-ID` paragraph with no contained paragraphs and divisions.

* Emit a "unimplemented" warning and exit.

Signed-off-by: Saurabh Kumar <developer.saurabh@outlook.com>
Signed-off-by: Saurabh Kumar <developer.saurabh@outlook.com>
@nberth
nberth force-pushed the oo4gnucobol-3.x branch from bcf2e70 to 1bcf9fb Compare May 13, 2026 19:08
kmr-srbh and others added 8 commits May 15, 2026 00:04
Signed-off-by: Saurabh Kumar <developer.saurabh@outlook.com>
Signed-off-by: Saurabh Kumar <developer.saurabh@outlook.com>
feat (parser): parse empty `CLASS-ID` paragraph with "unimplemented" warning
…plemented warning

Adds support for the following class attributes:

* AS literal
* IS FINAL
* INHERITS FROM {class-name ...}
* USING {param-name ...}

Also support MF extensions IS STATIC, IS ABSTRACT, IS PARTIAL,
IS PUBLIC, IS INTERNAL.

Signed-off-by: Saurabh Kumar <developer.saurabh@outlook.com>
Signed-off-by: Saurabh Kumar <developer.saurabh@outlook.com>
feat (parser): add parsing support for all class attributes with "unimplemented" warning
@GitMensch

Copy link
Copy Markdown
Collaborator

All checks having passed and no conflicts look nice.

Is it time to re-inspect this or another PR to work on integration to gc3.x?

@nberth

nberth commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

All checks having passed and no conflicts look nice.

Is it time to re-inspect this or another PR to work on integration to gc3.x?

Maybe a close look at #293 may be useful, to see what could still be missing before that can be pushed to 3.x upstream (maybe tests… but I lack sources for good example OO code).

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants