Re: [HACKERS] Modulo syntax

> > > I ran the query
> > > update producers SET cor_id = producer_id % 9 + 1;
> > > and found that result is eqiuvalent to
> > > update producers SET cor_id = producer_id % 9;
> > > I added parens:
> > > update producers SET cor_id = (producer_id % 9) + 1;
> > > and got what I needed.
> > Looks like a bug. We have associativity for +, -, * and /, but not %.
> > From gram.y:
> > %left '+' '-'
> > %left '*' '/'
> > I will add '%' to that.
>
> This will not fix the associativity problem, unless you *also* go
> through and add the explicit syntax *throughout* gram.y, as is
> currently done for '+', '-', etc.
>
> I'm pretty sure that we don't want to do this, since there are way too
> many other operators which would need the same treatment.

I did this for %. I felt it was common enough and similar to / that
people should expect it to have / associativity. I did not play with
any other operators.

>
> The correct solution will be to identify the operator as a particular
> class in scan.l, include that class in the associativity declarations,
> and then handle that class in the body of gram.y. Sort of like we do
> for generic operators already, but with some discrimination between
> them. To be done right, we should look up the precedence in a db
> table, to allow new operators to participate in the scheme. In any
> case, gram.y will become more complex...

Yikes. Don't think we want to go there.
>
> Unless we are going to solve this, I would suggest backing out the
> change in gram.y.

I would like to keep % as a special case like /.

--
Bruce Momjian | http://www.op.net/~candle
maillist(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026