Skip to content

Conversation

@narknon
Copy link
Collaborator

@narknon narknon commented Sep 3, 2025

Description

  • Implement TMap userdata handling in push_mapproperty Operation::Set
  • Support copying TMap userdata with proper key-value pair handling
  • Handle empty maps and rehashing after copy
  • Enable passing TMap properties as function parameters

Completes container userdata support for TSet, TArray, and TMap

Fixes # (issue) (if applicable)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How has this been tested?

Tested passing TMap properties as function parameters
Tested with various element types (int32, string, UObject references)

Checklist

  • I have commented my code, particularly in hard-to-understand areas.
  • I have added the necessary description of this PR to the changelog, and I have followed the same format as other entries.

Screenshots

Additional context

  - Implement TMap userdata handling in push_mapproperty Operation::Set
  - Support copying TMap userdata with proper key-value pair handling
  - Handle empty maps and rehashing after copy
  - Enable passing TMap properties as function parameters

Completes container userdata support for TSet, TArray, and TMap
@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2025

MSVC-Game__Debug__Win64 Download Logs
Build Details
Name Information
PR Commit 984e0d9
Merge Commit 8c5a69f
Size 47.86 MB
Last Updated Sep 3, 25, 1:58:31 AM UTC
Expires At Sep 17, 25, 1:58:26 AM UTC

MSVC-Game__Shipping__Win64 Download Logs
Build Details
Name Information
PR Commit 984e0d9
Merge Commit 8c5a69f
Size 29.39 MB
Last Updated Sep 3, 25, 2:02:30 AM UTC
Expires At Sep 17, 25, 2:02:27 AM UTC

@narknon narknon merged commit a23ca94 into main Sep 3, 2025
1 check passed
@narknon narknon deleted the LuaTMapFixes2 branch September 3, 2025 01:38
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