[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-5.0' - 2 commits - desktop/source sc/source
Jan Holesovsky
kendy at collabora.com
Fri Nov 13 02:37:51 PST 2015
desktop/source/lib/init.cxx | 10 +++++-----
sc/source/ui/view/gridwin.cxx | 12 +++++-------
2 files changed, 10 insertions(+), 12 deletions(-)
New commits:
commit 7e5f6bbc0811098db7337c0794acce51a5393a0d
Author: Jan Holesovsky <kendy at collabora.com>
Date: Thu Nov 12 18:22:06 2015 +0100
lok: Use reference instead of copy constructing in range-based for.
Change-Id: Ie5bf5d4ab139f22e67f3654b0bb31e10b8c9f337
(cherry picked from commit addd884799b88213df813fd8501c3ab8306593e1)
diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index 1ab5dc8..13312e0 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -1290,10 +1290,10 @@ static char* getStyles(LibreOfficeKitDocument* pThis, const char* pCommand)
if (sStyleFam == "ParagraphStyles"
&& doc_getDocumentType(pThis) == LOK_DOCTYPE_TEXT)
{
- for( OUString aStyle: aWriterStyles )
+ for (const OUString& rStyle: aWriterStyles)
{
uno::Reference< beans::XPropertySet > xStyle;
- xStyleFamily->getByName( aStyle ) >>= xStyle;
+ xStyleFamily->getByName(rStyle) >>= xStyle;
OUString sName;
xStyle->getPropertyValue("DisplayName") >>= sName;
if( !sName.isEmpty() )
@@ -1308,14 +1308,14 @@ static char* getStyles(LibreOfficeKitDocument* pThis, const char* pCommand)
}
uno::Sequence<OUString> aStyles = xStyleFamily->getElementNames();
- for ( OUString aStyle: aStyles )
+ for (const OUString& rStyle: aStyles )
{
// Filter out the default styles - they are already at the top
// of the list
- if (aDefaultStyleNames.find(aStyle) == aDefaultStyleNames.end())
+ if (aDefaultStyleNames.find(rStyle) == aDefaultStyleNames.end())
{
boost::property_tree::ptree aChild;
- aChild.put("", aStyle);
+ aChild.put("", rStyle);
aChildren.push_back(std::make_pair("", aChild));
}
}
commit 374fdbed82293470cd361a0b21d2ba0adcf2a991
Author: Andrzej Hunt <andrzej at ahunt.org>
Date: Fri Nov 13 11:17:43 2015 +0100
sc lok: during tiled rendering the cell-cursor is always visible
Change-Id: Ia802c19f5bfd2fe2e9909e3c611047c529a64200
(cherry picked from commit e77668eb1e7abe522493235dadfca08ca451ad99)
diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index 2531bc7..d884af8 100644
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -6023,7 +6023,9 @@ void ScGridWindow::UpdateCursorOverlay()
const ScPatternAttr* pPattern = pDoc->GetPattern(nX,nY,nTab);
- if (!maVisibleRange.isInside(nX, nY))
+ ScDrawLayer* pDrawLayer = pDoc->GetDrawLayer();
+
+ if (!pDrawLayer->isTiledRendering() && !maVisibleRange.isInside(nX, nY))
{
if (maVisibleRange.mnCol2 < nX || maVisibleRange.mnRow2 < nY)
return; // no further check needed, nothing visible
@@ -6042,13 +6044,11 @@ void ScGridWindow::UpdateCursorOverlay()
}
// don't show the cursor in overlapped cells
-
const ScMergeFlagAttr& rMergeFlag = static_cast<const ScMergeFlagAttr&>( pPattern->GetItem(ATTR_MERGE_FLAG) );
bool bOverlapped = rMergeFlag.IsOverlapped();
// left or above of the screen?
-
- bool bVis = ( nX>=pViewData->GetPosX(eHWhich) && nY>=pViewData->GetPosY(eVWhich) );
+ bool bVis = pDrawLayer->isTiledRendering() || ( nX>=pViewData->GetPosX(eHWhich) && nY>=pViewData->GetPosY(eVWhich) );
if (!bVis)
{
SCCOL nEndX = nX;
@@ -6078,7 +6078,7 @@ void ScGridWindow::UpdateCursorOverlay()
}
// in the tiled rendering case, don't limit to the screen size
- if (bMaybeVisible)
+ if (bMaybeVisible || pDrawLayer->isTiledRendering())
{
long nSizeXPix;
long nSizeYPix;
@@ -6121,8 +6121,6 @@ void ScGridWindow::UpdateCursorOverlay()
}
}
- ScDrawLayer* pDrawLayer = pDoc->GetDrawLayer();
-
if ( !aPixelRects.empty() )
{
if (pDrawLayer->isTiledRendering()) {
More information about the Libreoffice-commits
mailing list