Date: prev next · Thread: first prev next last


Olá José Guilerme

oox/source/dump/oledumper.cxx
        
 if ( maProgId == "Forms.CommandButton.1" )
AxCommandButtonObject( *this ).dump();
else if( maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM(
"Forms.TextBox.1" ) ) ||
maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Forms.ListBox.1" ) ) ||
maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Forms.ComboBox.1" ) ) ||
maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "Forms.CheckBox.1" ) ) ||
maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM(
"Forms.OptionButton.1" ) ) ||
maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM(
"Forms.ToggleButton.1" ) ) ||
maProgId.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM( "RefEdit.Ctrl" ) ) )


maProgId já é uma OUString, portanto essas comparações podem ser com ==

Quer tal um patch3?

Abraços

Olivier

Em 13-12-2012 22:22, José Guilherme Vanz escreveu:
Acredito que é uma mudança bacana. Torna o fonte mais limpo e facilita
leitura...
Se ninguém discordar já posso alterar o patch e manda-lo novamente.

Obrigado!
Abraço


Em 13 de dezembro de 2012 22:15, Olivier Hallot <
olivier.hallot@libreoffice.org> escreveu:

Olá pessoal

Sobre o patch recente do José Guilherme

oox/source/dump/oledumper.cxx

void OleStorageObject::implDumpStream( const Reference< XInputStream >&
rxStrm, const OUString& /*rStrgPath*/, const OUString& rStrmName, const
OUString& rSysFileName )
{
    if ( rStrmName == "\001CompObj" )
        OleCompObjObject( *this, rxStrm, rSysFileName ).dump();
    else if( rStrmName.equalsAsciiL( RTL_CONSTASCII_STRINGPARAM(
"\005SummaryInformation" ) ) || rStrmName.equalsAsciiL(
RTL_CONSTASCII_STRINGPARAM( "\005DocumentSummaryInformation" ) ) )
        OlePropertyStreamObject( *this, rxStrm, rSysFileName ).dump();
    else
        BinaryStreamObject( *this, rxStrm, rSysFileName ).dump();
}

podemos usar o operador == para continuar comparando ao invés do membro
.equalsAscii(...) em sStrmName. Isso por que o literal " ... " será
passado como um OUString.

Sempre consultem a clase OUString em
http://api.libreoffice.org/docs/cpp/ref/classrtl_1_1_o_u_string.html :

sal_Bool        operator== (const OUString &rStr1, const OUString &rStr2)
throw ()

O que acham?

Abraços
--
Olivier Hallot
Comunidade LibreOffice

--
Você está recebendo e-mails da lista dev@pt-br.libreoffice.org
# Informações sobre os comandos disponíveis (em inglês):
  mande e-mail vazio para dev+help@pt-br.libreoffice.org
# Cancelar sua assinatura: mande e-mail vazio para:
  dev+unsubscribe@pt-br.libreoffice.org
# Arquivo de mensagens: http://listarchives.libreoffice.org/pt-br/dev





-- 
Olivier Hallot
Comunidade LibreOffice

-- 
Você está recebendo e-mails da lista dev@pt-br.libreoffice.org
# Informações sobre os comandos disponíveis (em inglês):
  mande e-mail vazio para dev+help@pt-br.libreoffice.org
# Cancelar sua assinatura: mande e-mail vazio para:
  dev+unsubscribe@pt-br.libreoffice.org
# Arquivo de mensagens: http://listarchives.libreoffice.org/pt-br/dev

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.