[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [PATCH 3/3] Silence a compiler warning.



On Fri, Jan  7, 2011 at 00:31:07 +0100, Ferenc Wagner wrote:

> Ferenc Wagner <wferi@niif.hu> writes:
> 
> > -            execvp("udhcpc", arguments);
> > +            /* execvp doesn't like const strings for no reason, so we can
> > +               cast away the const to suppress the compiler warning */
> > +            execvp("udhcpc", (char **)arguments);
> 
> Actually, I started to feel bad about this part.  Maybe it would be
> wiser to use non-const strings from the beginning?  That would require a
> larger patch, though...  And ignoring the warning hasn't caused any
> trouble yet.  Does anybody know why execvp hates const strings?

execvp takes 'char *const argv[]', not 'const char **argv'.

Cheers,
Julien

Attachment: signature.asc
Description: Digital signature


Reply to: