[Libreoffice] OSL_ASSERT - remove backtrace or make abort
Thorsten Behrens
thb at documentfoundation.org
Wed Apr 20 09:51:46 PDT 2011
Lubos Lunak wrote:
> I'd like to remove the backtrace printing from OSL_ASSERT and friends, or,
> even better and if possible, make these functions work properly, i.e. abort
> on failure (I'm not really holding my breath on the second one, but refusing
> that one will at least support the first one).
>
> [...]
>
> Opinions?
>
Yes - first of all, there's SAL_DIAGNOSE_ABORT, to optionally enable
your desired behaviour. Then, there's sal-disable-backtrace.diff,
which I can happily merge - just set DISABLE_SAL_BACKTRACE then.
Regarding the problem itself, it's festering since many years, and
not easily reconcilable - in the sal/uno area, assertions *are*
serious, and should lead to aborts. Especially in the
application/filter area, though, those were indeed often used in a
"um, not sure, looks fishy here, let's do something"-kinda way.
Generally, cleaning that up (and converting the mis-used assertions
I mentioned into some warning) would be greatly appreciated.
Something for past-3-4 and a feature branch? IIRC OOo had some plans
to make at least smoketest completely 'assertion'-free.
Cheers,
-- Thorsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110420/19cf7970/attachment.pgp>
More information about the LibreOffice
mailing list