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?

OK.  If changing this argument it would seem natural to make it a bool 
(and update callers to use true/false), unless that would be confusing for 
other reasons (e.g. passing it through to other functions where the 
corresponding argument isn't boolean).

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to