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.

Sign up for GitHub

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

Jump to bottom

Memory leak of TContest.Stations, TContest.Agc and TQsb objects #31

Closed
w7sst opened this issue Sep 24, 2022 · 0 comments · Fixed by #35
Closed

Memory leak of TContest.Stations, TContest.Agc and TQsb objects #31

w7sst opened this issue Sep 24, 2022 · 0 comments · Fixed by #35
Assignees
@w7sst
Labels
bug Something isn't working

Comments

@w7sst
Copy link
Owner

w7sst commented Sep 24, 2022

Description

When testing original MorseRunner code under Lazarus compiler/debugger w/ memory checking enabled, three memory leaks were found.

Steps To Reproduce

  1. Compile with Lazarus (this must be done with a different code base; this code base compiles with Delphi).
  2. Enable memory checking
  3. Run the application
  4. Exit
  5. View memory error report dialog (appears after exit).

Expected behavior

No memory leaks should be reported.

Actual Behavior

Complaints about three memory leaks mentioned above.

Reproduces how often

Always

Version information

Latest VE3NEA code base.
@w7sst w7sst added the bug Something isn't working label Sep 24, 2022
@w7sst w7sst self-assigned this Sep 24, 2022
w7sst added a commit that referenced this issue Sep 28, 2022
@w7sst
Issue #31 - bug - fix memory leaks in TContest.Stations, TContest.Agc…
42273e0 
… and TQsb objects
@w7sst w7sst mentioned this issue Sep 28, 2022
Merged
@w7sst w7sst closed this as completed in #35 Sep 29, 2022
w7sst added a commit that referenced this issue Sep 29, 2022
@w7sst
Merge fixes for Issue #31, #32, #33, #34 into main branch (#35)
8a19732 
Fix several small bugs on original VE3NEA code base:
- Fixes #31
- Fixes #32
- Fixes #33
- Fixes #34

Will be merged into FD prototype after this is merged into main.
@scotthibbs scotthibbs mentioned this issue Oct 8, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@w7sst w7sst

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Merge fixes for Issue #31, #32, #33, #34 into main branch w7sst/MorseRunner
1 participant
@w7sst