Skip to content

fix(gocopy): handle timeout clipboard clear errors#78

Open
neomantra wants to merge 1 commit into
atotto:masterfrom
NimbleMarkets:nm-gocopy
Open

fix(gocopy): handle timeout clipboard clear errors#78
neomantra wants to merge 1 commit into
atotto:masterfrom
NimbleMarkets:nm-gocopy

Conversation

@neomantra
Copy link
Copy Markdown

There's a code path in gocopy where the error from err = clipboard.WriteAll("") is unhandled and the wrong exit code is disseminated.

The outer/lower err != nil is looking at an err in a different scope than the clipboard clear (declared at clipboard.RealAll()).

Found by LLM, but understood and reviewed by me.

Signed-off-by: Evan Wies <evan@neomantra.net>
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.

1 participant