I saw these warnings also on scons builds. Vinson - if you set "quiet=no", scons will print out the full gcc invocation, there may be a clue there what's causing this.
Keith ________________________________________ From: mesa-dev-bounces+keithw=vmware....@lists.freedesktop.org [mesa-dev-bounces+keithw=vmware....@lists.freedesktop.org] On Behalf Of Ian Romanick [...@freedesktop.org] Sent: Thursday, September 16, 2010 2:31 PM To: mesa-dev@lists.freedesktop.org Subject: Re: [Mesa-dev] Mesa (master): glsl: Fix ' format not a string literal and no format arguments' warning. -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Vinson Lee wrote: > Module: Mesa > Branch: master > Commit: f20f2cc3306310f6fc4c338f91cfac10f98335d3 > URL: > http://cgit.freedesktop.org/mesa/mesa/commit/?id=f20f2cc3306310f6fc4c338f91cfac10f98335d3 > > Author: Vinson Lee <v...@vmware.com> > Date: Wed Sep 15 05:17:57 2010 -0700 > > glsl: Fix 'format not a string literal and no format arguments' warning. > > Fix the following GCC warning. > loop_controls.cpp: In function 'int calculate_iterations(ir_rvalue*, > ir_rvalue*, ir_rvalue*, ir_expression_operation)': > loop_controls.cpp:88: warning: format not a string literal and no format > arguments > > --- > > src/glsl/loop_controls.cpp | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/src/glsl/loop_controls.cpp b/src/glsl/loop_controls.cpp > index 9619d8a..2ef3d30 100644 > --- a/src/glsl/loop_controls.cpp > +++ b/src/glsl/loop_controls.cpp > @@ -85,7 +85,7 @@ int > calculate_iterations(ir_rvalue *from, ir_rvalue *to, ir_rvalue *increment, > enum ir_expression_operation op) > { > - void *mem_ctx = talloc_init(__func__); > + void *mem_ctx = talloc_init("%s", __func__); If __func__ is not showing up as a string literal, something else is wrong. This is supposed to be a #define generated by the compiler. Is this a case where we're wrapping __func__ for Visual Studio and something is going wrong (since this is GCC)? I also find this odd because I don't get this warning on my builds... and I enable a lot of extra warning flags. > > ir_expression *const sub = > new(mem_ctx) ir_expression(ir_binop_sub, from->type, to, from); -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ iEYEARECAAYFAkySHDsACgkQX1gOwKyEAw/n/ACePdgvPa0xwSs02PqSQdnTF1pr HekAn07NDlPtU2bvUGQSf1W0tFLtOWa1 =R3UQ -----END PGP SIGNATURE----- _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
