diff options
| author | Bjoern Michaelsen <bjoern.michaelsen@canonical.com> | 2017-03-26 00:42:31 +0100 |
|---|---|---|
| committer | Björn Michaelsen <bjoern.michaelsen@canonical.com> | 2017-03-28 19:59:51 +0000 |
| commit | 539470ebd5c47a7876ccc19df6110acacd477bb8 (patch) | |
| tree | c056b2de1b815bc2aecbe30e1deb69e3d2943bf6 /sc/CppunitTest_sc_chart_regression_test.mk | |
| parent | 5d87c15ef0935081319afe2400c99df14d9fa258 (diff) | |
it makes no sense for CC to leak resources when its unaware of model
- if the UNO_QUERY_THROW cast throws, this skips all of the disposing
(and thus leaks)
- the whole try-catch-ignore case has been there since early c6b95527
- even in that commit message, no sane justification has been given
- apparently the try-block was Java-style "catch exceptions until it
runs" cargo-cult
- this change is still the more conservative change: probably the whole
try-catch framing should be removed
Change-Id: Ie3a71e48eb056afd8a844e19898dc1555de4297e
Reviewed-on: https://gerrit.libreoffice.org/35690
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Diffstat (limited to 'sc/CppunitTest_sc_chart_regression_test.mk')
0 files changed, 0 insertions, 0 deletions