diff --git a/src/hooks/usePatternRotationUpload.ts b/src/hooks/usePatternRotationUpload.ts index da09550..ececf57 100644 --- a/src/hooks/usePatternRotationUpload.ts +++ b/src/hooks/usePatternRotationUpload.ts @@ -119,7 +119,7 @@ export function usePatternRotationUpload({ pesData, // Original unrotated pattern for caching ); - return; // Early return + return; } // No rotation case diff --git a/src/hooks/usePatternValidation.ts b/src/hooks/usePatternValidation.ts index 4986b3d..4c55938 100644 --- a/src/hooks/usePatternValidation.ts +++ b/src/hooks/usePatternValidation.ts @@ -1,4 +1,4 @@ -import { useCallback, useMemo } from "react"; +import { useMemo } from "react"; import type { PesPatternData } from "../formats/import/pesImporter"; import type { MachineInfo } from "../types/machine"; import { calculateRotatedBounds } from "../utils/rotationUtils"; @@ -31,8 +31,8 @@ export function usePatternValidation({ patternOffset, patternRotation, }: UsePatternValidationParams): PatternBoundsCheckResult { - // Check if pattern (with offset and rotation) fits within hoop bounds - const checkPatternFitsInHoop = useCallback((): PatternBoundsCheckResult => { + // Memoize the bounds check calculation to avoid unnecessary recalculations + return useMemo((): PatternBoundsCheckResult => { if (!pesData || !machineInfo) { return { fits: true, error: null }; } @@ -94,12 +94,4 @@ export function usePatternValidation({ return { fits: true, error: null }; }, [pesData, machineInfo, patternOffset, patternRotation]); - - // Memoize the result to avoid unnecessary recalculations - const boundsCheck = useMemo( - () => checkPatternFitsInHoop(), - [checkPatternFitsInHoop], - ); - - return boundsCheck; }