]> Cypherpunks repositories - gostls13.git/commit
runtime: remove unused badsignal2 on windows
authorqmuntal <quimmuntal@gmail.com>
Mon, 23 Jan 2023 11:59:29 +0000 (12:59 +0100)
committerQuim Muntal <quimmuntal@gmail.com>
Tue, 24 Jan 2023 12:56:06 +0000 (12:56 +0000)
commit7b5a34418cd32ec4d2c0f7d6e4c7e6fe7585a17f
treea14b2e6edcbbdd789633f924f4d0af222c183e23
parentcf9263dee1bb160f013a080bbda3532a7d35da15
runtime: remove unused badsignal2 on windows

This CL removes badsignal2 function, as it is unused on Windows.

badsignal2 was originally intended to abort the process when
an exception was raised on a non-Go thread, following the same approach
as Linux and others.

Since it was added, back on https://golang.org/cl/5797068, it has caused
several issues on Windows, see #8224 and #50877. That's because we can't
know wether the signal is bad or not, as our trap might not be at the
end of the exception handler chain.

To fix those issues, https://golang.org/cl/104200046 and CL 442896
stopped calling badsignal2, and CL 458135 removed one last incorrect
call on amd64 and 386.

Change-Id: I5bd31ee2672118ae0f1a2c8b46a1bb0f4893a011
Reviewed-on: https://go-review.googlesource.com/c/go/+/463116
Reviewed-by: Bryan Mills <bcmills@google.com>
Run-TryBot: Quim Muntal <quimmuntal@gmail.com>
Reviewed-by: Alex Brainman <alex.brainman@gmail.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
src/runtime/os_windows.go
src/runtime/signal_windows.go
src/runtime/sys_windows_386.s
src/runtime/sys_windows_amd64.s
src/runtime/sys_windows_arm.s
src/runtime/sys_windows_arm64.s