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

Replace STOP RUN with GOBACK #205

Merged
merged 1 commit into from Jan 27, 2021

Conversation

tanto259
Copy link
Member

@tanto259 tanto259 commented Jan 27, 2021

Signed-off-by: Hartanto Ario Widjaya tanto259@users.noreply.github.com

One of the issue with having STOP RUN is that it causes problems with subroutine CALL. In particular if a user decide to call a subroutine twice, and the subroutine contain a STOP RUN, the entire thing will terminate before the second execution.

In particular, this may have causes this issue on Slack: https://openmainframeproject.slack.com/archives/C011NE32Z1T/p1611697726073500

Thus, I believe it's in the best interest to replace all occurrence of STOP RUN to GOBACK since the programs will terminate more gracefully (i.e. the CALL-ed program will return control back to the caller instead of terminating everything).

Signed-off-by: Hartanto Ario Widjaya <tanto259@users.noreply.github.com>
@MikeBauerCA
Copy link
Contributor

@paulnewt any thoughts on this? If accepted, we will need to update the COBOL programs on the system.

@paulnewt
Copy link
Contributor

STOP RUN replaced with GOBACK in all source located in ZOS.PUBLIC.SRC, ZOS.PUBLIC.ADHOC,CBL, and ZOS.PUBLIC.DB2.CBL

Copy link
Contributor

@MikeBauerCA MikeBauerCA left a comment

Choose a reason for hiding this comment

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

Okay, this looks good to me. I'm sure there are a few screenshots that should be updated, but I do not want to block this contribution for that reason. We can tackle that when we update all screenshots to solve #110 and #151.

@MikeBauerCA MikeBauerCA merged commit a2040fc into openmainframeproject:master Jan 27, 2021
@tanto259 tanto259 deleted the change-stop-run branch April 20, 2021 16:44
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

3 participants