Implement interactive command quit #9
Comments
jfkw
added a commit
to jfkw/uchess
that referenced
this issue
Mar 30, 2023
The quit command is not implemented although it is documented. cmd/uchess does handle Ctrl-C to exit currently. Add a pkg/cmd ProcessCmd() case handler for the command "quit". Add a private function pkg/cmd quit() where additional game state shutdown housekeeping could be processed, e.g. autosave on quit if desired. Currenly the function only returns the msg "quit", mirroring the hint command. Add a cmd/uchess Interact() handler for the msg "quit". On quit, ask for confirmation with Y or y. Exit process on confirm, without saving as is currently documented. Move the internal function Interact().quit() to a public function cmd/uchess Quit() where additional process shutdown housekeeping could be implemented. Currently the function only calls gamestate S.Fini() and os.Exit(0) the process with return code.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Interactive command
quitis documented in README. However, there is no case handler forquitinpkg/cmdfunc ProcessCmd().The interactive tcell loop in
cmd/uchessfunc Interact()handlesCtrl-C, so there is currently a way to quit the app.When implementing, avoid coupling
pkg/cmdandcmd/uchesssuch that they both handle process quit concerns with overlap. The goal should be forpkg/cmdto handle game state shutdown, andcmd/uchessto handle process shutdown.The text was updated successfully, but these errors were encountered: