mirrors/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity.git synced 2025-01-24 02:12:09 -05:00
Code Issues Projects Releases Packages Wiki Activity

LibLine: Convert strings to time_t using signed conversion methods

Browse source 
The time_t type itself is signed, and the result of this conversion is
always stored as time_t, so let's not use to_uint<>.
This commit is contained in:
Timothy Flynn 2022-12-15 17:27:38 -05:00 • committed by Andreas Kling
parent d4ecdf3ced
commit a67ad665b1
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibLine/Editor.cpp
View file

@ -259,7 +259,7 @@ bool Editor::load_history(DeprecatedString const& path)
auto hist = StringView { data.data(), data.size() };
for (auto& str : hist.split_view("\n\n"sv)) {
auto it = str.find("::"sv).value_or(0);
auto time = str.substring_view(0, it).to_uint<time_t>().value_or(0);
auto time = str.substring_view(0, it).to_int<time_t>().value_or(0);
auto string = str.substring_view(it == 0 ? it : it + 2);
m_history.append({ string, time });
}
@ -320,7 +320,7 @@ bool Editor::save_history(DeprecatedString const& path)
file->line_begin(), file->line_end(), m_history.begin(), m_history.end(), final_history,
[](StringView str) {
auto it = str.find("::"sv).value_or(0);
auto time = str.substring_view(0, it).to_uint<time_t>().value_or(0);
auto time = str.substring_view(0, it).to_int<time_t>().value_or(0);
auto string = str.substring_view(it == 0 ? it : it + 2);
return HistoryEntry { string, time };
},