Skip to content

Commit 1922ad8

Browse files
committed
reuse GetMaxNumberOfCompletionsToAdd
1 parent 8dc824d commit 1922ad8

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

lldb/include/lldb/Utility/CompletionRequest.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -167,12 +167,12 @@ class CompletionRequest {
167167
/// Returns true if the maximum number of completions has not been reached
168168
/// yet, hence we should keep adding completions.
169169
bool ShouldAddCompletions() const {
170-
return m_result.GetNumberOfResults() < m_max_return_elements;
170+
return GetMaxNumberOfCompletionsToAdd() > 0;
171171
}
172172

173173
/// Returns the maximum number of completions that need to be added
174174
/// until reaching the maximum
175-
size_t GetMaxNumberOfResultsToAdd() const {
175+
size_t GetMaxNumberOfCompletionsToAdd() const {
176176
const size_t number_of_results = m_result.GetNumberOfResults();
177177
if (number_of_results >= m_max_return_elements)
178178
return 0;

lldb/source/Commands/CommandCompletions.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ class SourceFileCompleter : public Completer {
167167
m_matching_files.AppendIfUnique(context.comp_unit->GetPrimaryFile());
168168
}
169169
}
170-
return m_matching_files.GetSize() >= m_request.GetMaxNumberOfResultsToAdd()
170+
return m_matching_files.GetSize() >= m_request.GetMaxNumberOfCompletionsToAdd()
171171
? Searcher::eCallbackReturnStop
172172
: Searcher::eCallbackReturnContinue;
173173
}
@@ -232,7 +232,7 @@ class SymbolCompleter : public Completer {
232232

233233
// Now add the functions & symbols to the list - only add if unique:
234234
for (const SymbolContext &sc : sc_list) {
235-
if (m_match_set.size() >= m_request.GetMaxNumberOfResultsToAdd())
235+
if (m_match_set.size() >= m_request.GetMaxNumberOfCompletionsToAdd())
236236
break;
237237

238238
ConstString func_name = sc.GetFunctionName(Mangled::ePreferDemangled);
@@ -244,7 +244,7 @@ class SymbolCompleter : public Completer {
244244
m_match_set.insert(func_name);
245245
}
246246
}
247-
return m_match_set.size() >= m_request.GetMaxNumberOfResultsToAdd()
247+
return m_match_set.size() >= m_request.GetMaxNumberOfCompletionsToAdd()
248248
? Searcher::eCallbackReturnStop
249249
: Searcher::eCallbackReturnContinue;
250250
}

0 commit comments

Comments
 (0)