Date: prev next · Thread: first prev next last


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

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.