#37 INI Parser should not expose its internal storage
Closed: Fixed None Opened 14 years ago by sgallagh.

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

7 years ago

Login to comment on this ticket.

Metadata