Skip to content

Commit

Permalink
chore: address review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
mcdurdin committed Dec 4, 2024
1 parent 4c3b213 commit 5ec2ce7
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 15 deletions.
2 changes: 1 addition & 1 deletion core/docs/api/changes.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ title: Changes - Keyman Core API
[km_core_keyboard_attrs]: keyboards#km_core_keyboard_attrs "km_core_keyboard_attrs struct"
[km_core_keyboard_key]: keyboards#km_core_keyboard_key "km_core_keyboard_key struct"
[km_core_keyboard_imx]: keyboards#km_core_keyboard_imx "km_core_keyboard_imx struct"
[km_core_keyboard_load]: keyboards#km_core_keyboard_load "km_core_keyboard_load function"
[km_core_keyboard_load_from_blob]: keyboards#km_core_keyboard_load_from_blob "km_core_keyboard_load_from_blob function"
[km_core_keyboard_dispose]: keyboards#km_core_keyboard_dispose "km_core_keyboard_dispose function"
[km_core_keyboard_get_attrs]: keyboards#km_core_keyboard_get_attrs "km_core_keyboard_get_attrs function"
[km_core_keyboard_get_key_list]: keyboards#km_core_keyboard_get_key_list "km_core_keyboard_get_key_list function"
Expand Down
4 changes: 2 additions & 2 deletions core/docs/api/keyboards.md
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,8 @@ typedef struct {

## Description

Parse and load keyboard from the supplied blob and size and a pointer to the
loaded keyboard into the out paramter.
Parse and load a keyboard from the supplied blob and return a pointer to the
loaded keyboard in the out parameter.

## Specification

Expand Down
12 changes: 0 additions & 12 deletions core/tests/unit/km_core_keyboard_api.tests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -10,18 +10,6 @@

km::core::path test_dir;

// #if defined(__GNUC__) || defined(__clang__)
// #define PRAGMA(X) _Pragma(#X)
// #define DISABLE_WARNING_PUSH PRAGMA(GCC diagnostic push)
// #define DISABLE_WARNING_POP PRAGMA(GCC diagnostic pop)
// #define DISABLE_WARNING(W) PRAGMA(GCC diagnostic ignored #W)
// #define DISABLE_WARNING_DEPRECATED_DECLARATIONS DISABLE_WARNING(-Wdeprecated-declarations)
// #else
// #define DISABLE_WARNING_PUSH
// #define DISABLE_WARNING_POP
// #define DISABLE_WARNING_DEPRECATED_DECLARATIONS
// #endif

class KmCoreKeyboardApiTests : public testing::Test {
protected:
km_core_keyboard* keyboard = nullptr;
Expand Down

0 comments on commit 5ec2ce7

Please sign in to comment.