[PATCH 0/2] Improve error reporting functions

[Thadeu Lima de Souza Cascardo]

On Fri, Oct 24, 2014 at 02:54:55PM -0400, Sergio Durigan Junior wrote:
> On Thursday, October 23 2014, Thadeu Lima de Souza Cascardo wrote:
> 
> > O que eu mudaria nesses commits são os cabeçalhos. stdlib não é
> > necessário no novo cabeçalho. No entanto, ainda é necessário nos
> > arquivos que o utilizam. E stdio pode ser necessário em um desses
> > arquivos também. Teste, por exemplo, mudar o cabeçalho de erro pra algo
> > assim:
> >
> > #ifdef DEBUG
> > #include <stdio.h>
> > #define ERROR(x) fprintf(stderr, x)
> > #else
> > #define ERROR(x) do { } while(0)
> > #endif
> >
> > Cheque se o código compila legal depois disso.
> 
> Acho que não entendi o que você quis dizer.  Você tá falando do
> rnet_defs.h né?
> 
> O propósito desse arquivo é ser incluído por todos os *.c do projeto.  E
> se você olhar os *.c, verá que todos incluem stdio.h e stdlib.h (menos o
> pmhash.c, que inclui apenas stdlib.h).  Pelo que entendi da sua
> mensagem, você prefere deixar a inclusão do stdio.h e stdlib.h dentro
> dos arquivos *.c, é isso?
> 
> Aliás, sua mensagem me fez notar que eu não fiz todos os arquivos *.c
> incluirem o rnet_defs.h.  Vou reenviar os patches com essa correção.
> 
> -- 
> Sergio
> GPG key ID: 0x65FC5E36
> Please send encrypted e-mail if possible
> http://sergiodj.net/

Yep, prefiro incluir só aqueles que precisarem dos headers. Como você
bem notou, há exceções, e dei um exemplo de alteração que eu aceitaria e
quebraria os dois módulos que você alterou.

Prefiro não incluir o rnet_defs.h gratuitamente nos demais módulos.

Faria sentido se todos os arquivos incluissem esse cabeçalho e o projeto
documentasse que todo novo módulo precisa incluí-lo e pode omitir a
inclusão do que está ali, mas acho isso difícil de manter e menos
modular, na verdade, já que dificulta o reuso dos módulos em outros
projetos.

Discorda?

Abraços.
Cascardo.
------------ próxima parte ------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://www.fsfla.org/pipermail/softwares-impostos/attachments/20141024/53409977/attachment.pgp>