summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to 'src')
-rw-r--r--src/querycompleter.cpp63
-rw-r--r--src/querycompleter.h7
2 files changed, 63 insertions, 7 deletions
diff --git a/src/querycompleter.cpp b/src/querycompleter.cpp
index 45ef581..6bdd494 100644
--- a/src/querycompleter.cpp
+++ b/src/querycompleter.cpp
@@ -46,6 +46,22 @@ int tokenStart(const QString &text, int cursor)
return start;
}
+/// End of the token the cursor sits in, using the same boundary characters as
+/// tokenStart plus ')'. The token must extend past the cursor: a range
+/// separator to the right of the cursor decides which bound is being edited,
+/// so truncating the token at the cursor would hide it.
+int tokenEnd(const QString &text, int cursor)
+{
+ int end = cursor;
+ while (end < text.size()) {
+ const QChar c = text.at(end);
+ if (c.isSpace() || c == QLatin1Char('(') || c == QLatin1Char(')'))
+ break;
+ ++end;
+ }
+ return end;
+}
+
} // namespace
CompletionContext completionContext(const QString &text, int cursor)
@@ -59,21 +75,54 @@ CompletionContext completionContext(const QString &text, int cursor)
return ctx; // kind stays None
const int start = tokenStart(text, cursor);
- const QString token = text.mid(start, cursor - start);
+ const int end = tokenEnd(text, cursor);
+ const QString token = text.mid(start, end - start);
+
+ // Everything the user has typed up to the caret. Candidates are matched
+ // against this, never against text still sitting to the right of it.
+ const QString typed = text.mid(start, cursor - start);
const int colon = token.indexOf(QLatin1Char(':'));
- if (colon < 0) {
+ if (colon < 0 || cursor <= start + colon) {
+ // No prefix yet, or the caret is still inside the keyword itself.
ctx.kind = CompletionContext::Prefix;
- ctx.stem = token;
+ ctx.stem = typed;
ctx.replaceFrom = start;
- ctx.replaceLength = token.size();
+ ctx.replaceLength = typed.size();
return ctx;
}
ctx.kind = CompletionContext::Value;
ctx.prefix = token.left(colon).toLower();
- ctx.stem = token.mid(colon + 1);
- ctx.replaceFrom = start + colon + 1;
- ctx.replaceLength = ctx.stem.size();
+
+ const QString value = token.mid(colon + 1);
+ const int valueStart = start + colon + 1;
+
+ // A range is two independent values. Complete whichever side the cursor
+ // is in, leaving the other untouched.
+ const int separator = value.indexOf(QStringLiteral(".."));
+ if (separator < 0) {
+ ctx.stem = text.mid(valueStart, cursor - valueStart);
+ ctx.replaceFrom = valueStart;
+ ctx.replaceLength = ctx.stem.size();
+ return ctx;
+ }
+
+ ctx.allowRangeEntries = false;
+
+ const int cursorInValue = cursor - valueStart;
+ if (cursorInValue <= separator) {
+ // stem uses the cursor offset while replaceLength covers the whole
+ // side: matching runs on what has been typed so far, but accepting
+ // replaces the entire bound, so completing mid-word leaves no tail.
+ ctx.stem = value.left(cursorInValue);
+ ctx.replaceFrom = valueStart;
+ ctx.replaceLength = separator;
+ } else {
+ const int upperStart = separator + 2;
+ ctx.stem = value.mid(upperStart, cursorInValue - upperStart);
+ ctx.replaceFrom = valueStart + upperStart;
+ ctx.replaceLength = value.size() - upperStart;
+ }
return ctx;
}
diff --git a/src/querycompleter.h b/src/querycompleter.h
index d9635c0..e2a983b 100644
--- a/src/querycompleter.h
+++ b/src/querycompleter.h
@@ -44,6 +44,13 @@ struct CompletionContext
/// being completed, so accepting never disturbs neighbouring text.
int replaceFrom = 0;
int replaceLength = 0;
+
+ /// Whether candidates that are themselves ranges may be offered.
+ ///
+ /// The relative date entries ("1week..") are complete open-ended ranges.
+ /// Offering one inside an existing range yields date:1week....today, which
+ /// is malformed, so they are withheld once a range is underway.
+ bool allowRangeEntries = true;
};
/// Decides what the cursor position implies about completion.