Return a FillResult struct from RunStyles::FillRange instead of modifying

arguments as that is clumsy when converting types.

1 files changed, 4 insertions, 4 deletions

diff --git a/src/Document.cxx b/src/Document.cxx
index cb2892c96..e0373a752 100644
--- a/src/Document.cxx
+++ b/src/Document.cxx
@@ -2243,11 +2243,11 @@ void SCI_METHOD Document::DecorationSetCurrentIndicator(int indicator) {
 }
 
 void SCI_METHOD Document::DecorationFillRange(Sci_Position position, int value, Sci_Position fillLength) {
-	Sci::Position sciPosition = static_cast<Sci::Position>(position);
-	Sci::Position sciFillLength = static_cast<Sci::Position>(fillLength);
-	if (decorations.FillRange(sciPosition, value, sciFillLength)) {
+	const FillResult<Sci::Position> fr = decorations.FillRange(
+		static_cast<Sci::Position>(position), value, static_cast<Sci::Position>(fillLength));
+	if (fr.changed) {
 		const DocModification mh(SC_MOD_CHANGEINDICATOR | SC_PERFORMED_USER,
-							sciPosition, sciFillLength);
+							fr.position, fr.fillLength);
 		NotifyModified(mh);
 	}
 }