https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66852
--- Comment #9 from Jeffrey Walton <noloader at gmail dot com> --- (In reply to Jonathan Wakely from comment #8) > The code in algparam.h is just disgusting. AssignFromHelperClass binds a > reference to NULL just to default-construct a temporary of some type, then > binds a const-reference to that temporary, then casts away const to modify > the value of that temporary. What seems to be a "VC60 workaround" introduces > undefined behaviour by trying to be far too clever. Apparently also too > clever to use compiler warnings. Lol... Yeah, some of the legacy code is awful. I'm not throwing stones because the library supported so many compilers and IDEs back then. I think its time to start cleaning up some of the cruft. I'm going to cite this comment when I propose some of the changes.
