Chromium Code Reviews| Index: base/debug/debugger_posix.cc |
| diff --git a/base/debug/debugger_posix.cc b/base/debug/debugger_posix.cc |
| index 0fbefdebb722db6b2859cff73348df8058c68220..53b09468d6a007e703476360cd2c1ed6a3c645c1 100644 |
| --- a/base/debug/debugger_posix.cc |
| +++ b/base/debug/debugger_posix.cc |
| @@ -218,16 +218,19 @@ bool BeingDebugged() { |
| // SIG triggered by native code. |
| // |
| // Use GDB to set |go| to 1 to resume execution. |
| -#define DEBUG_BREAK() do { \ |
| - if (!BeingDebugged()) { \ |
| - abort(); \ |
| - } else { \ |
| - volatile int go = 0; \ |
| - while (!go) { \ |
| - base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(100)); \ |
| - } \ |
| - } \ |
| -} while (0) |
| +void DebugBreak() { |
|
darin (slow to review)
2013/03/20 06:24:34
nit: oh, this should probably have static linkage
|
| + do { |
| + if (!BeingDebugged()) { |
| + abort(); |
| + } else { |
| + volatile int go = 0; |
| + while (!go) { |
| + base::PlatformThread::Sleep(base::TimeDelta::FromMilliseconds(100)); |
| + } |
| + } |
| + } while (0); |
| +} |
| +#define DEBUG_BREAK() DebugBreak() |
| #else |
| // ARM && !ANDROID |
| #define DEBUG_BREAK() asm("bkpt 0") |