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

Use single-line WTOs for single-line messages #513

Open
wants to merge 1 commit into
base: v3.x/staging
Choose a base branch
from

Conversation

ifakhrutdinov
Copy link
Contributor

Proposed changes

This pull-request adjusts the wtoPtintf2 function, which is intended for printing single-line messages, to use the proper WTO call and issue single-line WTO messages.

This PR addresses Issue: #509

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

PR Checklist

Please delete options that are not relevant.

  • If the changes in this PR are meant for the next release / mainline, this PR targets the "staging" branch.
  • My code follows the style guidelines of this project (see: Contributing guideline)
  • New and existing unit tests pass locally with my changes
  • Relevant update to CHANGELOG.md
  • My changes generate no new warnings

Testing

  • Start our ZIS;
  • Verify that, in the JESMSGLG log, the ZIS single-line WTO messages like ZWES0109I have no numbers at the end:
incorrect:    ZWES0109I Core server ready 690
correct:      ZWES0109I Core server ready
  • Multi-line messages (e.g., ZWES0200I) should still have the numbers indicating multiple-lines:
08.39.55 S0442115  ZWES0200I Modify commands:  165
   165                        DISPLAY [OPTION_NAME] - Print service information
   165                          OPTION_NAME:
   165                            CONFIG - Print server configuration information (defau
   165                        FLUSH   - Print all pending log messages
   165                        LOG <COMP_ID> <LOG_LEVEL> - Set log level
   165                          COMP_ID:
   165                            CMS   - Cross memory server
   165                            CMSPC - PC routines
   165                            STC   - STC base
   165                          LOG_LEVEL:
   165                            SEVERE
   165                            WARNING
   165                            INFO
   165                            DEBUG
   165                            DEBUG2
   165                            DEBUG3

Further comments

Fixes: #509

Signed-off-by: Irek Fakhrutdinov <ifakhrutdinov@rocketsoftware.com>
@ifakhrutdinov ifakhrutdinov added the bug Something isn't working label Feb 11, 2025
@ifakhrutdinov ifakhrutdinov self-assigned this Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant