修改诊断描述和范围

Author: CppCXY

CodeService/src/FormatElement/AlignmentElement.cpp

@@ -33,6 +33,6 @@ void AlignmentElement::Diagnosis(DiagnosisContext& ctx, int position, FormatElem
 	if (character != _alignmentPosition)
 	{
 		ctx.PushDiagnosis(format("'=' should align to character {}", _alignmentPosition),
-		                  TextRange(nextOffset, nextOffset));
+		                  TextRange(nextOffset, nextOffset + 1));
 	}
 }

CodeService/src/FormatElement/IndentElement.cpp

@@ -35,8 +35,12 @@ void IndentElement::Diagnosis(DiagnosisContext& ctx, int position, FormatElement
 			auto character = ctx.GetColumn(range.StartOffset);
 			if (character != static_cast<int>(ctx.GetCurrentIndent()))
 			{
-				ctx.PushDiagnosis(format("Incorrect indentation {}", character),
-				                  TextRange(range.StartOffset, range.StartOffset));
+				auto line = ctx.GetLine(range.StartOffset);
+				ctx.PushDiagnosis(format("incorrect indentation {}, here need {} indent",
+				                         character, ctx.GetCurrentIndent()),
+				                  LuaDiagnosisPosition(line, 0),
+				                  LuaDiagnosisPosition(line, character)
+				);
 			}
 		}
 

CodeService/src/FormatElement/KeepBlankElement.cpp

@@ -34,7 +34,7 @@ void KeepBlankElement::Diagnosis(DiagnosisContext& ctx, int position, FormatElem
 	{
 		if (nextOffset - lastOffset - 1 != _blank)
 		{
-			ctx.PushDiagnosis(format("must keep {} space", _blank), TextRange(lastOffset, nextOffset));
+			ctx.PushDiagnosis(format("here need keep {} space", _blank), TextRange(lastOffset, nextOffset));
 		}
 	}
 }

CodeService/src/FormatElement/KeepElement.cpp

@@ -54,7 +54,7 @@ void KeepElement::Diagnosis(DiagnosisContext& ctx, int position, FormatElement&
 	{
 		if (nextOffset - lastOffset - 1 != _keepBlank)
 		{
-			ctx.PushDiagnosis(format("must keep {} space", _keepBlank), TextRange(lastOffset, nextOffset));
+			ctx.PushDiagnosis(format("here need keep {} space", _keepBlank), TextRange(lastOffset, nextOffset));
 		}
 	}
 }

CodeService/src/FormatElement/KeepLineElement.cpp

@@ -52,7 +52,7 @@ void KeepLineElement::Diagnosis(DiagnosisContext& ctx, int position, FormatEleme
 
 		if (_line != (nextElementLine - lastElementLine - 1))
 		{
-			ctx.PushDiagnosis(format("need {} line", _line), TextRange(lastElementOffset, nextElementOffset));
+			ctx.PushDiagnosis(format("here need keep {} line", _line), TextRange(lastElementOffset, nextElementOffset));
 		}
 	}
 }

CodeService/src/FormatElement/MinLineElement.cpp

@@ -57,6 +57,6 @@ void MinLineElement::Diagnosis(DiagnosisContext& ctx, int position, FormatElemen
 
 	if (_line > (nextElementLine - lastElementLine - 1))
 	{
-		ctx.PushDiagnosis(format("need at least {} line", _line), TextRange(lastOffset, nextOffset));
+		ctx.PushDiagnosis(format("here need at least {} line", _line), TextRange(lastOffset, nextOffset));
 	}
 }