chore(core): rename keyboardprocessor.h to keyman_core_api.h

[mcdurdin]

Fixes #9721.

Also tweaks some documentation references to keyboardprocessor, but does not touch those relating to debian/control.

@keymanapp-test-bot skip

[keymanapp-test-bot]

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

• Developer
  • Keyman Developer
  • Compiler Regression Tests
  • kmcomp.zip
• Keyboards
  • Test Keyboards
• Linux
  • Keyman for Linux
• macOS
  • Keyman for macOS
• Web
  • KeymanWeb Test Home
• Windows
  • Keyman for Windows
  • FirstVoices Keyboards for Windows
  • Text Editor (32 bit)
  • Text Editor (64 bit)

[rc-swag]

Looks fine. I don't quite understand what is required but do we do need to make changes to the api_version.in or the name of the built library to match this comment fcitx/fcitx5-keyman#3 (comment).

[mcdurdin]

but do we do need to make changes to the api_version.in or the name of the built library to match this comment fcitx/fcitx5-keyman#3 (comment).

@ermshiperete thoughts?

[ermshiperete]

The names in linux/debian/libkmnkbp0-0.symbols should not have been renamed - that's the safety net that should trigger when we modify the API in ways that require a version change. Also, that file shows that we missed one: kbp_state_get_intermediate_context.

We should rename the library to libkmncore (or libkeymancore?) and probably also change the version.

#9733

[mcdurdin]

oops okay! Do you want to revert that specific change and put in place what should be done instead?

Also, kbp_state_get_intermediate_context was missed because it had the wrong name -- it was missing km_ prefix! So that's a double-oops.

[ermshiperete]

Ok, I'll take care of the .symbols file when I do #9733.

[keyman-server]

Changes in this pull request will be available for download in Keyman version 17.0.187-alpha