[Libreoffice-commits] core.git: bridges/source
Noel Grandin (via logerrit)
logerrit at kemper.freedesktop.org
Mon Jul 19 15:52:49 UTC 2021
bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx | 273 ++++++++++------------
1 file changed, 134 insertions(+), 139 deletions(-)
New commits:
commit bda0ea24f83ab5c8a6215f594d99ea680af55e5b
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Mon Jul 19 10:56:03 2021 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Mon Jul 19 17:52:15 2021 +0200
flatten RTTI code
Change-Id: If56de2708254348b6b759ecdd2b1fb259562af69
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119169
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx b/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx
index 839c25755098..30201a1739e3 100644
--- a/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx
+++ b/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx
@@ -117,163 +117,158 @@ std::type_info * RTTI::getRTTI(typelib_TypeDescription const & pTypeDescr)
std::type_info * RTTI::getRTTI_NoLock(typelib_TypeDescription const & pTypeDescr)
{
- std::type_info * rtti;
-
OUString const & unoName = OUString::unacquired(&pTypeDescr.pTypeName);
t_rtti_map::const_iterator iFind( m_rttis.find( unoName ) );
- if (iFind == m_rttis.end())
+ if (iFind != m_rttis.end())
+ return iFind->second;
+
+ std::type_info * rtti;
+
+ // RTTI symbol
+ OStringBuffer buf( 64 );
+ buf.append( "_ZTIN" );
+ sal_Int32 index = 0;
+ do
{
- // RTTI symbol
- OStringBuffer buf( 64 );
- buf.append( "_ZTIN" );
- sal_Int32 index = 0;
- do
- {
- OUString token( unoName.getToken( 0, '.', index ) );
- buf.append( token.getLength() );
- OString c_token( OUStringToOString( token, RTL_TEXTENCODING_ASCII_US ) );
- buf.append( c_token );
- }
- while (index >= 0);
- buf.append( 'E' );
+ OUString token( unoName.getToken( 0, '.', index ) );
+ buf.append( token.getLength() );
+ OString c_token( OUStringToOString( token, RTL_TEXTENCODING_ASCII_US ) );
+ buf.append( c_token );
+ }
+ while (index >= 0);
+ buf.append( 'E' );
- OString symName( buf.makeStringAndClear() );
+ OString symName( buf.makeStringAndClear() );
#if !defined ANDROID
- rtti = static_cast<std::type_info *>(dlsym( m_hApp, symName.getStr() ));
+ rtti = static_cast<std::type_info *>(dlsym( m_hApp, symName.getStr() ));
#else
- rtti = static_cast<std::type_info *>(dlsym( RTLD_DEFAULT, symName.getStr() ));
+ rtti = static_cast<std::type_info *>(dlsym( RTLD_DEFAULT, symName.getStr() ));
#endif
- if (rtti)
+ if (rtti)
+ {
+ std::pair< t_rtti_map::iterator, bool > insertion (
+ m_rttis.insert( t_rtti_map::value_type( unoName, rtti ) ) );
+ SAL_WARN_IF( !insertion.second, "bridges", "key " << unoName << " already in rtti map" );
+ return rtti;
+ }
+
+ // try to lookup the symbol in the generated rtti map
+ auto iFind2( m_generatedRttis.find( unoName ) );
+ if (iFind2 != m_generatedRttis.end())
+ {
+ // taking already generated rtti
+ rtti = iFind2->second->get();
+ return rtti;
+ }
+
+ // we must generate it !
+ // symbol and rtti-name is nearly identical,
+ // the symbol is prefixed with _ZTI
+ char const * rttiName = symName.getStr() +4;
+#if OSL_DEBUG_LEVEL > 1
+ fprintf( stderr,"generated rtti for %s\n", rttiName );
+#endif
+ std::unique_ptr<Generated> newRtti;
+ switch (pTypeDescr.eTypeClass) {
+ case typelib_TypeClass_EXCEPTION:
{
- std::pair< t_rtti_map::iterator, bool > insertion (
- m_rttis.insert( t_rtti_map::value_type( unoName, rtti ) ) );
- SAL_WARN_IF( !insertion.second, "bridges", "key " << unoName << " already in rtti map" );
+ typelib_CompoundTypeDescription const & ctd
+ = reinterpret_cast<
+ typelib_CompoundTypeDescription const &>(
+ pTypeDescr);
+ if (ctd.pBaseTypeDescription)
+ {
+ // ensure availability of base
+ std::type_info * base_rtti = getRTTI_NoLock(
+ ctd.pBaseTypeDescription->aBase);
+ m_rttiNames.emplace_back(OString(rttiName));
+ std::unique_ptr<std::type_info> info(
+ new __cxxabiv1::__si_class_type_info(
+ m_rttiNames.back().getStr(), static_cast<__cxxabiv1::__class_type_info *>(base_rtti) ));
+ newRtti.reset(new GeneratedPlain(std::move(info)));
+ }
+ else
+ {
+ // this class has no base class
+ m_rttiNames.emplace_back(OString(rttiName));
+ std::unique_ptr<std::type_info> info(
+ new __cxxabiv1::__class_type_info(m_rttiNames.back().getStr()));
+ newRtti.reset(new GeneratedPlain(std::move(info)));
+ }
+ break;
}
- else
+ case typelib_TypeClass_INTERFACE:
{
- // try to lookup the symbol in the generated rtti map
- auto iFind2( m_generatedRttis.find( unoName ) );
- if (iFind2 == m_generatedRttis.end())
- {
- // we must generate it !
- // symbol and rtti-name is nearly identical,
- // the symbol is prefixed with _ZTI
- char const * rttiName = symName.getStr() +4;
-#if OSL_DEBUG_LEVEL > 1
- fprintf( stderr,"generated rtti for %s\n", rttiName );
-#endif
- std::unique_ptr<Generated> newRtti;
- switch (pTypeDescr.eTypeClass) {
- case typelib_TypeClass_EXCEPTION:
- {
- typelib_CompoundTypeDescription const & ctd
- = reinterpret_cast<
- typelib_CompoundTypeDescription const &>(
- pTypeDescr);
- if (ctd.pBaseTypeDescription)
- {
- // ensure availability of base
- std::type_info * base_rtti = getRTTI_NoLock(
- ctd.pBaseTypeDescription->aBase);
- m_rttiNames.emplace_back(OString(rttiName));
- std::unique_ptr<std::type_info> info(
- new __cxxabiv1::__si_class_type_info(
- m_rttiNames.back().getStr(), static_cast<__cxxabiv1::__class_type_info *>(base_rtti) ));
- newRtti.reset(new GeneratedPlain(std::move(info)));
- }
- else
- {
- // this class has no base class
- m_rttiNames.emplace_back(OString(rttiName));
- std::unique_ptr<std::type_info> info(
- new __cxxabiv1::__class_type_info(m_rttiNames.back().getStr()));
- newRtti.reset(new GeneratedPlain(std::move(info)));
- }
- break;
- }
- case typelib_TypeClass_INTERFACE:
+ typelib_InterfaceTypeDescription const & itd
+ = reinterpret_cast<
+ typelib_InterfaceTypeDescription const &>(
+ pTypeDescr);
+ std::vector<std::type_info *> bases;
+ for (sal_Int32 i = 0; i != itd.nBaseTypes; ++i) {
+ bases.push_back(getRTTI_NoLock(itd.ppBaseTypes[i]->aBase));
+ }
+ switch (itd.nBaseTypes) {
+ case 0:
+ {
+ m_rttiNames.emplace_back(OString(rttiName));
+ std::unique_ptr<std::type_info> info(
+ new __cxxabiv1::__class_type_info(
+ m_rttiNames.back().getStr()));
+ newRtti.reset(new GeneratedPlain(std::move(info)));
+ break;
+ }
+ case 1:
+ {
+ m_rttiNames.emplace_back(OString(rttiName));
+ std::unique_ptr<std::type_info> info(
+ new __cxxabiv1::__si_class_type_info(
+ m_rttiNames.back().getStr(),
+ static_cast<
+ __cxxabiv1::__class_type_info *>(
+ bases[0])));
+ newRtti.reset(new GeneratedPlain(std::move(info)));
+ break;
+ }
+ default:
+ {
+ m_rttiNames.emplace_back(OString(rttiName));
+ auto pad = std::make_unique<char[]>(
+ sizeof (__cxxabiv1::__vmi_class_type_info)
+ + ((itd.nBaseTypes - 1)
+ * sizeof (
+ __cxxabiv1::__base_class_type_info)));
+ __cxxabiv1::__vmi_class_type_info * info
+ = new(pad.get())
+ __cxxabiv1::__vmi_class_type_info(
+ m_rttiNames.back().getStr(),
+ __cxxabiv1::__vmi_class_type_info::__flags_unknown_mask);
+ info->__base_count = itd.nBaseTypes;
+ for (sal_Int32 i = 0; i != itd.nBaseTypes; ++i)
{
- typelib_InterfaceTypeDescription const & itd
- = reinterpret_cast<
- typelib_InterfaceTypeDescription const &>(
- pTypeDescr);
- std::vector<std::type_info *> bases;
- for (sal_Int32 i = 0; i != itd.nBaseTypes; ++i) {
- bases.push_back(getRTTI_NoLock(itd.ppBaseTypes[i]->aBase));
- }
- switch (itd.nBaseTypes) {
- case 0:
- {
- m_rttiNames.emplace_back(OString(rttiName));
- std::unique_ptr<std::type_info> info(
- new __cxxabiv1::__class_type_info(
- m_rttiNames.back().getStr()));
- newRtti.reset(new GeneratedPlain(std::move(info)));
- break;
- }
- case 1:
- {
- m_rttiNames.emplace_back(OString(rttiName));
- std::unique_ptr<std::type_info> info(
- new __cxxabiv1::__si_class_type_info(
- m_rttiNames.back().getStr(),
- static_cast<
- __cxxabiv1::__class_type_info *>(
- bases[0])));
- newRtti.reset(new GeneratedPlain(std::move(info)));
- break;
- }
- default:
- {
- m_rttiNames.emplace_back(OString(rttiName));
- auto pad = std::make_unique<char[]>(
- sizeof (__cxxabiv1::__vmi_class_type_info)
- + ((itd.nBaseTypes - 1)
- * sizeof (
- __cxxabiv1::__base_class_type_info)));
- __cxxabiv1::__vmi_class_type_info * info
- = new(pad.get())
- __cxxabiv1::__vmi_class_type_info(
- m_rttiNames.back().getStr(),
- __cxxabiv1::__vmi_class_type_info::__flags_unknown_mask);
- info->__base_count = itd.nBaseTypes;
- for (sal_Int32 i = 0; i != itd.nBaseTypes; ++i)
- {
- info->__base_info[i].__base_type
- = static_cast<
- __cxxabiv1::__class_type_info *>(
- bases[i]);
- info->__base_info[i].__offset_flags
- = (__cxxabiv1::__base_class_type_info::__public_mask
- | ((8 * i) << __cxxabiv1::__base_class_type_info::__offset_shift));
- }
- newRtti.reset(new GeneratedPad(std::move(pad)));
- break;
- }
- }
- break;
+ info->__base_info[i].__base_type
+ = static_cast<
+ __cxxabiv1::__class_type_info *>(
+ bases[i]);
+ info->__base_info[i].__offset_flags
+ = (__cxxabiv1::__base_class_type_info::__public_mask
+ | ((8 * i) << __cxxabiv1::__base_class_type_info::__offset_shift));
}
- default:
- assert(false); // cannot happen
+ newRtti.reset(new GeneratedPad(std::move(pad)));
+ break;
}
- rtti = newRtti->get();
- if (newRtti) {
- auto insertion (
- m_generatedRttis.emplace(unoName, std::move(newRtti)));
- SAL_WARN_IF( !insertion.second, "bridges", "key " << unoName << " already in generated rtti map" );
- }
- }
- else // taking already generated rtti
- {
- rtti = iFind2->second->get();
}
+ break;
}
+ default:
+ assert(false); // cannot happen
}
- else
- {
- rtti = iFind->second;
+ rtti = newRtti->get();
+ if (newRtti) {
+ auto insertion (
+ m_generatedRttis.emplace(unoName, std::move(newRtti)));
+ SAL_WARN_IF( !insertion.second, "bridges", "key " << unoName << " already in generated rtti map" );
}
return rtti;
More information about the Libreoffice-commits
mailing list