diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-01-30 16:38:54 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-01-31 08:56:20 +0000 |
| commit | 1c3e84d8192218befebcddae2ed9842d081dc6c7 (patch) | |
| tree | f4fc5cddd181d6d49bf55f74e6d3d107b8ae8d18 /starmath | |
| parent | Updated core (diff) | |
| download | core-1c3e84d8192218befebcddae2ed9842d081dc6c7.tar.gz core-1c3e84d8192218befebcddae2ed9842d081dc6c7.zip | |
teach lolugin:stringconstant about calling constructors
so we can remove unnecessary calls to the OUString(literal) constructor
when calling constructors like this:
Foo(OUString("xxx"), 1)
Change-Id: I1de60ef561437c86b27dc9cb095a5deb2e103b36
Reviewed-on: https://gerrit.libreoffice.org/33698
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'starmath')
| -rw-r--r-- | starmath/source/cfgitem.cxx | 2 | ||||
| -rw-r--r-- | starmath/source/format.cxx | 8 | ||||
| -rw-r--r-- | starmath/source/node.cxx | 4 |
3 files changed, 7 insertions, 7 deletions
diff --git a/starmath/source/cfgitem.cxx b/starmath/source/cfgitem.cxx index 1ea663b8ddb9..b1805760ccea 100644 --- a/starmath/source/cfgitem.cxx +++ b/starmath/source/cfgitem.cxx @@ -330,7 +330,7 @@ const OUString SmFontFormatList::GetNewFontFormatId() const SmMathConfig::SmMathConfig() : - ConfigItem(OUString("Office.Math")) + ConfigItem("Office.Math") , pFormat() , pOther() , pFontFormatList() diff --git a/starmath/source/format.cxx b/starmath/source/format.cxx index b4b9793e4733..ed0adca219b7 100644 --- a/starmath/source/format.cxx +++ b/starmath/source/format.cxx @@ -135,10 +135,10 @@ SmFormat::SmFormat() vFont[FNT_FUNCTION] = vFont[FNT_NUMBER] = vFont[FNT_TEXT] = - vFont[FNT_SERIF] = SmFace(OUString(FNTNAME_TIMES), aBaseSize); - vFont[FNT_SANS] = SmFace(OUString(FNTNAME_HELV), aBaseSize); - vFont[FNT_FIXED] = SmFace(OUString(FNTNAME_COUR), aBaseSize); - vFont[FNT_MATH] = SmFace(OUString(FNTNAME_MATH), aBaseSize); + vFont[FNT_SERIF] = SmFace(FNTNAME_TIMES, aBaseSize); + vFont[FNT_SANS] = SmFace(FNTNAME_HELV, aBaseSize); + vFont[FNT_FIXED] = SmFace(FNTNAME_COUR, aBaseSize); + vFont[FNT_MATH] = SmFace(FNTNAME_MATH, aBaseSize); vFont[FNT_MATH].SetCharSet( RTL_TEXTENCODING_UNICODE ); diff --git a/starmath/source/node.cxx b/starmath/source/node.cxx index a30dc3e38da0..7cf4b7384839 100644 --- a/starmath/source/node.cxx +++ b/starmath/source/node.cxx @@ -531,7 +531,7 @@ void SmTableNode::Arrange(OutputDevice &rDev, const SmFormat &rFormat) SmTmpDevice aTmpDev (rDev, true); aTmpDev.SetFont(GetFont()); - SmRect aRect = (SmRect(aTmpDev, &rFormat, OUString("a"), + SmRect aRect = (SmRect(aTmpDev, &rFormat, "a", GetFont().GetBorderWidth())); mnFormulaBaseline = GetAlignM(); // move from middle position by constant - distance @@ -590,7 +590,7 @@ void SmLineNode::Arrange(OutputDevice &rDev, const SmFormat &rFormat) //! be sure to use a character that has explicitly defined HiAttribut //! line in rect.cxx such as 'a' in order to make 'vec a' look same to //! 'vec {a}'. - SmRect::operator = (SmRect(aTmpDev, &rFormat, OUString("a"), + SmRect::operator = (SmRect(aTmpDev, &rFormat, "a", GetFont().GetBorderWidth())); // make sure that the rectangle occupies (almost) no space SetWidth(1); |