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

Então, usaremos este documento no HackDay?

E aquela idéia de fazermos uma chacina de strings? Interessante?

Pois o Olivier chegou a ver as novas estruturas de layout, e não é algo
"tão" trivial assim... =/
Em 29 de novembro de 2012 08:48, Ricardo Montania <
ricardo@linuxafundo.com.br> escreveu:

> 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


-- 
Att,

Marcos Paulo de Souza
Acadêmico de Ciencia da Computação - FURB - SC
Github: https://github.com/marcosps/
"Uma vida sem desafios é uma vida sem razão"
"A life without challenges, is a non reason life"

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