input: prioritize builtin bindings matching longer key sequences

[guidocella]

This fixes 2 different bugs:

• mp.add_key_binding('c', ...) taking priority over the builtin g-c binding.

This follows up 994a08f5a7 which fixed this within the same input section. This fixes it across different input sections.

• mp.add_key_binding('g-c', ...) not taking priority over a c binding defined in input.conf.

This happened because is_builtin of bindings added with mp.add_key_binding is true though they're not actually builtin.

[github-actions[bot]]

Download the artifacts for this pull request:

Windows

* [mpv-i686-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/1511540333.zip) * [mpv-x86_64-windows-msvc](https://nightly.link/mpv-player/mpv/actions/artifacts/1511563231.zip) * [mpv-x86_64-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/1516082015.zip)

macOS

* [mpv-macos-14-arm](https://nightly.link/mpv-player/mpv/actions/artifacts/1511538411.zip) * [mpv-macos-12-intel](https://nightly.link/mpv-player/mpv/actions/artifacts/1511539071.zip) * [mpv-macos-13-intel](https://nightly.link/mpv-player/mpv/actions/artifacts/1511539447.zip)

[kasper93]

mp.add_key_binding('g', ...) still overrides g-*

[guidocella]

Why wouldn't it?

[kasper93]

Because I have to put g ignore or some other workaround, even if I have g-* in input.conf.

[guidocella]

This is the documented and expected behavior: "If a or a-b or b are already bound, this will run the first command that matches, and the multi-key command will never be called. Intermediate keys can be remapped to ignore in order to avoid this issue. The maximum number of (non-modifier) keys for combinations is currently 4."

[kasper93]

Fair, I guess.