[Libreoffice-commits] core.git: 2 commits - sw/inc sw/source vcl/unx
Caolán McNamara (via logerrit)
logerrit at kemper.freedesktop.org
Mon Oct 5 10:46:58 UTC 2020
sw/inc/flddat.hxx | 2 +-
sw/source/core/fields/docufld.cxx | 4 ++--
sw/source/core/fields/flddat.cxx | 10 +++++-----
sw/source/core/unocore/unocoll.cxx | 16 ++++++++--------
sw/source/core/unocore/unoidx.cxx | 12 ++++++------
vcl/unx/gtk3/gtk3gtksys.cxx | 37 +------------------------------------
6 files changed, 23 insertions(+), 58 deletions(-)
New commits:
commit 73b1188a3af5edce43508965f95fab1345ccda0e
Author: Caolán McNamara <caolanm at redhat.com>
AuthorDate: Sun Oct 4 20:57:30 2020 +0100
Commit: Caolán McNamara <caolanm at redhat.com>
CommitDate: Mon Oct 5 12:40:35 2020 +0200
we are guaranteed to have gdk_screen_get_primary_monitor now
Change-Id: I03419ff5233da53aeb3644f62357c983b0fe76ea
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103930
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
diff --git a/vcl/unx/gtk3/gtk3gtksys.cxx b/vcl/unx/gtk3/gtk3gtksys.cxx
index 379c87343ae8..a0627a6f81e2 100644
--- a/vcl/unx/gtk3/gtk3gtksys.cxx
+++ b/vcl/unx/gtk3/gtk3gtksys.cxx
@@ -186,46 +186,11 @@ bool GtkSalSystem::IsUnifiedDisplay()
return gdk_display_get_n_screens (mpDisplay) == 1;
}
-namespace {
-int _fallback_get_primary_monitor (GdkScreen *pScreen)
-{
- // Use monitor name as primacy heuristic
- int max = gdk_screen_get_n_monitors (pScreen);
- for (int i = 0; i < max; ++i)
- {
- char *name = gdk_screen_get_monitor_plug_name (pScreen, i);
- bool bLaptop = (name && !g_ascii_strncasecmp (name, "LVDS", 4));
- g_free (name);
- if (bLaptop)
- return i;
- }
- return 0;
-}
-
-int _get_primary_monitor (GdkScreen *pScreen)
-{
- static int (*get_fn) (GdkScreen *) = nullptr;
- get_fn = gdk_screen_get_primary_monitor;
- // Perhaps we have a newer gtk+ with this symbol:
- if (!get_fn)
- {
- get_fn = reinterpret_cast<int(*)(GdkScreen*)>(osl_getAsciiFunctionSymbol(nullptr,
- "gdk_screen_get_primary_monitor"));
- }
- if (!get_fn)
- get_fn = _fallback_get_primary_monitor;
- if (get_fn)
- return get_fn (pScreen);
- else
- return 0;
-}
-} // end anonymous namespace
-
unsigned int GtkSalSystem::GetDisplayBuiltInScreen()
{
GdkScreen *pDefault = gdk_display_get_default_screen (mpDisplay);
int idx = getScreenIdxFromPtr (pDefault);
- return idx + _get_primary_monitor (pDefault);
+ return idx + gdk_screen_get_primary_monitor(pDefault);
}
tools::Rectangle GtkSalSystem::GetDisplayScreenPosSizePixel (unsigned int nScreen)
commit 1d35f90dfe46cadc913e35f509669785cfa8ec18
Author: Caolán McNamara <caolanm at redhat.com>
AuthorDate: Sun Oct 4 19:58:51 2020 +0100
Commit: Caolán McNamara <caolanm at redhat.com>
CommitDate: Mon Oct 5 12:40:13 2020 +0200
SwDateTimeField::GetDateTime always dereferences its SwDoc*
ditto:
SwXFrameEnumeration ctor
lcl_ReAssignTOXType
Change-Id: Idaa6bef29cbc4c3a08d578ef4bedcaa071547944
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/103928
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
diff --git a/sw/inc/flddat.hxx b/sw/inc/flddat.hxx
index 66f13c98bf51..38c16c9039a3 100644
--- a/sw/inc/flddat.hxx
+++ b/sw/inc/flddat.hxx
@@ -68,7 +68,7 @@ public:
Date GetDate() const;
tools::Time GetTime() const;
void SetDateTime(const DateTime& rDT);
- static double GetDateTime(SwDoc* pDoc, const DateTime& rDT);
+ static double GetDateTime(SwDoc& rDoc, const DateTime& rDT);
virtual bool QueryValue( css::uno::Any& rVal, sal_uInt16 nMId ) const override;
virtual bool PutValue( const css::uno::Any& rVal, sal_uInt16 nMId ) override;
diff --git a/sw/source/core/fields/docufld.cxx b/sw/source/core/fields/docufld.cxx
index cca0e4ee8c7f..196dcba31d3b 100644
--- a/sw/source/core/fields/docufld.cxx
+++ b/sw/source/core/fields/docufld.cxx
@@ -952,7 +952,7 @@ OUString SwDocInfoFieldType::Expand( sal_uInt16 nSub, sal_uInt32 nFormat,
else
{
// start the number formatter
- double fVal = SwDateTimeField::GetDateTime( GetDoc(),
+ double fVal = SwDateTimeField::GetDateTime( *GetDoc(),
aDate);
aStr = ExpandValue(fVal, nFormat, nLang);
}
@@ -968,7 +968,7 @@ OUString SwDocInfoFieldType::Expand( sal_uInt16 nSub, sal_uInt32 nFormat,
else
{
// start the number formatter
- double fVal = SwDateTimeField::GetDateTime( GetDoc(),
+ double fVal = SwDateTimeField::GetDateTime( *GetDoc(),
aDate);
aStr = ExpandValue(fVal, nFormat, nLang);
}
diff --git a/sw/source/core/fields/flddat.cxx b/sw/source/core/fields/flddat.cxx
index f2a6a779a0ed..d68fe77beabe 100644
--- a/sw/source/core/fields/flddat.cxx
+++ b/sw/source/core/fields/flddat.cxx
@@ -67,7 +67,7 @@ OUString SwDateTimeField::ExpandImpl(SwRootFrame const*const) const
if (!(IsFixed()))
{
DateTime aDateTime( DateTime::SYSTEM );
- fVal = GetDateTime(GetDoc(), aDateTime);
+ fVal = GetDateTime(*GetDoc(), aDateTime);
}
else
fVal = GetValue();
@@ -115,12 +115,12 @@ OUString SwDateTimeField::GetPar2() const
void SwDateTimeField::SetDateTime(const DateTime& rDT)
{
- SetValue(GetDateTime(GetDoc(), rDT));
+ SetValue(GetDateTime(*GetDoc(), rDT));
}
-double SwDateTimeField::GetDateTime(SwDoc* pDoc, const DateTime& rDT)
+double SwDateTimeField::GetDateTime(SwDoc& rDoc, const DateTime& rDT)
{
- SvNumberFormatter* pFormatter = pDoc->GetNumberFormatter();
+ SvNumberFormatter* pFormatter = rDoc.GetNumberFormatter();
const Date& rNullDate = pFormatter->GetNullDate();
double fResult = rDT - DateTime(rNullDate);
@@ -133,7 +133,7 @@ double SwDateTimeField::GetValue() const
if (IsFixed())
return SwValueField::GetValue();
else
- return GetDateTime(GetDoc(), DateTime( DateTime::SYSTEM ));
+ return GetDateTime(*GetDoc(), DateTime( DateTime::SYSTEM ));
}
Date SwDateTimeField::GetDate() const
diff --git a/sw/source/core/unocore/unocoll.cxx b/sw/source/core/unocore/unocoll.cxx
index c9a2cdc0dff4..a6287cbd714d 100644
--- a/sw/source/core/unocore/unocoll.cxx
+++ b/sw/source/core/unocore/unocoll.cxx
@@ -1056,7 +1056,7 @@ namespace
protected:
virtual ~SwXFrameEnumeration() override {};
public:
- SwXFrameEnumeration(const SwDoc* const pDoc);
+ SwXFrameEnumeration(const SwDoc& rDoc);
//XEnumeration
virtual sal_Bool SAL_CALL hasMoreElements() override;
@@ -1070,12 +1070,12 @@ namespace
}
template<FlyCntType T>
-SwXFrameEnumeration<T>::SwXFrameEnumeration(const SwDoc* const pDoc)
+SwXFrameEnumeration<T>::SwXFrameEnumeration(const SwDoc& rDoc)
: m_aFrames()
{
SolarMutexGuard aGuard;
- const SwFrameFormats* const pFormats = pDoc->GetSpzFrameFormats();
- if(pFormats->empty())
+ const SwFrameFormats* const pFormats = rDoc.GetSpzFrameFormats();
+ if (pFormats->empty())
return;
// #i104937#
const size_t nSize = pFormats->size();
@@ -1090,7 +1090,7 @@ SwXFrameEnumeration<T>::SwXFrameEnumeration(const SwDoc* const pDoc)
const SwNodeIndex* pIdx = pFormat->GetContent().GetContentIdx();
if(!pIdx || !pIdx->GetNodes().IsDocNodes())
continue;
- const SwNode* pNd = pDoc->GetNodes()[ pIdx->GetIndex() + 1 ];
+ const SwNode* pNd = rDoc.GetNodes()[ pIdx->GetIndex() + 1 ];
if(UnoFrameWrap_traits<T>::filter(pNd))
m_aFrames.push_back(lcl_UnoWrapFrame<T>(pFormat));
}
@@ -1165,13 +1165,13 @@ uno::Reference<container::XEnumeration> SwXFrames::createEnumeration()
{
case FLYCNTTYPE_FRM:
return uno::Reference< container::XEnumeration >(
- new SwXFrameEnumeration<FLYCNTTYPE_FRM>(GetDoc()));
+ new SwXFrameEnumeration<FLYCNTTYPE_FRM>(*GetDoc()));
case FLYCNTTYPE_GRF:
return uno::Reference< container::XEnumeration >(
- new SwXFrameEnumeration<FLYCNTTYPE_GRF>(GetDoc()));
+ new SwXFrameEnumeration<FLYCNTTYPE_GRF>(*GetDoc()));
case FLYCNTTYPE_OLE:
return uno::Reference< container::XEnumeration >(
- new SwXFrameEnumeration<FLYCNTTYPE_OLE>(GetDoc()));
+ new SwXFrameEnumeration<FLYCNTTYPE_OLE>(*GetDoc()));
default:
throw uno::RuntimeException();
}
diff --git a/sw/source/core/unocore/unoidx.cxx b/sw/source/core/unocore/unoidx.cxx
index 9cfe87f04901..442527ef454b 100644
--- a/sw/source/core/unocore/unoidx.cxx
+++ b/sw/source/core/unocore/unoidx.cxx
@@ -98,13 +98,13 @@ static void lcl_BitMaskToAny(uno::Any & o_rValue,
}
static void
-lcl_ReAssignTOXType(SwDoc* pDoc, SwTOXBase& rTOXBase, const OUString& rNewName)
+lcl_ReAssignTOXType(SwDoc& rDoc, SwTOXBase& rTOXBase, const OUString& rNewName)
{
- const sal_uInt16 nUserCount = pDoc->GetTOXTypeCount( TOX_USER );
+ const sal_uInt16 nUserCount = rDoc.GetTOXTypeCount( TOX_USER );
const SwTOXType* pNewType = nullptr;
for(sal_uInt16 nUser = 0; nUser < nUserCount; nUser++)
{
- const SwTOXType* pType = pDoc->GetTOXType( TOX_USER, nUser );
+ const SwTOXType* pType = rDoc.GetTOXType( TOX_USER, nUser );
if (pType->GetTypeName()==rNewName)
{
pNewType = pType;
@@ -114,7 +114,7 @@ lcl_ReAssignTOXType(SwDoc* pDoc, SwTOXBase& rTOXBase, const OUString& rNewName)
if(!pNewType)
{
SwTOXType aNewType(TOX_USER, rNewName);
- pNewType = pDoc->InsertTOXType( aNewType );
+ pNewType = rDoc.InsertTOXType( aNewType );
}
rTOXBase.RegisterToTOXType( *const_cast<SwTOXType*>(pNewType) );
@@ -594,7 +594,7 @@ SwXDocumentIndex::setPropertyValue(
{
if (rTOXBase.GetTOXType()->GetTypeName() != sNewName)
{
- lcl_ReAssignTOXType(pSectionFormat->GetDoc(),
+ lcl_ReAssignTOXType(*pSectionFormat->GetDoc(),
rTOXBase, sNewName);
}
}
@@ -1332,7 +1332,7 @@ SwXDocumentIndex::attach(const uno::Reference< text::XTextRange > & xTextRange)
if ((TOX_USER == pTOXType->GetType()) &&
m_pImpl->m_pProps->GetTypeName() != pTOXType->GetTypeName())
{
- lcl_ReAssignTOXType(pDoc, rTOXBase, m_pImpl->m_pProps->GetTypeName());
+ lcl_ReAssignTOXType(*pDoc, rTOXBase, m_pImpl->m_pProps->GetTypeName());
}
//TODO: apply Section attributes (columns and background)
SwTOXBaseSection *const pTOX =
More information about the Libreoffice-commits
mailing list