Re: [RFA] "constify" parse_exp_1

[Keith Seitz <keiths at redhat dot com>]

On 03/12/2013 03:18 AM, Pedro Alves wrote:
> s/craete/create

Fixed.

> (Minor note: pedantically, uppercase is used when talking
> about the value of the variable, rather than the variable
> itself.)

Fixed/changed.

> > -extern char *skip_to_space (char *inp);
> > +#define skip_to_space(INP) ((char *) skip_to_space_const ((INP)))
>
> Double (())s not necessary:

Fixed.

>    #define skip_to_space(INP) ((char *) skip_to_space_const (INP))
>
> Can you please add:
>
>    + /* A const-correct version of the above.  */

Fixed.

> Otherwise this all looks good to me.

Committed. Thank you for taking a look at this.

Keith