From 40eab55e7d4e757d6b3fcf18d26db8c6c40bbc25 Mon Sep 17 00:00:00 2001 From: Linus Groh Date: Mon, 22 Mar 2021 22:56:47 +0100 Subject: LibJS: Remove as_size_t() Just like to_size_t() - which was already removed in f369229 - this is non-standard, use to_length() instead. One remaining use was removed, and I'm glad it's gone. :^) --- Userland/Applications/Spreadsheet/JSIntegration.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'Userland/Applications/Spreadsheet') diff --git a/Userland/Applications/Spreadsheet/JSIntegration.cpp b/Userland/Applications/Spreadsheet/JSIntegration.cpp index b62c89298e..b62a5985e9 100644 --- a/Userland/Applications/Spreadsheet/JSIntegration.cpp +++ b/Userland/Applications/Spreadsheet/JSIntegration.cpp @@ -449,7 +449,9 @@ JS_DEFINE_NATIVE_FUNCTION(WorkbookObject::sheet) return JS::Value(&sheet.global_object()); } } else { - auto index = name_value.as_size_t(); + auto index = name_value.to_length(global_object); + if (vm.exception()) + return {}; if (index < workbook.sheets().size()) return JS::Value(&workbook.sheets()[index].global_object()); } -- cgit v1.2.3