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

DxStation now sends 'R' after callsign correction #371

Merged

Conversation

w7sst
Copy link
Owner

@w7sst w7sst commented Oct 11, 2024

  • this problem was introduced in 1.85
  • Note: When MorePatience was introduced in May 2024, a bug (Issue Do not get an R from dx station after correcting a bad call #370) was introduced causing the DxStation to not send an 'R' after the user corrected a callsign. The case involved the user sending a corrected callsign using the Enter key while leaving the exchange fields blank (user sends ' ?'. In this case, the DxOperator.MsgReceived function would call MorePatience for the '?' and the Patience value was set to 4. This caused DxOperator.GetReply() to send the wrong response: DxOperator.GetReply(osNeedEnd, Patience=5) --> 'R ' DxOperator.GetReply(osNeedEnd, Patience=4) --> '' To fix this problem, MorePatience will maintain an existing Patience value of 5 (FULL_PATIENCE) and not set it to 4. Resolved in October 2024.
  • Issue Do not get an R from dx station after correcting a bad call #370

- this problem was introduced in 1.85
- Note: When MorePatience was introduced in May 2024, a bug (Issue #370) was
  introduced causing the DxStation to not send an 'R' after the user corrected
  a callsign. The case involved the user sending a corrected callsign using
  the Enter key while leaving the exchange fields blank (user sends
  '<his incorrect call> ?'. In this case, the DxOperator.MsgReceived function
  would call MorePatience for the '?' and the Patience value was set to 4.
  This caused DxOperator.GetReply() to send the wrong response:
      DxOperator.GetReply(osNeedEnd, Patience=5) --> 'R <HisCall>'
      DxOperator.GetReply(osNeedEnd, Patience=4) --> '<HisCall>'
  To fix this problem, MorePatience will maintain an existing Patience value
  of 5 (FULL_PATIENCE) and not set it to 4. Resolved in October 2024.
@w7sst w7sst requested review from f6fvy, scotthibbs and K6OK October 11, 2024 05:53
@w7sst w7sst self-assigned this Oct 11, 2024
@w7sst w7sst linked an issue Oct 11, 2024 that may be closed by this pull request
1 task
Copy link
Collaborator

@K6OK K6OK left a comment

Choose a reason for hiding this comment

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

👍

@w7sst w7sst merged commit 5767d92 into main Oct 14, 2024
@w7sst w7sst deleted the 370-do-not-get-an-r-from-dx-station-after-correcting-a-bad-call branch October 14, 2024 01:05
@w7sst w7sst added the verified Bug is fixed and verified label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified Bug is fixed and verified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Do not get an R from dx station after correcting a bad call
2 participants