Hi!

> I've spent some time "cleaning" the includes by using forward
> declarations. This means that not all headers are automatically
> included, and that the .cpp should include the appropriate header if it
> ever uses (by allocation/destruction or use of a method) something known
> through a forward declaration.
>
> This may be troublesome, but that what I add to use some time ago to fix
> a bug in Windows.
>
> It generally has multiple benefits (compile time, dependency rebuilding,
> ...) except of course bothering the developer, that now has to do the
> includes manually.
>   

Great work Kurosu! :) It compiles really faster on my computer !

> There are probably things that I missed because of #ifdef's, so don't be
> surprised if debug or some particular setup doesn't build anymore.
>   

You're right, it didn't compile anymore without --enable-debug flag for 
configure. The fault to NET_ASSERT macro defined in src/tool/error.h 
which use Network class when DEBUG is not defined. I should have fixed 
it in r2791.

Best regards,

Matt (gentildemon)

_______________________________________________
Wormux-dev mailing list
Wormux-dev@gna.org
https://mail.gna.org/listinfo/wormux-dev

Répondre à