page89
ESPAÇO ABERTO
Artigo
Sobraram apenas os “orcs” nas cavernas para revisar código. Paulatinamente, temos substituído os orcs por humanos. Se você desejar ser revisor, saiba que é muito bem-vindo!
Contudo, por conta da convivência, as vezes alguns humanos acabam adquirindo momentaneamente os hábitos dos orcs. Releve! Não é nada pessoal.
Após este breve esclarecimento sobre o grupo de revisores, quero salientar que este grupo é muito pequeno e muito trabalho cai sobre as costas deles. Então, pode ser que você tenha algum patch seu avaliado por alguém que não está no seu melhor dia, e ele possa estar mal-humorado e talvez um pouco rude. Mas lembre-se que não é pessoal.
Alguns revisores têm uma má reputação, por conta de suas análises críticas ao conteúdo de patches postados na lista. Contudo, saiba que para os desenvolvedores que estão diariamente envolvidos no código do Kernel, eles consideram cada palavra
destes revisores, pois eles estão nas listas com um papel muito importante: o de filtrar trabalho malfeito, mesmo que este trabalho malfeito seja de um destes desenvolvedores. Por isso, se algum revisor disser que seu patch está errado, e disser que você precisa corrigi-lo, faça e pronto! Não ignore o conselho, pois todo mundo está observando você, para ver se realmente consertou seu código como solicitado. Precisamos cada vez mais de revisores muito criteriosos. Se você gosta da qualidade do Linux que você usa, agradeça também ao trabalho deles.
Se todos pudessem gastar algumas horas por semana e rever os diferentes patches enviados para a lista de discussão, a qualidade dos patches enviados evoluiria muito. Mesmo se você não se sente como um bom desenvolvedor, não importa! Leia o código de outras pessoas e faça perguntas sobre ele. Se eles não puderem defender o código que fizeram, há algo de muito errado nele.