diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-12 17:12:22 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-10-15 19:22:21 +0200 |
commit | eaf0c263eb1a72a58d2a67cc0506ab022d7c4be4 (patch) | |
tree | 4c732e95b560235e83c6de4b5b96260b638fa88d /vcl/source | |
parent | 921ae49cd7e332d7e1ad702efe2198b2780cc829 (diff) |
loplugin:staticconstfield improvements
And fix
ScXMLCachedRowAttrAccess::Cache
which was never setting its mnTab field, and hence would never
be hit.
And fix oox::xls::CellBlockBuffer, which was never setting mnCurrRow.
Change-Id: I2c46aa050b9ebe3c2dc2e52579555f97945dd61c
Reviewed-on: https://gerrit.libreoffice.org/61772
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'vcl/source')
-rw-r--r-- | vcl/source/filter/ipdf/pdfdocument.cxx | 7 | ||||
-rw-r--r-- | vcl/source/gdi/octree.cxx | 13 |
2 files changed, 9 insertions, 11 deletions
diff --git a/vcl/source/filter/ipdf/pdfdocument.cxx b/vcl/source/filter/ipdf/pdfdocument.cxx index 09cafdea9f45..092df7cfbee7 100644 --- a/vcl/source/filter/ipdf/pdfdocument.cxx +++ b/vcl/source/filter/ipdf/pdfdocument.cxx @@ -2274,8 +2274,9 @@ size_t PDFDictionaryElement::Parse(const std::vector<std::unique_ptr<PDFElement> if (pThisDictionary) { pThisDictionary->SetKeyOffset(aName, nNameOffset); - pThisDictionary->SetKeyValueLength( - aName, pName->GetLocation() + pName->GetLength() - nNameOffset); + pThisDictionary->SetKeyValueLength(aName, pName->GetLocation() + + PDFNameElement::GetLength() + - nNameOffset); } aName.clear(); } @@ -2927,8 +2928,6 @@ const OString& PDFNameElement::GetValue() const { return m_aValue; } sal_uInt64 PDFNameElement::GetLocation() const { return m_nLocation; } -sal_uInt64 PDFNameElement::GetLength() const { return m_nLength; } - PDFStreamElement::PDFStreamElement(size_t nLength) : m_nLength(nLength) , m_nOffset(0) diff --git a/vcl/source/gdi/octree.cxx b/vcl/source/gdi/octree.cxx index 21af8416fee9..4635067901f8 100644 --- a/vcl/source/gdi/octree.cxx +++ b/vcl/source/gdi/octree.cxx @@ -232,14 +232,13 @@ void Octree::GetPalIndex( OctreeNode* pNode ) } } -InverseColorMap::InverseColorMap( const BitmapPalette& rPal ) : - nBits( 8 - OCTREE_BITS ) +InverseColorMap::InverseColorMap( const BitmapPalette& rPal ) { const int nColorMax = 1 << OCTREE_BITS; - const unsigned long xsqr = 1 << ( nBits << 1 ); + const unsigned long xsqr = 1 << ( gnBits << 1 ); const unsigned long xsqr2 = xsqr << 1; const int nColors = rPal.GetEntryCount(); - const long x = 1 << nBits; + const long x = 1 << gnBits; const long x2 = x >> 1; sal_uLong r, g, b; long rxx, gxx, bxx; @@ -258,9 +257,9 @@ InverseColorMap::InverseColorMap( const BitmapPalette& rPal ) : long bdist = cBlue - x2; rdist = rdist*rdist + gdist*gdist + bdist*bdist; - const long crinc = ( xsqr - ( cRed << nBits ) ) << 1; - const long cginc = ( xsqr - ( cGreen << nBits ) ) << 1; - const long cbinc = ( xsqr - ( cBlue << nBits ) ) << 1; + const long crinc = ( xsqr - ( cRed << gnBits ) ) << 1; + const long cginc = ( xsqr - ( cGreen << gnBits ) ) << 1; + const long cbinc = ( xsqr - ( cBlue << gnBits ) ) << 1; sal_uLong* cdp = reinterpret_cast<sal_uLong*>(pBuffer.get()); sal_uInt8* crgbp = pMap.get(); |