Cleanup #3977
closedDo a great stream operator const cleanup!
0%
Description
I just spent a little while trying to figure out why the compiler couldn't resolve operator<< (the stream operator) on a function whose operator<< function was included in the list of candidates. The calling function was const and the stream operator didn't mark it that way, was why. A quick grep indicates that a number of such sins remain in the code base. Clean them up!
Updated by Jos Collin almost 7 years ago
Greg,
Is this appears as an error during make ? Could you please show me an example of your Description ?
Thanks,
Jos Collin
Updated by Greg Farnum almost 7 years ago
Woah, blast from the past! :)
The issue described here is that we had a bunch of operator<< functions with non-const inputs. That precludes using them inside of other const contexts, which isn't a problem with building the existing source but makes ongoing development a bit more annoying.
Updated by Jos Collin almost 7 years ago
- Status changed from New to Fix Under Review
Fixed the instances inside src/ folder utmost. This fix avoids the following directories inside src: rocksdb/ zstd/ boost/ rapidjson/ googletest/ Beast/.
Updated by Jos Collin almost 7 years ago
- Status changed from Fix Under Review to In Progress
Updated by Jos Collin almost 7 years ago
- Status changed from In Progress to Resolved