SDL_TriggerBreakpoint on Android is incorrect

[madebr]

SDL/include/SDL3/SDL_assert.h

Lines 135 to 137 in 9ad04ff

	#elif defined(ANDROID) || defined(__SYMBIAN32__)
	#include <assert.h>
	#define SDL_TriggerBreakpoint() assert(0)

The assert macro is disabled when building with -DNDEBUG, so SDL_TriggerBreakpoint might inadvertently become a no-op.

The Android case was added by @VladSerhiienko in 47deebe (#7202)
The Symbian case was added by @mupfdev in 7ae6459

I propose to remove this case, or if no replacement can be found, add a && !defined(NDEBUG) to the conditional

I suppose the N-Gage ARM processor has some builtin and/or assembly operation to replace this with?

[mupfdev]

Hey! Possibly, however, since we don't have any way to debug a native Symbian application, it's irrelevant to the N-Gage port.

[madebr]

I just tested this on aarch64 Android, and removing this case will cause __builtin_trap() to be used.
This gets replaced with brk #0xf000.
This breakpoint is fatal. I could not find out how to continue execution after it.
But the original assert(0) is also fatal, so nothing changed.

The debug experience is even better by using __builtin_trap(): the debugger stops in the correct frame.
Using assert(0), the debugger stops inside abort().

[madebr]

@mupfdev
Can you test #13979 for N-Gage?
Build a test app and put SDL_TriggerBreakpoint(); somewhere.
I know you don't have a debug setup, but at the least it should not fail the build.

32-bit ARM has bkpt #imm16, but this is armv5t+.
Looks like N-Gage is armv4t.

[madebr]

Fixed by #13979

[madebr]

Re-opening and assigning to @mupfdev for N-gage verification