Prathamesh Kulkarni <prathamesh.kulka...@linaro.org> writes:
> @@ -186,6 +186,23 @@ skip_alternative (const char *p)
>  /* Nonzero means volatile operands are recognized.  */
>  extern int volatile_ok;
>  
> +/* RAII class for temporarily setting volatile_ok.  */
> +
> +class temporary_volatile_ok
> +{
> +public:
> +  temporary_volatile_ok (int value): save_volatile_ok (volatile_ok)

Missing space before the ":".

> +  {
> +    volatile_ok = value;
> +  }
> +
> +  ~temporary_volatile_ok () { volatile_ok = save_volatile_ok; }
> +
> +private:
> +  temporary_volatile_ok (const temporary_volatile_ok&);

Missing space before the "&".

OK with those changes, thanks.

Richard


> +  int save_volatile_ok;
> +};
> +
>  /* Set by constrain_operands to the number of the alternative that
>     matched.  */
>  extern int which_alternative;

Reply via email to