Stop storing a pointer to the selection on LineLayout objects where it may have

a lifetime that is too long. Instead provide it as an argument to BreakFinder
where it is only needed during intialisation.

1 files changed, 8 insertions, 0 deletions

diff --git a/src/Selection.cxx b/src/Selection.cxx
index 4c2ab0508..b46dca890 100644
--- a/src/Selection.cxx
+++ b/src/Selection.cxx
@@ -226,10 +226,18 @@ SelectionRange &Selection::Range(size_t r) {
 	return ranges[r];
 }
 
+const SelectionRange &Selection::Range(size_t r) const {
+	return ranges[r];
+}
+
 SelectionRange &Selection::RangeMain() {
 	return ranges[mainRange];
 }
 
+const SelectionRange &Selection::RangeMain() const {
+	return ranges[mainRange];
+}
+
 SelectionPosition Selection::Start() const {
 	if (IsRectangular()) {
 		return rangeRectangular.Start();