auto: increment master version to 17.0.221

fix(core): check for null termination

Determine the best way to check for null termination all compilers

fix(core): set_if_needed updates an empty cached context

Co-authored-by: Marc Durdin <marc@durdin.net>

In the case when the cached context had been cleared the
km_core_state_context_set_if_needed call would just compare
the null terminations of both strings and not set the cached
context to the application context.

auto: increment master version to 17.0.220

fix(web): Also move source map

fix(web): Fix attachment-api tests

fix(developer): projects 2.0 internal path enumeration 🦕

fix(developer): path separator for kmc-package

Previously we renamed `dom-keyboard-loader.mjs` but forgot to move
the source map along.

auto: increment master version to 17.0.219

feat(developer): ldml: err/hint on illegal/pua chars 🙀 

- use constants for PUA

For: #9446

Co-authored-by: Marc Durdin <marc@durdin.net>

auto: increment master version to 17.0.218

chore(core): Add test keyboard for text selection tests

chore(linux): Update debian changelog 🍒

fix(core): memory management of options in action struct

feat(developer): warn on usage of virtual keys in rule output

Fixes #10067.

Management of memory for persisted options was wrong in the action
struct, as the members key and value would be freed immediately after
being added to the temporary vector (because the vector was of the
struct rather than of the class).

Given the struct is a C struct, we need the memory management to be
explicit, so we now release() each option into the vector as we create
it, which means that its member values will not be freed when the option
is then immediately deleted. (This allows us to use the initial copy of
the members of option that option() constructor does.)

Added the release() function as that was a relatively clear way of
indicating that the contents of the structure are now owned by the
caller, following the pattern from std::unique_ptr.

Finally, the unit test for persisted options was in the action_api.cpp
test module, but it was never called, so this was not being tested. Now
it is.

auto: increment master version to 17.0.217

Part of #9073.

Fixes #10027.

Fixes #10059.

Use of the unsupported and undocumented virtual key output, that doesn't
work in recent Keyman versions, at all, now results in a build warning.

Only a warning, because it did kinda work in old versions of Keyman.

fix(web): app/webview did not clear deadkeys on context-reset

chore(core): remove vkey output from kmx processor

chore(windows): remove unused globals relating to old keyboard debugging

chore(windows): remove legacy LoadKeyboard