On 8/12/20 6:28 AM, Martin Liška wrote:
On 8/11/20 4:58 PM, Martin Sebor wrote:
On 8/11/20 5:36 AM, Martin Liška wrote:
Hello.

All right, I did it in 3 steps:
1) - new exact argument is added (no default value) - I tested the on x86_64-linux-gnu
and I build all cross targets.
2) set default value of exact = false
3) change places which calculate its own growth to use the default argument

The usual intent of a default argument is to supply a value the function
is the most commonly invoked with.   But in this case it looks like it's
the opposite: most of the callers (hundreds) provide the non-default
value (true) and only a handful make use of the default.  I feel I must
be  missing something.  What is it?

You are right, but Richi wanted to make this transformation in more defensive way. I'm eventually planning to drop the explicit 'true' argument for most of the places
except selective scheduling and LTO streaming.

If it's just transitional on the way toward the usual approach
then that seems fine (although even then I can't say I understand
the rationale for going this circuitous route).

Thanks
Martin


I guess Richi can defend his strategy for this ;) ?

Martin


Martin


I would like to install first 1) and then wait some time before the rest is installed.

Thoughts?
Martin



Reply via email to