brson · GitHub

@brson

The panic hook already exits unconditionally, so all the unwinding
code is dead. This saves some build time and code size.
Signed-off-by: Brian Anderson <andersrb@gmail.com>

@brson

@ngaut

@brson

Signed-off-by: Brian Anderson <andersrb@gmail.com>

@brson

nrc

nrc previously approved these changes May 9, 2019

Signed-off-by: Brian Anderson <andersrb@gmail.com>

BusyJay

BusyJay previously approved these changes May 9, 2019

@brson

@brson

@brson

@brson

@brson

There's an interaction here that I don't quite understand,
that makes my change not work, so just reverting for now.
Signed-off-by: Brian Anderson <andersrb@gmail.com>

@brson

nrc

nrc approved these changes May 14, 2019

@brson brson mentioned this pull request

May 16, 2019

Merged

@brson

overvenus

@kennytm

sticnarf pushed a commit to sticnarf/tikv that referenced this pull request

Oct 27, 2019
* tikv: set panic=abort for dev and release
The panic hook already exits unconditionally, so all the unwinding
code is dead. This saves some build time and code size.
Signed-off-by: Brian Anderson <andersrb@gmail.com>
* panic_hook: make catching panics only possible in cfg(test)
Signed-off-by: Brian Anderson <andersrb@gmail.com>
* panic_hook: Clarify comments
Signed-off-by: Brian Anderson <andersrb@gmail.com>
* import: revert changes to test_panic_not_deadlocking
There's an interaction here that I don't quite understand,
that makes my change not work, so just reverting for now.
Signed-off-by: Brian Anderson <andersrb@gmail.com>

Read the original on github.com ↗