Skip to content

Adding O_spec feature#487

Open
Monikacopilot2 wants to merge 3 commits intocodefori:mainfrom
Monikacopilot2:O-spec_feature
Open

Adding O_spec feature#487
Monikacopilot2 wants to merge 3 commits intocodefori:mainfrom
Monikacopilot2:O-spec_feature

Conversation

@Monikacopilot2
Copy link
Copy Markdown

Added comprehensive O-spec (Output Specification) parsing support to the fixed-form RPG parser in language/models/fixed.js.
Changes Made
New Function: parseOLine()
Implemented a complete parser for fixed-form RPG Output specifications that extracts all standard O-spec fields according to IBM i RPG column positions

@mounigopisetti1162
Copy link
Copy Markdown

I tested the code, and it’s working as expected. I’m able to see the O-specs.

@bobcozzi
Copy link
Copy Markdown
Contributor

I've added basic O and I spec "Ruler" support to the RPGLE extension. To support O and I better.

@worksofliam
Copy link
Copy Markdown
Member

  1. There are not enough test cases for O-spec support. Need a lot more, since there are lots of edge cases.
  2. The bigger discussion that is needed is I and O spec support in general. I did start on I support and made good progress, but I never finished it. (Branch feature/i_spec)

Copy link
Copy Markdown
Member

@worksofliam worksofliam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comment above

@Monikaibmi15
Copy link
Copy Markdown

Added testcase for O-spec for support.

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.

5 participants