On Friday, 25 April 2014 at 11:11:18 UTC, Atila Neves wrote:
On Friday, 25 April 2014 at 10:20:47 UTC, Rikki Cattermole wrote:
On Friday, 25 April 2014 at 10:02:45 UTC, Dicebot wrote:
On Friday, 25 April 2014 at 09:45:06 UTC, Rikki Cattermole wrote:
Also when using things like __LINE__ keep them to template args, as they are inferred to the initiation if possible.

This is antipattern. Default function arguments for __LINE__ and __FILE__ are also evaluated at call site. Moving this to template parameter creates huge amount of template bloat and must be used only if there is no other way around (that usually implies variadic arguments)

True in this specific case it might be over the top.

It was a template parameter before anyway. Also, this is for acceptance/feature/integration testing, so I doubt anyone would care how much bloat it generates as long as it gets the job done.

It will hurt build-times, so it's worth avoiding.

Reply via email to