nonius icon indicating copy to clipboard operation
nonius copied to clipboard

Directly construct stringstream without copy/move

Open tbleher opened this issue 7 years ago • 4 comments

The libstdc++ bundled with G++ 4.8.1 on OpenSUSE 13.1 does not have a move constructor for std::stringstream, and copying is forbidden. Work around this by constructing the std::stringstream directly.

tbleher avatar Jun 19 '18 05:06 tbleher

GCC 4.8.1 has been released May 31, 2013. That's more than 5 years ago, an era in compilers development. Does it really make sense to contaminate the style of the library with such minor fix?

michalfita avatar Oct 22 '18 21:10 michalfita

The change is so small that I'd hardly call it a style contamination, would you?

mquinson avatar Oct 22 '18 21:10 mquinson

It is small. However, in my personal opinion not worth the hassle. You already solved the problem for yourself, highly unlikely ever going to use never version of nonius (last release two years ago).

I understand the importance of it for you, but looking at other PR's the average time to get merged being about two years I would abandon the hope.

michalfita avatar Oct 22 '18 21:10 michalfita

I understand that people are reluctant to support old compilers, but the fact is that these systems are still in use (otherwise I would not report that issue). I can understand that people do not want to chase bug reports for old systems, but this is different: this is a very simple fix for a concrete problem that can be applied immediately. Honestly I cannot see any "contamination". Surely the authors of nonius published nonius in the hope that people would use it. This patch helps people use the library. What's gained by not accepting this patch?

tbleher avatar Oct 26 '18 21:10 tbleher