Remove ConvenientIterator::ref usage from oldcpp codebase.
Review Request #124237 - Created July 2, 2015 and submitted
This requires some changes, as we cannot take the ptr address of the macro anymore. Instead, we use them as value classes everywhere. This may potentially degrade performance, but I don't have the time to measure it right now with a release build and compare the impact.
TBH, I'm fine either way. Technically, it shouldn't make a difference whether we're using it as a reference or as a pointer, which makes me think we could be introducing a bug somewhere because we just changed the semantics, so if the tests pass, ship it.
Now, we should really be focusing in kdev-clang, as we're not going to fix oldcpp, ever.