From: Emil Dotchevski (emildotchevski_at_[hidden])
Date: 2021-08-03 01:46:27
On Mon, Aug 2, 2021 at 5:12 PM Marshall Clow via Boost <
> On Aug 2, 2021, at 5:01 PM, Emil Dotchevski via Boost <
> > Technically not a Beta bug fix but I think it would be nice if these
> > make it in the next release, both reported after the Beta lock:
> This one looks worrisome to me; thatâs a significant change.
> Letâs get the bots back up, look at the results, and talk tomorrow or
Perhaps misunderstanding? The bots are up, tests all pass:
They weren't passing before this commit because github had removed some gcc
versions from the base ubuntu image used by GHA, so now they need to be
To clarify, this change touches many files and looks scary but it is all
along these lines:
T const mag=sqrt<T>(m2);
Now we have:
T const mag=sqrt(m2);
The sqrt function template is defined in, and invoked from, namespace
boost::qvm. The difference is that now we're using ADL, which could bind
some other sqrt function (m2 may be of user-defined type).
The same is done for other math functions, e.g. sin, cos, etc.
> [ Iâm not against merging it, Iâm just concerned ]
Yeah, I'm on the edge for this one too. Seems simple and safe though. :)
> > (Issue 33).
> This one is fine. Go ahead.
I'd rather not merge this one if the other one isn't merged in also.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk