Skip to content

single instance only#46

Merged
Kyome22 merged 2 commits into
runcat-dev:masterfrom
0x00000FF:master
Nov 26, 2021
Merged

single instance only#46
Kyome22 merged 2 commits into
runcat-dev:masterfrom
0x00000FF:master

Conversation

@0x00000FF
Copy link
Copy Markdown
Contributor

This kind of proposal was already submitted, #42, however, single instance with process name can be makeshift.
Mutex-based would be more useful, I think.

@Kyome22
Copy link
Copy Markdown
Collaborator

Kyome22 commented Nov 19, 2021

@0x00000FF Thanks for your contribution!
Reading the Mutex reference, it seems like we need to call ReleaseMutex(), but is it unnecessary?
What are you referring to for implementation?

@Kyome22 Kyome22 mentioned this pull request Nov 19, 2021
@0x00000FF
Copy link
Copy Markdown
Contributor Author

0x00000FF commented Nov 21, 2021

Oh I forgot that, sorry! 😅

Actually the mutex will be released automatically when the process exited, but that would be more desirable.
So I added a commit, please check it out!

@Kyome22
Copy link
Copy Markdown
Collaborator

Kyome22 commented Nov 26, 2021

@0x00000FF
LGTM🎉

@Kyome22 Kyome22 merged commit 8339a1e into runcat-dev:master Nov 26, 2021
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.

2 participants