LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-21 02:08:12 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Remove as_size_t()

Browse Source 
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. :^)
This commit is contained in:
Linus Groh 2021-03-22 22:56:47 +01:00 committed by Andreas Kling
parent a1014d25de
commit 40eab55e7d
Notes: sideshowbarker 2024-07-18 21:07:57 +09:00 
Author: https://github.com/linusg
Commit: https://github.com/SerenityOS/serenity/commit/40eab55e7d4
Pull-request: https://github.com/SerenityOS/serenity/pull/5919
3 changed files with 3 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -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());
}

6
Userland/Libraries/LibJS/Runtime/Value.cpp
View File

@ -517,12 +517,6 @@ u32 Value::as_u32() const
return min((double)as_i32(), MAX_U32);
}
size_t Value::as_size_t() const
{
VERIFY(as_double() >= 0);
return min((double)as_i32(), MAX_ARRAY_LIKE_INDEX);
}
double Value::to_double(GlobalObject& global_object) const
{
auto number = to_number(global_object);

1
Userland/Libraries/LibJS/Runtime/Value.h
View File

@ -261,7 +261,6 @@ public:
i32 as_i32() const;
u32 as_u32() const;
size_t as_size_t() const;
String to_string(GlobalObject&, bool legacy_null_to_empty_string = false) const;
PrimitiveString* to_primitive_string(GlobalObject&);