diff options
| author | Jelle van der Waa <jelle@vdwaa.nl> | 2013-08-17 01:11:11 +0200 |
|---|---|---|
| committer | Andrzej J.R. Hunt <andrzej@ahunt.org> | 2013-08-17 07:48:21 +0000 |
| commit | 4f20c9f6f95c117bcdb520682df4fa1429a56477 (patch) | |
| tree | f0e1ec5882b9d67a23f99135f91ac86b8f9f832c /xmlsecurity/source/dialogs/certificatechooser.cxx | |
| parent | 523a1c00e1bf4da53daaa0fbaadc670f8b89ea37 (diff) | |
fdo#57950: Remove some chained appends in xmlsecurity
Change-Id: If70fa38ad4836b6dacdd0c5577c66b78be6826a6
Reviewed-on: https://gerrit.libreoffice.org/5465
Reviewed-by: Andrzej J.R. Hunt <andrzej@ahunt.org>
Tested-by: Andrzej J.R. Hunt <andrzej@ahunt.org>
Diffstat (limited to 'xmlsecurity/source/dialogs/certificatechooser.cxx')
| -rw-r--r-- | xmlsecurity/source/dialogs/certificatechooser.cxx | 17 |
1 files changed, 5 insertions, 12 deletions
diff --git a/xmlsecurity/source/dialogs/certificatechooser.cxx b/xmlsecurity/source/dialogs/certificatechooser.cxx index d339cde84220..aa829172641c 100644 --- a/xmlsecurity/source/dialogs/certificatechooser.cxx +++ b/xmlsecurity/source/dialogs/certificatechooser.cxx @@ -65,12 +65,8 @@ CertificateChooser::CertificateChooser( Window* _pParent, uno::Reference< uno::X m_pCertLB = new SvSimpleTable(*pSignatures); static long nTabs[] = { 3, 0, 30*nControlWidth/100, 60*nControlWidth/100 }; m_pCertLB->SetTabs( &nTabs[0] ); - OUStringBuffer sHeader; - sHeader.append(get<FixedText>("issuedto")->GetText()) - .append("\t").append(get<FixedText>("issuedby")->GetText()) - .append("\t").append(get<FixedText>("expiration")->GetText()); - m_pCertLB->InsertHeaderEntry(sHeader.makeStringAndClear()); - + m_pCertLB->InsertHeaderEntry(get<FixedText>("issuedto")->GetText() + "\t" + get<FixedText>("issuedby")->GetText() + + "\t" + get<FixedText>("expiration")->GetText()); m_pCertLB->SetSelectHdl( LINK( this, CertificateChooser, CertificateHighlightHdl ) ); m_pCertLB->SetDoubleClickHdl( LINK( this, CertificateChooser, CertificateSelectHdl ) ); m_pViewBtn->SetClickHdl( LINK( this, CertificateChooser, ViewButtonHdl ) ); @@ -174,12 +170,9 @@ void CertificateChooser::ImplInitialize() // fill list of certificates; the first entry will be selected for ( sal_Int32 nC = 0; nC < nCertificates; ++nC ) { - OUStringBuffer sEntry( XmlSec::GetContentPart( maCerts[ nC ]->getSubjectName() ) ); - sEntry.append( '\t' ); - sEntry.append( XmlSec::GetContentPart( maCerts[ nC ]->getIssuerName() ) ); - sEntry.append( '\t' ); - sEntry.append( XmlSec::GetDateString( maCerts[ nC ]->getNotValidAfter() ) ); - SvTreeListEntry* pEntry = m_pCertLB->InsertEntry( sEntry.makeStringAndClear() ); + SvTreeListEntry* pEntry = m_pCertLB->InsertEntry( XmlSec::GetContentPart( maCerts[ nC ]->getSubjectName() ) + + "\t" + XmlSec::GetContentPart( maCerts[ nC ]->getIssuerName() ) + + "\t" + XmlSec::GetDateString( maCerts[ nC ]->getNotValidAfter() ) ); pEntry->SetUserData( ( void* )(sal_IntPtr)nC ); // missuse user data as index } |