Make ResultIterator operator* return non-const Result, this lowers the number of copies we need considerably

(bzr r2547.3.5)

2 files changed, 2 insertions, 2 deletions

diff --git a/UnityCore/ResultIterator.cpp b/UnityCore/ResultIterator.cpp
index d4a5d1971..aa569e952 100644
--- a/UnityCore/ResultIterator.cpp
+++ b/UnityCore/ResultIterator.cpp
@@ -129,7 +129,7 @@ ResultIterator ResultIterator::operator-(int count) const
 return tmp;
 }
 
-Result const& ResultIterator::operator*()
+Result& ResultIterator::operator*()
 {
 if (cache_invalidated_)
 iter_result_ = Result(model_, iter_, tag_);
diff --git a/UnityCore/ResultIterator.h b/UnityCore/ResultIterator.h
index a68aafb8f..402357390 100644
--- a/UnityCore/ResultIterator.h
+++ b/UnityCore/ResultIterator.h
@@ -82,7 +82,7 @@ public:
 {
 return !(lhs == rhs);
 }
- Result const& operator*();
+ Result& operator*();
 
 /* convenience methods */
 bool const IsLast();