Re: [pt-br-dev] sal_Bool operator== (const OUString &rStr1, const OUString &rStr2) throw ()

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