Age | Commit message (Collapse) | Author |
|
which is (a) a little tricky and (b) needs to be on the base class to
get the locking just right
Change-Id: I425d4747e7e0ca0741fded9a6a70f6fa9995e102
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/174312
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
in the same way I added WeakComponentImplHelperBase
Change-Id: I26d93004c3ddf716063cfb2ded1bbda1b76c2a98
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165449
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I0cf6f675483bddf82e7347b484a874c71963bfd7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156984
Tested-by: Jenkins
Reviewed-by: Gabor Kelemen <kelemeng@ubuntu.com>
|
|
Which means creating a variant of the cppuhelper::OPropertySetHelper
helper class which uses a std::mutex.
Since we can do virtual base classes now (which the original could not), use that and a new helper
class comphelper::UnoImplBase to share
std::mutex m_aMutex;
bool m_bDisposing;
fields that the OPropertySetHelper wants to share with the parent class.
Change-Id: I05ad465a3d3653ed4e109137e3e1c58190da8d97
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148474
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ic73a84eb793d45e1041698e45e2c844a1e2f2f66
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/147415
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: Ie3df092b013f10b36c85028fe3b9966d58b22005
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/147264
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Which was not hanling queries for sub-interfaces.
We need to copy the complicated query function from cppuhelper,
unfortunatly. It has a signature that includes the base class, so we
cant re-use it.
Change-Id: I3668abb67c6cb669086ce32505b383ac11d3c3ab
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127511
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
This reverts commit c1f21a6ce6491ac82a022502c618e2758d3b3a31.
Reason for revert: a better fix is implemented in commit
f0c0c62c10e4ffdb9ce44202e2540d2f39a9fcb5
author Noel Grandin <noel.grandin@collabora.co.uk>
date Sat Dec 25 11:43:11 2021 +0200
uninitialised field in comphelper::WeakComponentImplHelperBase
That change allows to keep the ctor implicitly defined and inline.
Change-Id: Icb8064ebc77d4120d279d30df5de4ebee5479b4b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127474
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
|
|
so that subclasses can drop the lock while calling listeners
Change-Id: I6105438110af08f10b0ed50057b28172c3442b95
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127434
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
Change-Id: I7ab702c0dd61231b334aac647e87233095cc69e0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127450
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|
|
From the Jenkins_Linux_Ubsan TB:
/comphelper/source/misc/compbase.cxx:20:9: runtime error: load \
of value 190, which is not a valid value for type ´bool´
... in comphelper::WeakComponentImplHelperBase::dispose() \
/comphelper/source/misc/compbase.cxx:20:9
Maybe also the problem breaking Java for the Jenkins_Linux_Dbg
since the comphelper::WeakComponentImplHelperBase patches
went in.
Regression from commit 252dd254b5b29298457b889623783152e9bed534
("new comphelper::WeakImplComponentHelper").
Change-Id: Idebf6dfcc051f8fcb4cde1fb66d40b38e6f9dddc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127449
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
|
|
to replace the cppu:: equivalent with a lightweight version
that uses std::mutex instead of osl::Mutex
Change-Id: I1b7873a0c2d9cda21f529e43a4ac2fa7574c91a5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127335
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
|