On 08/07/2013 04:10 PM, [email protected] wrote: > Hi! >> void tst_run_cmd_fds(void (cleanup_fn)(void), >> - char *const argv[], >> + const char *const argv[], >> int stdout_fd, >> int stderr_fd); >> >> @@ -236,7 +236,7 @@ void tst_run_cmd_fds(void (cleanup_fn)(void), >> * not needed. >> */ >> void tst_run_cmd(void (cleanup_fn)(void), >> - char *const argv[], >> + const char *const argv[], >> const char *stdout_path, >> const char *stderr_path); > Gosh, that fixed the warning in the test but I've overlooked that > execvp() takes char *const argv[] argument so it introduced warning in > the lib. Now I wonder why execvp() has such strange parameter type...
> > I'm trying to figure out how to fix it 'right' but it gets unnecessary > complicated. I guess that we can fix this patch by casting the array in > the lib before it gets to execvp(). I think that casting mod_name to (char *) is easier than casting array before it gets to execvp(). And it doesn't require any changes of tst_run_cmd interfaces (ie they accepts argv the same way as execvp does)... Maybe leave everything as is and just cast mod_name to (char *)? ------------------------------------------------------------------------------ Get 100% visibility into Java/.NET code with AppDynamics Lite! It's a free troubleshooting tool designed for production. Get down to code-level detail for bottlenecks, with <2% overhead. Download for free and get started troubleshooting in minutes. http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
