Replace a bunch of manual impls with #[derive(Default)]

Thanks to the #[default] tag we can avoid a bunch of such manual derives
on enums.

Author: linkmauve

core/engine/src/builtins/set/ordered_set.rs

@@ -6,7 +6,7 @@ use indexmap::IndexSet;
 use std::fmt::Debug;
 
 /// A type wrapping `indexmap::IndexSet`
-#[derive(Clone, Finalize, JsData)]
+#[derive(Default, Clone, Finalize, JsData)]
 pub struct OrderedSet {
     inner: IndexSet<MapKey>,
     lock: u32,
@@ -29,21 +29,11 @@ impl Debug for OrderedSet {
     }
 }
 
-impl Default for OrderedSet {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl OrderedSet {
     /// Creates a new empty `OrderedSet`.
     #[must_use]
     pub fn new() -> Self {
-        Self {
-            inner: IndexSet::new(),
-            lock: 0,
-            empty_count: 0,
-        }
+        Self::default()
     }
 
     /// Creates a new empty `OrderedSet` with the specified capacity.

core/engine/src/bytecompiler/jump_control.rs

@@ -46,7 +46,7 @@ pub(crate) enum JumpRecordAction {
     ///         if (cond) {
     ///             continue;
     ///         }
-    ///         
+    ///
     ///         break;
     ///     } finally {
     ///         // Must execute the finally, even if `continue` is executed or `break` is executed.
@@ -177,7 +177,7 @@ pub(crate) struct JumpControlInfo {
 
 bitflags! {
     /// A bitflag that contains the type flags and relevant booleans for `JumpControlInfo`.
-    #[derive(Debug, Clone, Copy)]
+    #[derive(Default, Debug, Clone, Copy)]
     pub(crate) struct JumpControlInfoFlags: u8 {
         const LOOP = 0b0000_0001;
         const SWITCH = 0b0000_0010;
@@ -201,12 +201,6 @@ bitflags! {
     }
 }
 
-impl Default for JumpControlInfoFlags {
-    fn default() -> Self {
-        Self::empty()
-    }
-}
-
 /// ---- `JumpControlInfo` Creation Methods ----
 impl JumpControlInfo {
     fn new(current_open_environments_count: u32) -> Self {

core/engine/src/object/shape/shared_shape/mod.rs

@@ -42,19 +42,13 @@ const RESEREVED_TRANSITION_TYPE: u8 = 0b0000_0011;
 
 bitflags! {
     /// Flags of a shape.
-    #[derive(Debug, Clone, Copy, PartialEq, Eq, Finalize)]
+    #[derive(Default, Debug, Clone, Copy, PartialEq, Eq, Finalize)]
     pub struct ShapeFlags: u8 {
         /// Represents the transition type of a [`SharedShape`].
         const TRANSITION_TYPE = 0b0000_0011;
     }
 }
 
-impl Default for ShapeFlags {
-    fn default() -> Self {
-        Self::empty()
-    }
-}
-
 impl ShapeFlags {
     // NOTE: Remove type bits and set the new ones.
     fn insert_property_transition_from(previous: Self) -> Self {

tests/tester/src/main.rs

@@ -71,7 +71,7 @@ impl Config {
 
 /// Structure to allow defining ignored tests, features and files that should
 /// be ignored even when reading.
-#[derive(Debug, Deserialize)]
+#[derive(Default, Debug, Deserialize)]
 struct Ignored {
     #[serde(default)]
     tests: FxHashSet<Box<str>>,
@@ -112,16 +112,6 @@ impl Ignored {
     }
 }
 
-impl Default for Ignored {
-    fn default() -> Self {
-        Self {
-            tests: FxHashSet::default(),
-            features: FxHashSet::default(),
-            flags: TestFlags::empty(),
-        }
-    }
-}
-
 /// Boa test262 tester
 #[derive(Debug, Parser)]
 #[command(author, version, about, name = "Boa test262 tester")]