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

Olá José

Justamente acabo de aprovar... mas vc pode melhorar ainda mais nas
seguintes situações:

1)
OUString m_sDescription;
(...)
- m_sDescription += OUString(RTL_CONSTASCII_USTRINGPARAM(",uniqueValue="));

+ m_sDescription += ",uniqueValue=";
// por que m_sDescription já foi declarado um OUString

2)
- OUString error = OUString(RTL_CONSTASCII_USTRINGPARAM("io.acceptor:
Couldn't setup pipe ));

+ OUString error("io.acceptor: Couldn't setup pipe");
// forma canonica de criar um OUString


3)
#define SERVICE_NAME "com.sun.star.connection.Acceptor"
(...)
Sequence< OUString > seqNames(1);
- seqNames.getArray()[0] =
OUString(RTL_CONSTASCII_USTRINGPARAM(SERVICE_NAME));

- seqNames.getArray()[0] = SERVICE_NAME;
// porque seqName já é uma sequencia de OUString e SERVICE_NAME foi
defined mais acima como string.

Abraços

Olivier



Em 29-11-2012 07:56, José Guilherme Vanz escreveu:
> Aproveitando as dicas do Ricardo, nosso String Killer... mandei um patch de
> string tbm. :)
>
>
> ---------- Forwarded message ----------
> From: José Guilherme Vanz (via Code Review) <gerrit@gerrit.libreoffice.org>
> Date: 2012/11/29
> Subject: [PATCH] Removal ::rtl:: prefixes and macros in IO
> To:
> Cc: LibreOffice Developer List <libreoffice@lists.freedesktop.org>
>
>
> Hi,
>
> I have submitted a patch for review:
>
>     https://gerrit.libreoffice.org/1194
>
> To pull it, you can do:
>
>     git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/94/1194/1
>
> Removal ::rtl:: prefixes and macros in IO
>
> This commit removes some ::rtl:: prefixes and RTL_CONSTASCII_STRINGPARAM,
>  RTL_CONSTASCII_USTRINGPARAM macros in IO.
>
> Change-Id: I823fcdfe46d4d4093118f6eaab89c1ce42883a6c
> Signed-off-by: José Guilherme Vanz <guilherme.sft@gmail.com>
> ---
> M io/source/acceptor/acc_pipe.cxx
> M io/source/acceptor/acc_socket.cxx
> M io/source/acceptor/acceptor.cxx
> M io/source/acceptor/acceptor.hxx
> 4 files changed, 33 insertions(+), 37 deletions(-)
>
>
> --
> To view, visit https://gerrit.libreoffice.org/1194
> To unsubscribe, visit https://gerrit.libreoffice.org/settings
>
> Gerrit-MessageType: newchange
> Gerrit-Change-Id: I823fcdfe46d4d4093118f6eaab89c1ce42883a6c
> 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