Patches
Thiago Macieira
thiago at macieira.org
Mon May 12 18:55:05 PDT 2014
Em ter 13 maio 2014, às 10:01:12, Dirk Hohndel escreveu:
> I disagree. Crashing in a release build is NOT a good way to let the user
> know that something is wrong. We should avoid crashes as much as humanly
> possible.
Q_ASSERT in debug mode is not an answer either, since the effect is pretty much
equivalent to that of a crash (except one gets a SIGABRT instead of SIGSEGV).
Q_ASSERTs and C assert()s are meant for devs only. For that reason, Tomaz's
patch is fine: it doesn't remove or introduce any new protection.
That said, the question stands: can it happen?
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
Software Architect - Intel Open Source Technology Center
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
More information about the subsurface
mailing list