codemaker/source/cppumaker/cpputype.cxx |   52 +++++++++++++++++---------------
 1 file changed, 28 insertions(+), 24 deletions(-)

New commits:
commit 911486c4624bdf45f3fb13ef705017475c6554b9
Author: Stephan Bergmann <sberg...@redhat.com>
Date:   Tue Oct 1 14:33:56 2013 +0200

    rhbz#1014010: Missing dependencies in isBootstrapType list
    
    ...the list has been fixed now by copying its elements into an ENTRIES file 
and
    running "unoidl-write udkapi/ @ENTITIES TEMP && unoidl-read TEMP 
>/dev/null" and
    adding any reported unknown entities until it succeeds.
    
    However, the updated list lead to deadlock when css.reflection.ParamInfo 
UnoType
    resolves css.reflection.XIdlClass UnoType resolves css.reflection.XIdlMethod
    UnoType resolves css.reflection.ParamInfo UnoType, so broke the circle by no
    longer resolving the interface methods' return and parameter types in
    InterfaceType::dumpMethodsCppuDecl (which is why those type infos are only
    generated on demand anyway; looks like this had been a careless thinko in 
the
    generation of comprehensive type info that had remained unnoticed all the 
time).
    
    (cherry picked from commit 254f59f623f58c320175a06a2c93bcee7868b623)
    Conflicts:
        codemaker/source/cppumaker/cpputype.cxx
    
    Change-Id: I50ef2fde16242298e055c6fa5971e70fad1a2b68
    Reviewed-on: https://gerrit.libreoffice.org/6106
    Reviewed-by: Caolán McNamara <caol...@redhat.com>
    Tested-by: Caolán McNamara <caol...@redhat.com>
    (cherry picked from commit 395b860fe19b786a096a2533a09d0c45c11ed8b4)
    Reviewed-on: https://gerrit.libreoffice.org/6221
    Reviewed-by: Michael Stahl <mst...@redhat.com>
    Tested-by: Miklos Vajna <vmik...@collabora.co.uk>
    Reviewed-by: Miklos Vajna <vmik...@collabora.co.uk>

diff --git a/codemaker/source/cppumaker/cpputype.cxx 
b/codemaker/source/cppumaker/cpputype.cxx
index 09de49c..eb6a92b 100644
--- a/codemaker/source/cppumaker/cpputype.cxx
+++ b/codemaker/source/cppumaker/cpputype.cxx
@@ -72,17 +72,25 @@ rtl::OString translateSimpleUnoType(rtl::OString const & 
unoType, bool cppuUnoTy
 
 bool isBootstrapType(rtl::OString const & name) {
     static char const * const names[] = {
+        "com/sun/star/beans/Property",
         "com/sun/star/beans/PropertyAttribute",
+        "com/sun/star/beans/PropertyChangeEvent",
+        "com/sun/star/beans/PropertyState",
         "com/sun/star/beans/PropertyValue",
         "com/sun/star/beans/XFastPropertySet",
         "com/sun/star/beans/XMultiPropertySet",
+        "com/sun/star/beans/XPropertiesChangeListener",
         "com/sun/star/beans/XPropertyAccess",
+        "com/sun/star/beans/XPropertyChangeListener",
         "com/sun/star/beans/XPropertySet",
+        "com/sun/star/beans/XPropertySetInfo",
         "com/sun/star/beans/XPropertySetOption",
+        "com/sun/star/beans/XVetoableChangeListener",
         "com/sun/star/bridge/UnoUrlResolver",
         "com/sun/star/bridge/XUnoUrlResolver",
         "com/sun/star/connection/SocketPermission",
         "com/sun/star/container/XElementAccess",
+        "com/sun/star/container/XEnumeration",
         "com/sun/star/container/XEnumerationAccess",
         "com/sun/star/container/XHierarchicalNameAccess",
         "com/sun/star/container/XNameAccess",
@@ -92,6 +100,7 @@ bool isBootstrapType(rtl::OString const & name) {
         "com/sun/star/io/FilePermission",
         "com/sun/star/io/IOException",
         "com/sun/star/lang/DisposedException",
+        "com/sun/star/lang/EventObject",
         "com/sun/star/lang/WrappedTargetRuntimeException",
         "com/sun/star/lang/XComponent",
         "com/sun/star/lang/XEventListener",
@@ -103,11 +112,19 @@ bool isBootstrapType(rtl::OString const & name) {
         "com/sun/star/lang/XSingleServiceFactory",
         "com/sun/star/lang/XTypeProvider",
         "com/sun/star/loader/XImplementationLoader",
+        "com/sun/star/reflection/FieldAccessMode",
+        "com/sun/star/reflection/MethodMode",
+        "com/sun/star/reflection/ParamInfo",
+        "com/sun/star/reflection/ParamMode",
+        "com/sun/star/reflection/TypeDescriptionSearchDepth",
         "com/sun/star/reflection/XArrayTypeDescription",
         "com/sun/star/reflection/XCompoundTypeDescription",
         "com/sun/star/reflection/XEnumTypeDescription",
+        "com/sun/star/reflection/XIdlArray",
         "com/sun/star/reflection/XIdlClass",
+        "com/sun/star/reflection/XIdlField",
         "com/sun/star/reflection/XIdlField2",
+        "com/sun/star/reflection/XIdlMethod",
         "com/sun/star/reflection/XIdlReflection",
         "com/sun/star/reflection/XIndirectTypeDescription",
         "com/sun/star/reflection/XInterfaceAttributeTypeDescription",
@@ -119,18 +136,28 @@ bool isBootstrapType(rtl::OString const & name) {
         "com/sun/star/reflection/XMethodParameter",
         "com/sun/star/reflection/XStructTypeDescription",
         "com/sun/star/reflection/XTypeDescription",
+        "com/sun/star/reflection/XTypeDescriptionEnumeration",
         "com/sun/star/reflection/XTypeDescriptionEnumerationAccess",
         "com/sun/star/reflection/XUnionTypeDescription",
+        "com/sun/star/registry/RegistryKeyType",
+        "com/sun/star/registry/RegistryValueType",
         "com/sun/star/registry/XImplementationRegistration",
         "com/sun/star/registry/XRegistryKey",
         "com/sun/star/registry/XSimpleRegistry",
         "com/sun/star/security/RuntimePermission",
+        "com/sun/star/security/XAccessControlContext",
         "com/sun/star/security/XAccessController",
+        "com/sun/star/security/XAction",
         "com/sun/star/uno/DeploymentException",
         "com/sun/star/uno/RuntimeException",
+        "com/sun/star/uno/TypeClass",
+        "com/sun/star/uno/Uik",
+        "com/sun/star/uno/XAdapter",
         "com/sun/star/uno/XAggregation",
         "com/sun/star/uno/XComponentContext",
         "com/sun/star/uno/XCurrentContext",
+        "com/sun/star/uno/XInterface",
+        "com/sun/star/uno/XReference",
         "com/sun/star/uno/XUnloadingPreference",
         "com/sun/star/uno/XWeak",
         "com/sun/star/util/XMacroExpander"
@@ -2172,36 +2199,13 @@ void InterfaceType::dumpAttributesCppuDecl(FileStream& 
o, StringSet* pFinishedTy
 void InterfaceType::dumpMethodsCppuDecl(FileStream& o, StringSet* 
pFinishedTypes, CppuTypeDecl eDeclFlag)
 {
     sal_uInt16      methodCount = m_reader.getMethodCount();
-    OString         returnType, paramType, excType;
-    sal_uInt16      paramCount = 0;
+    OString         excType;
     sal_uInt16      excCount = 0;
 
     for (sal_uInt16 i=0; i < methodCount; i++)
     {
-        returnType = rtl::OUStringToOString(
-            m_reader.getMethodReturnTypeName(i), RTL_TEXTENCODING_UTF8);
-        paramCount = m_reader.getMethodParameterCount(i);
         excCount = m_reader.getMethodExceptionCount(i);
-
-        if (pFinishedTypes->count(returnType) == 0)
-        {
-            pFinishedTypes->insert(returnType);
-            dumpCppuGetType(o, returnType, sal_True, eDeclFlag);
-        }
         sal_uInt16 j;
-        for (j=0; j < paramCount; j++)
-        {
-            paramType = rtl::OUStringToOString(
-                m_reader.getMethodParameterTypeName(i, j),
-                RTL_TEXTENCODING_UTF8);
-
-            if (pFinishedTypes->count(paramType) == 0)
-            {
-                pFinishedTypes->insert(paramType);
-                dumpCppuGetType(o, paramType, sal_True, eDeclFlag);
-            }
-        }
-
         for (j=0; j < excCount; j++)
         {
             excType = rtl::OUStringToOString(
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to