| Summary: | add support for 'o', 'g' and 'as' in dictionaries | ||
|---|---|---|---|
| Product: | dbus | Reporter: | Robert McQueen <robert> |
| Component: | GLib | Assignee: | Robert McQueen <robert> |
| Status: | RESOLVED FIXED | QA Contact: | John (J5) Palmieri <johnp> |
| Severity: | normal | ||
| Priority: | medium | CC: | marcandre.lureau, walters |
| Version: | unspecified | Keywords: | patch |
| Hardware: | Other | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Bug Depends on: | 17797 | ||
| Bug Blocks: | 17329 | ||
| Attachments: | add support for 'o', 'g' and 'as' in dictionaries | ||
Created attachment 19245 [details] [review] add support for 'o', 'g' and 'as' in dictionaries One thing that occurs to me is if it's worth opening a GObject bug for adding support for hash functions to boxed registration. Then we could remove this special casing. Maybe it's not worth it though. Can you also add a test for the STRV case? Otherwise looks like it should work, please commit. There is a test for the STRV case - it's not a valid dictionary key and both the a{gas} and a{oas} tests test it as a value. Merged.
|
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.
This teaches the parameterised hash table about how to hash, compare and free object paths and signatures, allowing them to be used as hash keys and values, and also how to free strvs, so they can be used as values. Adds some simple test methods which echo a{gas} and a{oas} dictionaries back immediately and compare the results.