[PR #83] [MERGED] Allow NSObject Wrap to handle lists of maps/dicts. #112

Closed
opened 2026-01-30 21:27:12 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/claunia/plist-cil/pull/83
Author: @pkillboredom
Created: 4/28/2025
Status: Merged
Merged: 6/25/2025
Merged by: @claunia

Base: masterHead: pkillboredom-handle-list-of-maps


📝 Commits (1)

  • 742e1fa Allow NSObject Wrap to handle lists of maps/dicts.

📊 Changes

2 files changed (+29 additions, -2 deletions)

View changed files

📝 plist-cil.test/ParseTest.cs (+20 -1)
📝 plist-cil/NSObject.cs (+9 -1)

📄 Description

Allows NSObject's Wrap method to handle List<Dictionary<string,object>>. This came up while migrating some very old (BinaryFormatter reliant) plist code to this library. Lists of Dicts appear in AirPlay configuration requests.

Sorry for the whitespace(?) change on line 1 in both files... VS's diff tool doesnt show any difference there, not sure what changed.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/claunia/plist-cil/pull/83 **Author:** [@pkillboredom](https://github.com/pkillboredom) **Created:** 4/28/2025 **Status:** ✅ Merged **Merged:** 6/25/2025 **Merged by:** [@claunia](https://github.com/claunia) **Base:** `master` ← **Head:** `pkillboredom-handle-list-of-maps` --- ### 📝 Commits (1) - [`742e1fa`](https://github.com/claunia/plist-cil/commit/742e1fa57a542d3d494a85882900f1c0278c3119) Allow NSObject Wrap to handle lists of maps/dicts. ### 📊 Changes **2 files changed** (+29 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `plist-cil.test/ParseTest.cs` (+20 -1) 📝 `plist-cil/NSObject.cs` (+9 -1) </details> ### 📄 Description Allows NSObject's Wrap method to handle `List<Dictionary<string,object>>`. This came up while migrating some very old (BinaryFormatter reliant) plist code to this library. Lists of Dicts appear in AirPlay configuration requests. Sorry for the whitespace(?) change on line 1 in both files... VS's diff tool doesnt show any difference there, not sure what changed. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-30 21:27:12 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: claunia/plist-cil#112