[Libreoffice-commits] core.git: include/comphelper

Noel Grandin (via logerrit) logerrit at kemper.freedesktop.org
Sat Aug 7 07:24:10 UTC 2021


 include/comphelper/interfacecontainer2.hxx |   12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

New commits:
commit e54c167de6a15adc61dea8a1ac3a31e202573afa
Author:     Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Fri Aug 6 15:40:23 2021 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Sat Aug 7 09:23:35 2021 +0200

    address some review comments
    
    from
       https://gerrit.libreoffice.org/c/core/+/21643/6/
    
    Change-Id: I6df2de42e99b30b5a9c9d098b0873d2390fe5e8b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120118
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/include/comphelper/interfacecontainer2.hxx b/include/comphelper/interfacecontainer2.hxx
index 626ef830211d..e001f56e6245 100644
--- a/include/comphelper/interfacecontainer2.hxx
+++ b/include/comphelper/interfacecontainer2.hxx
@@ -53,11 +53,11 @@ namespace detail {
 
 class OInterfaceContainerHelper2;
 /**
-  This is the iterator of an InterfaceContainerHelper. Typically
+  This is the iterator of an OInterfaceContainerHelper2. Typically
   one constructs an instance on the stack for one firing session.
   It is not allowed to assign or copy an instance of this class.
 
-  @see OInterfaceContainerHelper
+  @see OInterfaceContainerHelper2
  */
 class COMPHELPER_DLLPUBLIC OInterfaceIteratorHelper2
 {
@@ -111,9 +111,13 @@ private:
 
 /**
   A container of interfaces. To access the elements use an iterator.
-  This implementation is thread save.
+  This implementation is thread-safe.
 
-  @see OInterfaceIteratorHelper
+  This is a copy of the code at include/cppuhelper/interfacecontainer.h,
+  Except that it uses a std::vector instead of a Sequence for the mutable listener
+  list, which provides far better performance.
+
+  @see OInterfaceIteratorHelper2
  */
 class COMPHELPER_DLLPUBLIC OInterfaceContainerHelper2
 {


More information about the Libreoffice-commits mailing list