Skip to content

Enhancement of PPK_API object destructor to reset the PPK2 before closing the serial connection#40

Merged
wlgrd merged 1 commit intoIRNAS:masterfrom
rgroh1996:enhancement/destructor-ppk2-reset
Nov 30, 2023
Merged

Enhancement of PPK_API object destructor to reset the PPK2 before closing the serial connection#40
wlgrd merged 1 commit intoIRNAS:masterfrom
rgroh1996:enhancement/destructor-ppk2-reset

Conversation

@rgroh1996
Copy link
Copy Markdown
Contributor

The PPK2 was not reset when the object was deleted. The PPK2 is reset with the PPK2 RESET command. This allows a new connection to be established immediately.

@wlgrd
Copy link
Copy Markdown
Collaborator

wlgrd commented Nov 21, 2023

Nice addition. Please update your commit title to adhere to github's max length.
Suggestion, do git commit --amend, and change to

send reset command in destructor

The PPK2 was not reset when the object was deleted.
The PPK2 is reset with the PPK2 RESET command.
This allows a new connection to be established immediately.

then do a git push --force after saving the new commit

The PPK2 was not reset when the object was deleted.
The PPK2 is reset with the PPK2 RESET command.
This allows a new connection to be established immediately.
@rgroh1996 rgroh1996 force-pushed the enhancement/destructor-ppk2-reset branch from f81d579 to abefe62 Compare November 22, 2023 15:35
@wlgrd wlgrd self-requested a review November 30, 2023 10:10
@wlgrd wlgrd merged commit e3fb545 into IRNAS:master Nov 30, 2023
@juhhov
Copy link
Copy Markdown

juhhov commented Aug 26, 2024

Could this be included into release? @wlgrd @NejcKle

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.

3 participants