On Wed, Feb 25, 2015 at 5:30 PM, Brian Paul <bri...@vmware.com> wrote: > Don't include stuff we don't need. Fix a few #includes elsewhere to > keep thing building. > --- > src/mesa/main/compiler.h | 3 --- > src/mesa/main/cpuinfo.c | 2 +- > src/mesa/main/imports.h | 2 ++ > src/mesa/program/prog_parameter_layout.c | 2 +- > 4 files changed, 4 insertions(+), 5 deletions(-) > > diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h > index 12c02b2..a700ff9 100644 > --- a/src/mesa/main/compiler.h > +++ b/src/mesa/main/compiler.h > @@ -36,9 +36,6 @@ > > #include <assert.h> > #include <math.h> > -#include <stdlib.h> > -#include <stdio.h> > -#include <string.h> > > #include "util/macros.h" > > diff --git a/src/mesa/main/cpuinfo.c b/src/mesa/main/cpuinfo.c > index 0755d6b..67f22ab 100644 > --- a/src/mesa/main/cpuinfo.c > +++ b/src/mesa/main/cpuinfo.c > @@ -23,7 +23,7 @@ > */ > > > -#include "main/compiler.h" > +#include "main/imports.h"
Including imports.h seems strange, since it wasn't including it, even indirectly, before. Is imports.h really right? > #include "main/cpuinfo.h" > > > diff --git a/src/mesa/main/imports.h b/src/mesa/main/imports.h > index d886427..2af45a3 100644 > --- a/src/mesa/main/imports.h > +++ b/src/mesa/main/imports.h > @@ -36,6 +36,8 @@ > #define IMPORTS_H > > > +#include <stdlib.h> > +#include <string.h> > #include "compiler.h" > #include "glheader.h" > #include "errors.h" > diff --git a/src/mesa/program/prog_parameter_layout.c > b/src/mesa/program/prog_parameter_layout.c > index e834690..282a367 100644 > --- a/src/mesa/program/prog_parameter_layout.c > +++ b/src/mesa/program/prog_parameter_layout.c > @@ -28,7 +28,7 @@ > * \author Ian Romanick <ian.d.roman...@intel.com> > */ > > -#include "main/compiler.h" > +#include "main/imports.h" Same thing here. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev