Remove dead code from the WebCore bindings, IDL converters, and Rust FFI wrappers

hawk.toml

@@ -651,107 +651,3 @@ item = "webcore::bake_response::SSRKind::Regular"
 kind = "enum_variant"
 level = "expect"
 reason = "external code table: SSR kind tag consumed by generated code"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_IN_TRANS"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_AUTOCOMMIT"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_NO_GOOD_INDEX_USED"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_NO_INDEX_USED"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_CURSOR_EXISTS"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_LAST_ROW_SENT"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_DB_DROPPED"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_NO_BACKSLASH_ESCAPES"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_METADATA_CHANGED"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_QUERY_WAS_SLOW"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_PS_OUT_PARAMS"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_STATUS_IN_TRANS_READONLY"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"
-
-[[override]]
-lint = "hawk::dead_public"
-crate = "bun_sql"
-item = "mysql::status_flags::StatusFlag::SERVER_SESSION_STATE_CHANGED"
-kind = "enum_variant"
-level = "expect"
-reason = "external code table: MySQL protocol status flag bits"

src/css/context.rs

@@ -20,7 +20,6 @@ pub struct SupportsEntry {
 pub enum DeclarationContext {
     None,
     StyleRule,
-    Keyframes,
     StyleAttribute,
 }
 

src/css/values/color_generated.rs

@@ -5,11 +5,6 @@ use super::color::{
     XYZd65,
 };
 
-/// Marker module so `use super::color_generated::generated_color_conversions;`
-/// in `color.rs` resolves. The actual `impl From<_> for _` blocks live at the
-/// parent scope (they apply crate-wide regardless of the module they sit in).
-pub mod generated_color_conversions {}
-
 // Helper: every generated conversion routes through exactly one intermediate
 // colorspace (`const xyz: Via = this.into(.Via); return xyz.into(.Dst);`).
 macro_rules! convert_via {

src/jsc/bindings/BakeAdditionsToGlobalObject.h

@@ -34,7 +34,6 @@ struct BakeAdditionsToGlobalObject {
         visitor.append(this->m_wrapComponent);
         visitor.append(this->m_asyncLocalStorageInstance);
 
-        this->m_bakeGetAsyncLocalStorage.visit(visitor);
         this->m_bakeEnsureAsyncLocalStorage.visit(visitor);
         this->m_bakeGetBundleNewRoute.visit(visitor);
         this->m_bakeGetNewRouteParams.visit(visitor);
@@ -47,11 +46,6 @@ struct BakeAdditionsToGlobalObject {
                 Bun::setupJSBakeResponseClassStructure(init);
             });
 
-        m_bakeGetAsyncLocalStorage.initLater(
-            [](const LazyProperty<JSGlobalObject, JSFunction>::Initializer& init) {
-                init.set(JSFunction::create(init.vm, init.owner, 0, String("bakeGetAsyncLocalStorage"_s), jsFunctionBakeGetAsyncLocalStorage, ImplementationVisibility::Public, NoIntrinsic));
-            });
-
         m_bakeEnsureAsyncLocalStorage.initLater(
             [](const LazyProperty<JSGlobalObject, JSFunction>::Initializer& init) {
                 init.set(JSFunction::create(init.vm, init.owner, 1, String("bakeSetAsyncLocalStorage"_s), jsFunctionBakeEnsureAsyncLocalStorage, ImplementationVisibility::Public, NoIntrinsic));
@@ -100,9 +94,6 @@ struct BakeAdditionsToGlobalObject {
 
     JSC::JSFunction* wrapComponent(JSGlobalObject*);
 
-    template<typename T>
-    using LazyPropertyOfGlobalObject = LazyProperty<JSGlobalObject, T>;
-
     JSC::JSObject* JSBakeResponseConstructor(const JSGlobalObject* global) const { return m_JSBakeResponseClassStructure.constructorInitializedOnMainThread(global); }
     JSC::Structure* JSBakeResponseStructure(const JSGlobalObject* global) const { return m_JSBakeResponseClassStructure.getInitializedOnMainThread(global); }
 
@@ -125,7 +116,6 @@ struct BakeAdditionsToGlobalObject {
     WriteBarrier<JSFunction> m_wrapComponent;
 
     WriteBarrier<Unknown> m_asyncLocalStorageInstance;
-    LazyProperty<JSGlobalObject, JSFunction> m_bakeGetAsyncLocalStorage;
     LazyProperty<JSGlobalObject, JSFunction> m_bakeEnsureAsyncLocalStorage;
     LazyProperty<JSGlobalObject, JSFunction> m_bakeGetBundleNewRoute;
     LazyProperty<JSGlobalObject, JSFunction> m_bakeGetNewRouteParams;

src/jsc/bindings/BunCommonStrings.h

@@ -16,7 +16,6 @@
     macro(httpCHECKOUT, "CHECKOUT") \
     macro(httpCONNECT, "CONNECT") \
     macro(httpCOPY, "COPY") \
-    macro(ConnectionWasClosed, "The connection was closed.") \
     macro(httpDELETE, "DELETE") \
     macro(httpGET, "GET") \
     macro(httpHEAD, "HEAD") \
@@ -35,8 +34,6 @@
     macro(httpMSEARCH, "M-SEARCH") \
     macro(httpNOTIFY, "NOTIFY") \
     macro(httpOPTIONS, "OPTIONS") \
-    macro(OperationFailed, "The operation failed.") \
-    macro(OperationTimedOut, "The operation timed out.") \
     macro(OperationWasAborted, "The operation was aborted.") \
     macro(httpPATCH, "PATCH") \
     macro(httpPOST, "POST") \
@@ -62,8 +59,6 @@
     macro(binaryTypeNodeBuffer, "nodebuffer") \
     macro(binaryTypeUint8Array, "uint8array") \
     macro(buffer, "buffer") \
-    macro(ec, "ec") \
-    macro(ed25519, "ed25519") \
     macro(fetchCors, "cors") \
     macro(fetchError, "error") \
     macro(fetchFollow, "follow") \
@@ -83,18 +78,14 @@
     macro(latin1, "latin1") \
     macro(lax, "lax") \
     macro(none, "none") \
-    macro(rsa, "rsa") \
-    macro(rsaPss, "rsa-pss") \
     macro(s3Error, "S3Error") \
     macro(strict, "strict") \
     macro(jwkCrv, "crv") \
     macro(jwkD, "d") \
     macro(jwkDp, "dp") \
     macro(jwkDq, "dq") \
-    macro(jwkDsa, "DSA") \
     macro(jwkE, "e") \
     macro(jwkEc, "EC") \
-    macro(jwkG, "g") \
     macro(jwkK, "k") \
     macro(jwkP, "p") \
     macro(jwkQ, "q") \
@@ -106,11 +97,9 @@
     macro(jwkRsa, "RSA") \
     macro(jwkX, "x") \
     macro(jwkY, "y") \
-    macro(systemError, "SystemError") \
     macro(ucs2, "ucs2") \
     macro(utf16le, "utf16le") \
-    macro(utf8, "utf8") \
-    macro(x25519, "x25519")
+    macro(utf8, "utf8")
 
 // clang-format on
 

src/jsc/bindings/EventLoopTask.h

@@ -13,11 +13,6 @@ class EventLoopTask {
     {
     }
 
-    EventLoopTask(Function<void()>&& task)
-        : m_task([task = WTF::move(task)](ScriptExecutionContext&) { task(); })
-    {
-    }
-
     void performTask(ScriptExecutionContext& context)
     {
         m_task(context);

src/jsc/bindings/Exception.h

@@ -42,7 +42,6 @@ class Exception {
     String&& releaseMessage() { return WTF::move(m_message); }
     // Optional secondary payload for codes that need more than one string to
     // shape the JS error (currently InvalidURLError's `error.base`).
-    const String& extra() const { return m_extra; }
     String&& releaseExtra() { return WTF::move(m_extra); }
 
     Exception isolatedCopy() const
@@ -56,18 +55,11 @@ class Exception {
     String m_extra;
 };
 
-Exception isolatedCopy(Exception&&);
-
 inline Exception::Exception(ExceptionCode code, String message, String extra)
     : m_code { code }
     , m_message { WTF::move(message) }
     , m_extra { WTF::move(extra) }
 {
 }
 
-inline Exception isolatedCopy(Exception&& value)
-{
-    return Exception { value.code(), value.releaseMessage().isolatedCopy(), value.releaseExtra().isolatedCopy() };
-}
-
 }

src/jsc/bindings/ExceptionCode.h

@@ -81,52 +81,3 @@ enum ExceptionCode : uint8_t {
 };
 
 } // namespace WebCore
-
-namespace WTF {
-
-template<> struct EnumTraits<WebCore::ExceptionCode> {
-    using values = EnumValues<
-        WebCore::ExceptionCode,
-        WebCore::ExceptionCode::IndexSizeError,
-        WebCore::ExceptionCode::HierarchyRequestError,
-        WebCore::ExceptionCode::WrongDocumentError,
-        WebCore::ExceptionCode::InvalidCharacterError,
-        WebCore::ExceptionCode::NoModificationAllowedError,
-        WebCore::ExceptionCode::NotFoundError,
-        WebCore::ExceptionCode::NotSupportedError,
-        WebCore::ExceptionCode::InUseAttributeError,
-        WebCore::ExceptionCode::InvalidStateError,
-        WebCore::ExceptionCode::SyntaxError,
-        WebCore::ExceptionCode::InvalidModificationError,
-        WebCore::ExceptionCode::NamespaceError,
-        WebCore::ExceptionCode::InvalidAccessError,
-        WebCore::ExceptionCode::TypeMismatchError,
-        WebCore::ExceptionCode::SecurityError,
-        WebCore::ExceptionCode::NetworkError,
-        WebCore::ExceptionCode::AbortError,
-        WebCore::ExceptionCode::URLMismatchError,
-        WebCore::ExceptionCode::QuotaExceededError,
-        WebCore::ExceptionCode::TimeoutError,
-        WebCore::ExceptionCode::InvalidNodeTypeError,
-        WebCore::ExceptionCode::DataCloneError,
-        WebCore::ExceptionCode::EncodingError,
-        WebCore::ExceptionCode::NotReadableError,
-        WebCore::ExceptionCode::UnknownError,
-        WebCore::ExceptionCode::ConstraintError,
-        WebCore::ExceptionCode::DataError,
-        WebCore::ExceptionCode::TransactionInactiveError,
-        WebCore::ExceptionCode::ReadonlyError,
-        WebCore::ExceptionCode::VersionError,
-        WebCore::ExceptionCode::OperationError,
-        WebCore::ExceptionCode::NotAllowedError,
-        WebCore::ExceptionCode::RangeError,
-        WebCore::ExceptionCode::TypeError,
-        WebCore::ExceptionCode::JSSyntaxError,
-        WebCore::ExceptionCode::StackOverflowError,
-        WebCore::ExceptionCode::ExistingExceptionError,
-        WebCore::ExceptionCode::InvalidThisError,
-        WebCore::ExceptionCode::InvalidURLError,
-        WebCore::ExceptionCode::CryptoOperationFailedError>;
-};
-
-} // namespace WTF