Skip to content

Add Sendable conformance to value types#272

Merged
muukii merged 1 commit intomainfrom
feat/add-sendable-conformance
Jun 21, 2025
Merged

Add Sendable conformance to value types#272
muukii merged 1 commit intomainfrom
feat/add-sendable-conformance

Conversation

@muukii
Copy link
Collaborator

@muukii muukii commented Jun 20, 2025

Summary

  • Added Sendable conformance to various value types throughout the codebase
  • Ensures thread safety for these types in concurrent environments
  • No breaking changes - all types already met the requirements for Sendable

Changes

Core Types

  • ParameterRange<T, Target> - Generic parameter range type
  • PixelAspectRatio - Aspect ratio struct
  • Geometry - Geometry calculation enum
  • OvalBrush - Oval brush configuration struct

Filter Types (12 filters)

  • FilterBrightness, FilterContrast, FilterSaturation
  • FilterExposure, FilterHighlights, FilterShadows
  • FilterTemperature, FilterSharpen, FilterGaussianBlur
  • FilterUnsharpMask, FilterVignette, FilterFade

Error Types

  • EngineRuntimeError - Engine runtime errors
  • EditingStackError - Editing stack errors

UI Configuration Types

  • ClassicImageEditOptions and nested structs
  • ClassicImageEditStyle and nested structs

Editing Types

  • EditingCrop - Crop information struct
  • EditingCrop.Rotation - Rotation angle enum
  • ImageTool - Image processing tool enum

Test Plan

  • Build succeeds without warnings
  • No Sendable-related compilation errors
  • Existing tests pass (please run CI)

🤖 Generated with Claude Code

This commit adds Sendable conformance to various value types throughout the codebase to ensure thread safety in concurrent environments.

Changes include:
- Core types: ParameterRange, PixelAspectRatio, Geometry, OvalBrush
- All Filter structs (12 filters)
- Error types: EngineRuntimeError, EditingStackError
- UI configuration types: ClassicImageEditOptions, ClassicImageEditStyle
- Editing types: EditingCrop, ImageTool

These changes enable safer usage of these types across different actors and threads without risk of data races.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@muukii muukii merged commit 421e0af into main Jun 21, 2025
1 of 2 checks passed
@muukii muukii deleted the feat/add-sendable-conformance branch June 21, 2025 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant