LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-01-06 02:55:49 +03:00
Code Issues Packages Projects Releases Wiki Activity

Assistant+Spreadsheet: Convert usages of Value::TDSWOSE to Value::TSWOSE

Browse Source
This commit is contained in:
Timothy Flynn 2023-02-12 22:09:04 -05:00 committed by Linus Groh
parent 527edb55ad
commit a2f1b80c36
Notes: sideshowbarker 2024-07-17 21:11:12 +09:00 
Author: https://github.com/trflynn89
Commit: https://github.com/SerenityOS/serenity/commit/a2f1b80c36
Pull-request: https://github.com/SerenityOS/serenity/pull/17461
Reviewed-by: https://github.com/linusg
4 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Userland/Applications/Assistant/Providers.cpp
View File

@ -105,7 +105,7 @@ void CalculatorProvider::query(DeprecatedString const& query, Function<void(Nonn
if (!result.is_number()) {
calculation = "0";
} else {
calculation = result.to_deprecated_string_without_side_effects();
calculation = result.to_string_without_side_effects().release_value_but_fixme_should_propagate_errors().to_deprecated_string();
}
NonnullRefPtrVector<Result> results;

2
Userland/Applications/Spreadsheet/Cell.cpp
View File

@ -41,7 +41,7 @@ void Cell::set_data(JS::Value new_data)
StringBuilder builder;
builder.append(new_data.to_deprecated_string_without_side_effects());
builder.append(new_data.to_string_without_side_effects().release_value_but_fixme_should_propagate_errors());
m_data = builder.to_deprecated_string();
m_evaluated_data = move(new_data);

6
Userland/Applications/Spreadsheet/Spreadsheet.cpp
View File

@ -64,7 +64,7 @@ Sheet::Sheet(Workbook& workbook)
if (result.is_error()) {
warnln("Spreadsheet: Failed to run runtime code:");
auto thrown_value = *result.throw_completion().value();
warn("Threw: {}", thrown_value.to_deprecated_string_without_side_effects());
warn("Threw: {}", MUST(thrown_value.to_string_without_side_effects()));
if (thrown_value.is_object() && is<JS::Error>(thrown_value.as_object())) {
auto& error = static_cast<JS::Error const&>(thrown_value.as_object());
warnln(" with message '{}'", error.get_without_side_effects(interpreter().vm().names.message));
@ -562,7 +562,7 @@ JsonObject Sheet::to_json() const
auto json = interpreter().realm().global_object().get_without_side_effects("JSON");
auto stringified_or_error = JS::call(vm, json.as_object().get_without_side_effects("stringify").as_function(), json, it.value->evaluated_data());
VERIFY(!stringified_or_error.is_error());
data.set("value", stringified_or_error.release_value().to_deprecated_string_without_side_effects());
data.set("value", stringified_or_error.release_value().to_string_without_side_effects().release_value_but_fixme_should_propagate_errors().to_deprecated_string());
} else {
data.set("value", it.value->data());
}
@ -693,7 +693,7 @@ JsonObject Sheet::gather_documentation() const
if (!doc.is_string())
return;
JsonParser parser(doc.to_deprecated_string_without_side_effects());
JsonParser parser(doc.to_string_without_side_effects().release_value_but_fixme_should_propagate_errors());
auto doc_object = parser.parse();
if (!doc_object.is_error())

4
Userland/Applications/Spreadsheet/SpreadsheetModel.cpp
View File

@ -33,7 +33,7 @@ GUI::Variant SheetModel::data(const GUI::ModelIndex& index, GUI::ModelRole role)
auto message = object.get_without_side_effects("message");
auto error = message.to_deprecated_string(vm);
if (error.is_throw_completion())
builder.append(message.to_deprecated_string_without_side_effects());
builder.append(message.to_string_without_side_effects().release_value_but_fixme_should_propagate_errors());
else
builder.append(error.release_value());
return builder.to_deprecated_string();
@ -120,7 +120,7 @@ GUI::Variant SheetModel::data(const GUI::ModelIndex& index, GUI::ModelRole role)
auto& error = static_cast<JS::Error&>(object);
auto const& trace = error.traceback();
StringBuilder builder;
builder.appendff("{}\n", error.get_without_side_effects(object.vm().names.message).to_deprecated_string_without_side_effects());
builder.appendff("{}\n", error.get_without_side_effects(object.vm().names.message).to_string_without_side_effects().release_value_but_fixme_should_propagate_errors());
for (auto const& frame : trace.in_reverse()) {
if (frame.source_range.filename().contains("runtime.js"sv)) {
if (frame.function_name == "<unknown>")