#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 09c4b07

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

The wrappers were created:
commit 09c4b07

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

The wrappers were created:
commit 09c4b07

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

Fields changed

rhbz: => 0

Fields changed

rhbz: => 0

Metadata Update from @sgallagh:
- Issue assigned to dpal

7 years ago

Metadata Update from @sgallagh:
- Issue set to the milestone: void

7 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/1079

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata