Re: [pt-br-dev] Limpeza OUString

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