diff options
| author | Johannes Altmanninger <aclopte@gmail.com> | 2022-07-19 11:26:56 +0200 |
|---|---|---|
| committer | Johannes Altmanninger <aclopte@gmail.com> | 2022-07-28 15:10:28 +0200 |
| commit | 5aa02411242468963b8b445a4977f0f4884eba4a (patch) | |
| tree | 53a8ff4d1216ec067c968ff7e822a4be2958931b /src/shared_string.cc | |
| parent | 23fcf771608fe48d548930fe0097286fbace1ef4 (diff) | |
Use and extract functions for completing scope arguments
Not really attached to this but it enables the next commit to use
menu() for completing scopes.
This refactoring is possible because we always have
params[token_to_complete].length() == pos_in_token
---
Instead of three separate functions, I originally tried to add
template arguments to complete_scope(). That worked fine with g++
12.1 but clang 14.0 complained when wrapping a menu() around a
complete_scope() that relied on defaulted template arguments:
commands.cc:1087:20: error: no matching function for call to 'menu'
make_completer(menu(complete_scope), menu(complete_hooks), complete_nothing,
^~~~
commands.cc:116:6: note: candidate template ignored: couldn't infer template argument 'Completer'
auto menu(Completer completer)
^
Diffstat (limited to 'src/shared_string.cc')
0 files changed, 0 insertions, 0 deletions
