-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Convert TrailingZerosTruncatingPolicy into an enum class #17186
Convert TrailingZerosTruncatingPolicy into an enum class #17186
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
2828448
to
c563ce3
Compare
https://bugs.webkit.org/show_bug.cgi?id=260863 rdar://114628958 Reviewed by Tim Nguyen. * Source/JavaScriptCore/runtime/NumberPrototype.cpp: (JSC::JSC_DEFINE_HOST_FUNCTION): * Source/WTF/wtf/text/StringConcatenateNumbers.h: (WTF::FormattedNumber::fixedPrecision): * Source/WTF/wtf/text/WTFString.cpp: (WTF::String::numberToStringFixedPrecision): * Source/WTF/wtf/text/WTFString.h: * Source/WebCore/accessibility/AccessibilityObjectInterface.h: (WebCore::AXCoreObject::value): * Source/WebCore/page/cocoa/ResourceUsageOverlayCocoa.mm: (WebCore::ResourceUsageOverlay::platformDraw): * Tools/TestWebKitAPI/Tests/WTF/StringConcatenate.cpp: (TestWebKitAPI::TEST): * Tools/WebKitTestRunner/InjectedBundle/atspi/AccessibilityUIElementAtspi.cpp: (WTR::attributesOfElement): Canonical link: https://commits.webkit.org/267447@main
c563ce3
to
137065a
Compare
Committed 267447@main (137065a): https://commits.webkit.org/267447@main Reviewed commits have been landed. Closing PR #17186 and removing active labels. |
@@ -573,7 +573,6 @@ inline String operator"" _str(const UChar* characters, size_t length) | |||
|
|||
} // namespace WTF | |||
|
|||
using WTF::KeepTrailingZeros; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WTF design has been to use these using
statements rather than WTF::
prefixes at site of use. I know this is unconventional, but itβs the pattern we use in most cases across the library. To do that we would have using WTF::TrailingZerosPolicy
here and fewer (maybe no) WTF::
prefixes elsewhere.
Iβd be open to changing the WTF approach, and if weβd agreed that weβd not do it for anything new that would be OK with me.
But if I was making this change I would have stayed consistent with the approach and done the above.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair! Created #17375.
137065a
c563ce3
π§ͺ jsc-armv7-tests