On Mon, Mar 11, 2013 at 5:28 PM, Brian Paul <bri...@vmware.com> wrote: > On Sat, Mar 2, 2013 at 7:29 AM, Brian Paul <bri...@vmware.com> wrote: >> I've respun my remove-mfeatures branch as remove-mfeatures-2. It's in my >> repo on freedesktop.org >> >> This removes the mfeatures.h file and the last of the #ifdef FEATURE_x >> code >> from core Mesa. >> >> Note, however, that the scons/makefiles still define FEATURE_GL, >> FEATURE_ES1, FEATURE_ES2, etc. because that controls whether some source >> files are built and in some places (like egl-static/egl_st.c) we need to >> test those symbols to avoid calling non-existant functions. >> >> If a few people can test, that'd be great. > > > Ping. Can anyone test the updated remove-mfeatures-2 branch? Thanks.
I think I see an issue. In src/mesa/drivers/dri/intel/intel_screen.c, we use: #ifndef FEATURE_ES1 and #ifndef FEATURE_ES2 I think these chunks of code should be removed in your series, right? -Jordan _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
