git grep m_bPreferrFirstInterceptor
seems to show a global static variable that is always true. It'd be great to cut that out along with the code paths where it would have been false - apparently they're not used =)
sanity-check appreciated =)
*** Bug 96950 has been marked as a duplicate of this bug. ***
Also would be good to kill:
and conditionally include those ifdefs.
(In reply to Michael Meeks from comment #1)
> sanity-check appreciated =)
yes, looks like m_bPreferrFirstInterceptor
was dead-on-arrival ever since its introduction in 648106f879fdf6609db1e0d4c8acb43b3a997833, f12bd2305aaadb6a5ccb8249c80b7865e4613005 "INTEGRATION: CWS fwk01: #107642# change order of used interception objects," and FORCE_DESTRUCTION_OF_INTERCEPTION_CHAIN was dead-on-arrival ever since its introduction in 96dcdde3aa9ada7f8edfd11f4ad5cd47f22c8436 "INTEGRATION: CWS mbapp3: #i64598# destruct interception chain explicitly"
shubhamtibra committed a patch related to this issue.
It has been pushed to "master":
tdf#96949 Remove global static variable m_bPreferrFirstInterceptor and unused code
It will be available in 5.2.0.
The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
Affected users are encouraged to test the fix and report feedback.
Readded whiteboard keyword "target:5.2.0"
JanI is default CC for Easy Hacks (Add Jan; remove LibreOffice Dev List from CC)