mirror of
https://github.com/mozilla-firefox/firefox
synced 2026-08-12 12:35:33 +00:00
[[likely]] and [[unlikely]] are standard C++, allow us to reduce the number of headers included (Likely.h), and simplify the if statement's expressions. https://en.cppreference.com/w/cpp/language/attributes/likely Differential Revision: https://phabricator.services.mozilla.com/D299297
57 lines
1.8 KiB
C++
57 lines
1.8 KiB
C++
/* This Source Code Form is subject to the terms of the Mozilla Public
|
|
* License, v. 2.0. If a copy of the MPL was not distributed with this file,
|
|
* You can obtain one at http://mozilla.org/MPL/2.0/. */
|
|
|
|
#ifndef mozilla_dom_ProxyHandlerUtils_h
|
|
#define mozilla_dom_ProxyHandlerUtils_h
|
|
|
|
#include "js/Id.h"
|
|
#include "js/Object.h" // JS::GetClass
|
|
#include "js/PropertyDescriptor.h"
|
|
#include "js/String.h" // JS::AtomToLinearString, JS::GetLinearString{CharAt,Length}
|
|
#include "js/TypeDecls.h"
|
|
#include "jsfriendapi.h" // js::StringIsArrayIndex
|
|
#include "mozilla/TextUtils.h"
|
|
|
|
namespace mozilla::dom {
|
|
|
|
extern jsid s_length_id;
|
|
|
|
// A return value of UINT32_MAX indicates "not an array index". Note, in
|
|
// particular, that UINT32_MAX itself is not a valid array index in general.
|
|
inline uint32_t GetArrayIndexFromId(JS::Handle<jsid> id) {
|
|
// Much like js::IdIsIndex, except with a fast path for "length" and another
|
|
// fast path for starting with a lowercase ascii char. Is that second one
|
|
// really needed? I guess it is because StringIsArrayIndex is out of line...
|
|
// as of now, use id.get() instead of id otherwise operands mismatch error
|
|
// occurs.
|
|
if (id.isInt()) [[likely]] {
|
|
return id.toInt();
|
|
}
|
|
if (id.get() == s_length_id) [[likely]] {
|
|
return UINT32_MAX;
|
|
}
|
|
if (!id.isAtom()) [[unlikely]] {
|
|
return UINT32_MAX;
|
|
}
|
|
|
|
JSLinearString* str = JS::AtomToLinearString(id.toAtom());
|
|
if (JS::GetLinearStringLength(str) == 0) [[unlikely]] {
|
|
return UINT32_MAX;
|
|
}
|
|
|
|
char16_t firstChar = JS::GetLinearStringCharAt(str, 0);
|
|
if (IsAsciiLowercaseAlpha(firstChar)) [[likely]] {
|
|
return UINT32_MAX;
|
|
}
|
|
|
|
uint32_t i;
|
|
return js::StringIsArrayIndex(str, &i) ? i : UINT32_MAX;
|
|
}
|
|
|
|
inline bool IsArrayIndex(uint32_t index) { return index < UINT32_MAX; }
|
|
|
|
} // namespace mozilla::dom
|
|
|
|
#endif /* mozilla_dom_ProxyHandlerUtils_h */
|