summaryrefslogtreecommitdiff
path: root/vbahelper
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2020-04-14 08:35:46 +0200
committerStephan Bergmann <sbergman@redhat.com>2020-04-14 10:39:21 +0200
commit94a7ceae287a7967e8f013d012673e26637c6bb5 (patch)
tree142efe73a90ae4a5664ca09305be492e70087688 /vbahelper
parent35fc5ef0a759884b24ed8b83cd05702a0fab64cc (diff)
basctl functions cannot be missing here
This is a follow-up to 2d64651e889e55f63c90ded24e63e17eaf52102f "Fix loading of basctl with --enable-mergelibs", removing checks that can never be satisfied. (There is no plausible scenario where basctl would legitimately not be installed. All uses of sfx2::getBasctlFunction are conditional on HAVE_FEATURE_SCRIPTING, and basctl is included in section ooo in Repository.mk together with sfx, so the latter cannot be installed without the former being installed. That is also true for Linux distros like e.g. Fedora, where both libraries are included in the core package.) Two of the redundant checks had been added with c6fc2647c8df5b9a7c41a192f2b78812d1f3656d "coverity#1132691 Dereference null return value" and 5fb8f03be61add8320045471edc55ce3f7c4fa8d "coverity#1224981 Dereference null return value", presumably in a misguided attempt to silence false Coverity warnings. The third check had been added with 0b21b8b146fc4b982c7c9bbb866b9ff18a29332a "initial commit for vba blob ( not including container_control stuff )", for unclear reasons. Change-Id: I4a6137edfe4b51d297b115990e16d910325694af Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92143 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'vbahelper')
0 files changed, 0 insertions, 0 deletions