Re: [C PATCH] Tidy build_unary_op

2016-09-13 Thread Marek Polacek
On Mon, Sep 12, 2016 at 09:53:52PM +, Joseph Myers wrote: > On Sat, 10 Sep 2016, Marek Polacek wrote: > > > A minor cleanup. First, FLAG is a really bad name for a parameter, and > > second, > > using two vars for the same thing is redundant (we never modify any of > > them). > > Also some

Re: [C PATCH] Tidy build_unary_op

2016-09-12 Thread Joseph Myers
On Sat, 10 Sep 2016, Marek Polacek wrote: > A minor cleanup. First, FLAG is a really bad name for a parameter, and > second, > using two vars for the same thing is redundant (we never modify any of them). > Also some bad formatting. > > Bootstrap/regtest on x86_64-linux running. Ok for trunk?