On Thu, Mar 22, 2012 at 03:47:00PM -0700, Ben Pfaff wrote: > On Fri, Mar 23, 2012 at 07:40:24AM +0900, Simon Horman wrote: > > On Thu, Mar 22, 2012 at 10:07:06AM -0700, Ben Pfaff wrote: > > > On Thu, Mar 22, 2012 at 03:43:35PM +0900, Simon Horman wrote: > > > > Make the name and description members of struct stress_option > > > > const as they are used to store constant strings. > > > > > > > > Signed-off-by: Simon Horman <[email protected]> > > > > > > With this patch, I get from GCC: > > > > > > ../lib/process.h:26: note: expected ‘char **’ but argument is of type > > > ‘const char **' > > > > > > and from sparse: > > > > > > ../ovsdb/ovsdb-server.c:128:32: warning: incorrect type in argument 1 > > > (different modifiers) > > > ../ovsdb/ovsdb-server.c:128:32: expected char **argv > > > ../ovsdb/ovsdb-server.c:128:32: got char const **<noident> > > > ../ovsdb/ovsdb-server.c:128: error: passing argument 1 of > > > ‘process_start’ from incompatible pointer type > > > > I'm not doing very well with these const patches am I :( > > Well, when you send one that doesn't add warnings, I'll apply it ;-)
Yes, sorry about the noise. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
