check value correctly with xkb_state_layout_index_is_active()

[ikalco]

Describe your PR, what does it fix/add?

fixes
language onevent with error #5923

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

xkb_state_layout_index_is_active()
Test whether a layout is active in a given keyboard state by index.

Returns
1 if the layout is active, 0 if it is not. If the layout index is not valid in the keymap, returns -1.

in cpp -1 is truthy and Hyprland uses this

std::string IKeyboard::getActiveLayout() {
    const auto WLRKB      = wlr();
    const auto KEYMAP     = WLRKB->keymap;
    const auto STATE      = WLRKB->xkb_state;
    const auto LAYOUTSNUM = xkb_keymap_num_layouts(KEYMAP);

    for (uint32_t i = 0; i < LAYOUTSNUM; ++i) {
        if (xkb_state_layout_index_is_active(STATE, i, XKB_STATE_LAYOUT_EFFECTIVE)) {
            const auto LAYOUTNAME = xkb_keymap_layout_get_name(KEYMAP, i);

            if (LAYOUTNAME)
                return std::string(LAYOUTNAME);
            return "error";
        }
    }

    return "none";
}

Not sure how it would get an invalid index though

Is it ready for merging, or does it need work?

Yes

[vaxerski]

ah yeah I should rtfm indeed

[vaxerski]

thanks!