Am 08.02.2010 um 16:33 schrieb Matteo Bruni:
> Other than the things Henri last mentioned, there are some more small
> changes (make some struct const/static, whitespace fixes, remove some
> useless cruft, ...) particularly in patches 2 and 3.
> <0001-d3dx9-Partial-implementation-of-D3DXAssembleShader-fun.txt>
Henri is on holidays this week(I think), here are 3 things I noticed:
> + static const struct wpp_callbacks wpp_callbacks = {
> + wpp_lookup_mem,
> + wpp_open_mem,
The placement of the { is not consistent with the rest of the patch
(and the entire patch 1 is inconsistent with patch 2 and 3). It seems to me the predominant style in d3dx9 is the one used by patch 1, although there are some inconsistencies.
> +void wpp_write_message(const char *fmt, va_list args)
> +{
Unless I am missing some wpp magic here you can make this function(and others) static as well.
> + * Copyright 2009 Matteo Bruni
Maybe make this 2009-2010?