diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-29 11:18:21 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2019-04-30 08:43:51 +0200 |
| commit | dd8d5e5795358d732a9f7a8af7c35f662321e332 (patch) | |
| tree | 9983c2a5f0bc3f2c29133aa57e4ceb510eb68a11 /vcl/unx/generic/print | |
| parent | 22f2cf3ccc6d0c9ba2c2860735e789d6b3a25f72 (diff) | |
improve loplugin:stringconstant
to find more places we can elide the OUString() constructor at call
sites
Change-Id: Ie09f3c61f2c4b4959c97dc98ebcbaf7c51d5d713
Reviewed-on: https://gerrit.libreoffice.org/71514
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'vcl/unx/generic/print')
| -rw-r--r-- | vcl/unx/generic/print/printerjob.cxx | 2 | ||||
| -rw-r--r-- | vcl/unx/generic/print/prtsetup.cxx | 12 |
2 files changed, 7 insertions, 7 deletions
diff --git a/vcl/unx/generic/print/printerjob.cxx b/vcl/unx/generic/print/printerjob.cxx index f79e174225bf..4b00e8924b47 100644 --- a/vcl/unx/generic/print/printerjob.cxx +++ b/vcl/unx/generic/print/printerjob.cxx @@ -789,7 +789,7 @@ void PrinterJob::writeJobPatch( osl::File* pFile, const JobData& rJobData ) const PPDKey* pKey = nullptr; if( rJobData.m_pParser ) - pKey = rJobData.m_pParser->getKey( OUString( "JobPatchFile" ) ); + pKey = rJobData.m_pParser->getKey( "JobPatchFile" ); if( ! pKey ) return; diff --git a/vcl/unx/generic/print/prtsetup.cxx b/vcl/unx/generic/print/prtsetup.cxx index 6a38db31b1d9..d3c565846c17 100644 --- a/vcl/unx/generic/print/prtsetup.cxx +++ b/vcl/unx/generic/print/prtsetup.cxx @@ -178,7 +178,7 @@ void RTSPaperPage::update() // duplex if( m_pParent->m_aJobData.m_pParser && - (pKey = m_pParent->m_aJobData.m_pParser->getKey( OUString( "Duplex" ) )) ) + (pKey = m_pParent->m_aJobData.m_pParser->getKey( "Duplex" )) ) { m_pParent->insertAllPPDValues( *m_xDuplexBox, m_pParent->m_aJobData.m_pParser, pKey ); } @@ -190,7 +190,7 @@ void RTSPaperPage::update() // paper if( m_pParent->m_aJobData.m_pParser && - (pKey = m_pParent->m_aJobData.m_pParser->getKey( OUString( "PageSize" ) )) ) + (pKey = m_pParent->m_aJobData.m_pParser->getKey( "PageSize" )) ) { m_pParent->insertAllPPDValues( *m_xPaperBox, m_pParent->m_aJobData.m_pParser, pKey ); } @@ -202,7 +202,7 @@ void RTSPaperPage::update() // input slots if( m_pParent->m_aJobData.m_pParser && - (pKey = m_pParent->m_aJobData.m_pParser->getKey( OUString("InputSlot") )) ) + (pKey = m_pParent->m_aJobData.m_pParser->getKey( "InputSlot" )) ) { m_pParent->insertAllPPDValues( *m_xSlotBox, m_pParent->m_aJobData.m_pParser, pKey ); } @@ -236,17 +236,17 @@ IMPL_LINK( RTSPaperPage, SelectHdl, weld::ComboBox&, rBox, void ) if( &rBox == m_xPaperBox.get() ) { if( m_pParent->m_aJobData.m_pParser ) - pKey = m_pParent->m_aJobData.m_pParser->getKey( OUString( "PageSize" ) ); + pKey = m_pParent->m_aJobData.m_pParser->getKey( "PageSize" ); } else if( &rBox == m_xDuplexBox.get() ) { if( m_pParent->m_aJobData.m_pParser ) - pKey = m_pParent->m_aJobData.m_pParser->getKey( OUString( "Duplex" ) ); + pKey = m_pParent->m_aJobData.m_pParser->getKey( "Duplex" ); } else if( &rBox == m_xSlotBox.get() ) { if( m_pParent->m_aJobData.m_pParser ) - pKey = m_pParent->m_aJobData.m_pParser->getKey( OUString( "InputSlot" ) ); + pKey = m_pParent->m_aJobData.m_pParser->getKey( "InputSlot" ); } else if( &rBox == m_xOrientBox.get() ) { |