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

Opa, excelente José. Olivier, vou acrescentar esses comentários e sugestões
no material de strings que tenho aqui.

Ricardo.

2012/11/29 José Guilherme Vanz <guilherme.sft@gmail.com>

> Olá, Olivier!
>
> Ok, obrigado pelas dicas!
> Nos próximos patches irei tomar mais cuidado e irei melhorar-los.
> :)
>
> Abraço!
> Att. Guilherme
>
>
> 2012/11/29 Olivier Hallot <olivier.hallot@libreoffice.org>
>
> > 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/corerefs/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
>
>
> --
> Att. José Guilherme Vanz
> "Se você é capaz de tremer de indignação cada vez que se comete uma
> injustiça no mundo, então somos companheiros" - Che Guevara
>
> --
> 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


-- 
Ricardo Montania Prado de Campos
http://www.linuxafundo.com.br

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