0ct0pu5/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

LibKeyboard: Use ErrorOr<T> for CharacterMap::fetch_system_map()

Browse source
This commit is contained in:
Andreas Kling 2021-11-07 11:25:25 +01:00
parent e76b21a66f
commit c6cc0a88a1
Notes: sideshowbarker 2024-07-18 01:23:40 +09:00
2 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Userland/Libraries/LibKeyboard/CharacterMap.cpp
View file

@ -40,14 +40,13 @@ int CharacterMap::set_system_map()
return setkeymap(m_character_map_name.characters(), m_character_map_data.map, m_character_map_data.shift_map, m_character_map_data.alt_map, m_character_map_data.altgr_map, m_character_map_data.shift_altgr_map);
}
Result<CharacterMap, OSError> CharacterMap::fetch_system_map()
ErrorOr<CharacterMap> CharacterMap::fetch_system_map()
{
CharacterMapData map_data;
char keymap_name[50 + 1] = { 0 };
if (getkeymap(keymap_name, sizeof(keymap_name), map_data.map, map_data.shift_map, map_data.alt_map, map_data.altgr_map, map_data.shift_altgr_map) < 0) {
return OSError(errno);
}
if (getkeymap(keymap_name, sizeof(keymap_name), map_data.map, map_data.shift_map, map_data.alt_map, map_data.altgr_map, map_data.shift_altgr_map) < 0)
return Error::from_errno(errno);
return CharacterMap { keymap_name, map_data };
}

5
Userland/Libraries/LibKeyboard/CharacterMap.h
View file

@ -7,8 +7,7 @@
#pragma once
#ifndef KERNEL
# include <AK/OSError.h>
# include <AK/Result.h>
# include <AK/Error.h>
#endif
#include <AK/String.h>
#include <Kernel/API/KeyCode.h>
@ -24,7 +23,7 @@ public:
#ifndef KERNEL
int set_system_map();
static Result<CharacterMap, OSError> fetch_system_map();
static ErrorOr<CharacterMap> fetch_system_map();
#endif
u32 get_char(KeyEvent) const;