The INI Parser API should be opaque as to what its internal storage looks like. Instead of returning collection_items, it should be returning opaque ini_config_items (probably a typedef for collection_item, internally).
Furthermore, there should be a free_ini_config() function that acts as a wrapper around delete_collection.
This will give us more flexibility in the future if we decide to change the underlying data type from "collection" to something else.
The wrappers were created: commit 09c4b073ca0bdf8d208cbcc7e84a7d33629213d2
After discussion we decided not to hide the "struct collection_item *" from the caller so this part won't be fixed.
fixedin: => 0.5.0 resolution: => fixed status: new => closed
Fields changed
rhbz: => 0
Metadata Update from @sgallagh: - Issue assigned to dpal - Issue set to the milestone: void
Login to comment on this ticket.