chore: use Set for checking defaultProps keys for small perf gain over 'in' operator

Author: charles4221

src/components/FontAwesomeIcon.tsx

@@ -22,7 +22,7 @@ import { typedObjectKeys } from '../utils/typed-object-keys'
 
 const logger = new Logger('FontAwesomeIcon')
 
-const defaultProps = {
+const DEFAULT_PROPS = {
   border: false,
   className: '',
   mask: undefined,
@@ -53,6 +53,8 @@ const defaultProps = {
   widthAuto: false,
 }
 
+const DEFAULT_PROP_KEYS = new Set(Object.keys(DEFAULT_PROPS))
+
 const EMPTY_OBJECT: Record<string, never> = {}
 
 export interface FontAwesomeIconProps
@@ -153,7 +155,7 @@ export const FontAwesomeIcon = React.forwardRef(
     props: FontAwesomeIconProps,
     ref: React.Ref<SVGSVGElement>,
   ): React.JSX.Element | null => {
-    const allProps: FontAwesomeIconProps = { ...defaultProps, ...props }
+    const allProps: FontAwesomeIconProps = { ...DEFAULT_PROPS, ...props }
 
     const {
       icon: iconArgs,
@@ -199,12 +201,14 @@ export const FontAwesomeIcon = React.forwardRef(
 
     for (const key of typedObjectKeys(allProps)) {
       // Skip default props
-      if (key in defaultProps) {
+      if (DEFAULT_PROP_KEYS.has(key)) {
         continue
       }
 
       // Add all other props to the extraProps object
-      // @ts-expect-error TypeScript doesn't know that allProps[key] is valid
+      // @ts-expect-error since `key` can be any of the keys in FontAwesomeIconProps,
+      // TypeScript widens the type of the `obj[key]` lookups to a union of all possible values,
+      // which will not correctly overlap each other.
       extraProps[key] = allProps[key]
     }