On Thu, Feb 26, 2015 at 7:34 AM, Brian Paul <bri...@vmware.com> wrote: > On 02/25/2015 08:36 PM, Matt Turner wrote: >> >> 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. >> > > Those files use functions like strlen(), malloc(), free(), etc, which no > longer get pulled in by compiler.h
I guess my question is whether we need imports.h or whether we can simply include stdlib.h, string.h, and friends directly? _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev