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