On 04/25/2011 10:47 AM, Kenneth Graunke wrote:
On 04/25/2011 09:37 AM, Nicolas Kaiser wrote:
Remove duplicated includes of guarded headers.

Signed-off-by: Nicolas Kaiser<ni...@nikai.net>
---
src/mesa/main/pack.c | 1 -
src/mesa/main/texstore.c | 1 -
2 files changed, 0 insertions(+), 2 deletions(-)

What's the point?

The only justification I've ever heard is for (theoretically) speeding
up build times (by some non-measurable amount). This is really only
interesting if you're continually compiling Mesa (for development or
nightly builds for testing). But if you're doing that, use ccache, as
it'll make the build go orders of magnitude faster (by not actually
bothering with the work).

For those of us who build with ccache, patches like this cause a
pointless rebuild, actually making us spend -more- time compiling.

I like to see sloppiness cleaned up.  I'll commit this soon.

-Brian
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to