Re: [pt-br-dev] Fwd: [PATCH] Removal ::rtl:: prefixes and macros in oxx

José Guilherme

Neste patch, dá pra ir mais a fundo...

em oox/source/core/filterdetect.cxx

há linhas com

return CREATE_OUSTRING( "writer_MS_Word_2007" );

Acontece que a macro CREATE_OUSTRING está definida em

oox/inc/oox/helper/helper.hxx

na forma

#define CREATE_OUSTRING( ascii ) \
    ::rtl::OUString::intern( RTL_CONSTASCII_USTRINGPARAM( ascii ) )

Assim, vocẽ pode substituir

return CREATE_OUSTRING( "writer_MS_Word_2007" );

por
return OUString( "writer_MS_Word_2007" );

eliminando a necessidade da macro

OK?
Olivier


Em 01-12-2012 11:05, José Guilherme Vanz escreveu:
> ---------- Forwarded message ----------
> From: José Guilherme Vanz (via Code Review) <gerrit@gerrit.libreoffice.org>
> Date: 2012/12/1
> Subject: [PATCH] Removal ::rtl:: prefixes and macros in oxx
> To:
> Cc: LibreOffice Developer List <libreoffice@lists.freedesktop.org>
>
>
> Hi,
>
> I have submitted a patch for review:
>
>     https://gerrit.libreoffice.org/1211
>
> To pull it, you can do:
>
>     git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/11/1211/1
>
> Removal ::rtl:: prefixes and macros in oxx
>
> This commit removes some ::rtl:: prefixes and RTL_CONSTASCII_STRINGPARAM,
>  RTL_CONSTASCII_USTRINGPARAM macros in oxx.
>
> Change-Id: I12bfb2528e8a9aaf3f119a1e6f0ae0246d16a772
> Signed-off-by: José Guilherme Vanz <guilherme.sft@gmail.com>
> ---
> M oox/source/core/binarycodec.cxx
> M oox/source/core/filterdetect.cxx
> M oox/source/core/fragmenthandler2.cxx
> M oox/source/core/xmlfilterbase.cxx
> 4 files changed, 30 insertions(+), 30 deletions(-)
>
>
> --
> To view, visit https://gerrit.libreoffice.org/1211
> To unsubscribe, visit https://gerrit.libreoffice.org/settings
>
> Gerrit-MessageType: newchange
> Gerrit-Change-Id: I12bfb2528e8a9aaf3f119a1e6f0ae0246d16a772
> Gerrit-PatchSet: 1
> Gerrit-Project: core
> Gerrit-Branch: master
> Gerrit-Owner: José Guilherme Vanz <guilherme.sft@gmail.com>
>
> _______________________________________________
> LibreOffice mailing list
> LibreOffice@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice

-- 
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