Eu queria complementar...
Nenhum trabalho no código deve ser minimizado, mas é claro que há
patches e patches.
A melhora do aspecto visual da leitura não é de se desprezar. Veja esse
exemplo:
(...)
- aString = ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("Olá Mundo"));
+ aString = "Olá Mundo";
(...)
Vamos combinar que a segunda instrução é bem mais fácil de ler. De certo
não teremos ganho (significativo) de performance na execução, no melhor
um pequeno ganho microscópico de tempo de compilação, mas queremos
desencavar desse código toda a poeira de gerações passadas de técnicas e
avanços do compilador e da linguagem C++. É uma certa arqueologia.
Aos poucos vamos avançando. Tem muita coisa pra fazer. A lista dos
easy-hacks está ai pra isso. Alias, a ideia do "easy-hack" pegou tão bem
que muitas atividades no portal, no wiki, na documentação do LibreOffice
estão sendo formatadas como easy-hacks. Tem pra tudo e pra todos que
gostam do assunto.
Por mais singelo que seja a remoção do rtl:: do código, considero um
avanço significativo e é um começo viável para todos que queiram
contribuir, aperfeiçoar-se, estudar, aprender, criar currículo, etc..
Nesse particular: Fez o patch e entrou, tá feito. Tá gravado, pode
mostrar, levantar o braço, entregar a prova e conferir o gabarito.
E vamos combinar também que remover ::rtl:: não é nada mágico, e o
Ricardo não tem o privilégio dessa técnica. Eu mesmo ja fiz alguns, e
estou ansioso de ver mais gente fazendo. O Marcos e o Jose Guilherme
também limparam trocentas linhas inúteis no java do LibreOffice.
Abraços
Em 26-10-2012 23:26, Ricardo Montania escreveu:
Boa noite Anderson.
Olha, vou te responder o que foi respondido pelo Lubos Lunak, sobre a mesma
dúvida, na lista em inglês:
Yes, it is more of a cosmetic cleanup - the classes are already in practice
namespaced by the O prefix and so not explicitly adding the rtl:: prefix is
avoiding the nuisance of the extra writing and also, since these are very
commonly uses classes, makes the code more readable.
É mais do que apenas uma limpeza "visual". As classes estão declaradas
diretamente nos namespace por serem largamente utilizados, então as
declarações [::rtl::] podem ser removidas, removendo código extra, e
deixanto o código mais legível.
Ricardo.
2012/10/26 Anderson Unsonst <andersonunsonst@gmail.com>
vocês estão fazendo varias limpezas, substituindo o const ::rtl::OUString&
por const OUString& na verdade a maionese aqui está grossa hauhauhau,
como essas substituições melhoram o desempenho do sistema ?
--
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
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.