summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* INI: Bump version-infoHEADmasterOndrej Kos2013-05-061-1/+1
| | | | https://fedorahosted.org/sssd/ticket/1908
* DHASH: minor fixesOndrej Kos2013-04-101-1/+3
| | | | | | | | https://fedorahosted.org/sssd/ticket/1877 fix coverity issue 11620 fix compilation warning (missing HI_STATE_0 in switch)
* libiniconfig_devel must require libref_array-devel and libbasicobjects-develJakub Hrozek2013-04-051-0/+2
|
* DHASH: Don't use backward jumpsOndrej Kos2013-04-051-23/+56
| | | | https://fedorahosted.org/sssd/ticket/1855
* INI: Remove dead codeOndrej Kos2013-04-051-6/+1
| | | | | | | | https://fedorahosted.org/sssd/ticket/1857 Discovered by coverity The code was inaccessibile, if no key is present, the error will occur before this check.
* PATH_UTILS: check against character representation of NULLOndrej Kos2013-04-041-12/+12
| | | | https://fedorahosted.org/sssd/ticket/1859
* DHASH: Check before dereferencingOndrej Kos2013-04-041-3/+6
| | | | https://fedorahosted.org/sssd/ticket/1858
* COLLECTION: Fix comparisionOndrej Kos2013-04-041-1/+1
| | | | https://fedorahosted.org/sssd/ticket/1856
* Bump version for 0.3.0.1 releaseding_libs-0_3_0_1Ondrej Kos2013-03-291-2/+2
|
* Remove spurious castJakub Hrozek2013-03-291-1/+1
|
* Bump version for 0.3.0 releaseding_libs-0_3_0Ondrej Kos2013-03-281-2/+2
|
* Add a release scriptJakub Hrozek2013-03-281-0/+29
|
* Run autoreconf before configure in the specfileJakub Hrozek2013-03-251-0/+1
|
* Include AM_PROG_AR in configure.ac to get rid of warningsJakub Hrozek2013-03-251-1/+3
|
* ini_comment.h needs to be installed as well for the new ini interface.Günther Deschner2013-01-282-1/+2
| | | | Guenther
* Expose collected stat dataDmitri Pal2013-01-283-0/+45
| | | | | Added a new top level interface funtion and a unit test for it.
* Check is the stats we collectedDmitri Pal2013-01-283-0/+45
| | | | | This patch corrects a problem related to stats being accessed and evaluated without being initialized.
* Unit test for parsing flags.Dmitri Pal2013-01-243-0/+89
| | | | Added unit test.
* More interface refactoringDmitri Pal2013-01-246-174/+171
| | | | | | | | | | | | | | | | | I also realized that error list processing should not be bound to the file object. This patch corrects that by moving the error_list and corresponding count from file object to the config object. Things updated by the patch: 1. The internal variables are moved from file obj to config obj. 2. The external header is updated to reflect the change 3. Functions are moved from file obj module to config obj module. 4. Parser code is updated because error validation was in the wrong place 5. Unit test is adjusted to get error list from the right object. I had to adjust the copy function for the config object. Copy function does not copy over the error list.
* Introducing parsing flagsDmitri Pal2013-01-244-65/+187
| | | | | | | | | | | | | | | | 1. Added definition of the parsing flags to the header. 2. Added new argument to the parser. 3. Added function to check the correctness of the line. 4. Added new constants and strings for errors. 5. Refactored function that handles spaces. 6. Moved handle_space since it now calls handle_kvp. 7. Updated handle_kvp to trim spaces at the beginning. 8. Adjusted line inspection to handle any space character. 9. Adjusted exising unit tests. Added parameter to the call. 10. Found couple places where there was a space at the end of the line and cleaned them. The additional unit tests will be added in the follow up patch.
* Refactor interface a bitDmitri Pal2013-01-246-90/+89
| | | | | | | | | | | | | | | | | After some evaluation it became apparent that error level and collision flags should not be a part of the fileobj but rather passed directly to the parser function. This way the interface is more clean and logical. This patch: 1) Removes the error level and collision flags from the fileobj 2) Moves validation function from ini_fileobj.c to ini_configobj.c where it belongs. 3) Moves error level and collision flags arguments from file open function to the parser function. 4) The internals of the file obj are cleaned 5) Validation of the arguments is moved to parser function. 6) Unit tests are updated.
* Replacing sprintf with snprintfDmitri Pal2013-01-246-55/+79
| | | | | | | | Replaced sprintf in the unit test. Defined constants for sizes and used them. Wrpapped lines where noticed that they are longer than 80. Added comments to the places where sprintf is still used but it is safe to use.
* Fixing headersDmitri Pal2013-01-2436-67/+36
| | | | | | | Move GNU_SOUCE definition to configure.ac Change all headers to use config.h. Move config.h to the top. Remove errno.h if it is not needed.
* Use ENOMEM instead of errnoDmitri Pal2013-01-245-41/+18
| | | | | Reviewed all uses or errno. Cleaned places that needed cleaning. Did not remove <errno.h> yet where it is not needed.
* Converting errors to enumDmitri Pal2013-01-241-32/+25
| | | | | Patch converts error and error level to enum instead of defines.
* Add INI_GET_LAST_VALUEDmitri Pal2013-01-073-8/+44
| | | | | Refactored the defines to enum. Added support for fetching last value from the section.
* Fix CLANG errors in unit testDmitri Pal2013-01-075-40/+44
|
* Make CLANG happyDmitri Pal2013-01-044-11/+23
| | | | Address CLANG issues in the main code.
* Fixing coverity issue 13105Dmitri Pal2012-12-121-2/+2
|
* Update unit testsDmitri Pal2012-12-061-6/+368
| | | | | | The parser unit test have been updated to execute the changes in the parser code. A new test was added to execute merge code in all possible modes.
* Update files used in the unit testDmitri Pal2012-12-065-106/+659
|
* Update comments in the public interfaceDmitri Pal2012-12-061-2/+7
|
* Improve parserDmitri Pal2012-12-065-34/+98
| | | | | | | | | | | | | | | | | | 1. Move the last comment into the config object Last comment in the file was previously treated as a special value in the last section in the configuration object. That created difficulties while merging. Storing an orphan comment as part of the config object is a better approach 2. Since the structure of the config object changed the internal header and serialization required update. 3. Parser code was updated to: - deal with the last comment on the top level - make sure the right error codes are returned in different modes - pass right information through the layers which caused some refactoring of the internal calls - prevent using an already used configuration object as an input to parser.
* Ability to merge configurationsDmitri Pal2012-12-061-5/+581
| | | | This patch adds capability to merge two configuration objects.
* Fix trace messageDmitri Pal2012-12-061-1/+1
|
* Merge comments from different valuesDmitri Pal2012-12-063-35/+134
| | | | | Patch allows merging comments that are part of the two different value objects. The values are not merged.
* Merge and serialize commentsDmitri Pal2012-12-063-2/+246
| | | | | | Patch adds ability to combine two different comments into one. It includes changes to the comment API, header file and unit test. It also fixes an invalid free in one of the error cases.
* Adding a trace statementDmitri Pal2012-12-061-1/+2
|
* Remove code that allows dup sectionsDmitri Pal2012-12-065-201/+30
| | | | | | | | | | | | Duplicate sections in the config tree is a bad idea. After more evaluation I decided to not allow more than one section with the same name to be acceptable in the resulting object. All duplicate sections need to be processed so that only one section is left as a result. The collision flags control that. It is OK to remove it as this code has not been released yet. I also spotted that unit test does not exit if the system() call was successful but the command it invoked was not. Fixed that. Also fixed couple spelling mistakes here and there.
* Fixed libcollection dependency and header files inclusionOndrej Kos2012-10-251-0/+3
| | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=637955 Also while fixing this, I found that rpmbuild was broken due to missing includes in ini_config-devel spec.
* Clean doxygen configurationsDmitri Pal2012-10-245-170/+5
| | | | | The doxygen files contained some deprecated values and typos. This patch cleans them for all interfaces exposed by ding libs.
* Fix doxygen warnings for the interfaceDmitri Pal2012-10-241-49/+50
| | | | This patch fixes the doxygen warnings in the new interface.
* Fixing coverity issue 11089Dmitri Pal2012-10-171-1/+1
| | | | Pointer was not initialized before being read.
* Fix two issues in Makefile.amDmitri Pal2012-10-171-2/+2
| | | | | 1. The header files were not included into the package. 2. The `make distcheck` commnd was failing due to typo.
* Build docs for new interfaceDmitri Pal2012-10-171-1/+1
| | | | | This patch stops building interface docs for the old interface and starts generating description for the new interface.
* Added new tests for the multi value keysDmitri Pal2012-10-172-0/+1136
|
* Start building the new interfaceDmitri Pal2012-10-171-0/+3
|
* The implementation of the new interfaceDmitri Pal2012-10-173-0/+1319
| | | | | | | | | The three files in the patch implement the functions of the new interface. It is mostly inspired by the old interface except that the string value is replaced with the value object that hides all the metadata about the value itself. The main function to get value now allows more than one value per key.
* Add a search state to the config objectDmitri Pal2012-10-172-0/+40
| | | | | | | The search state allows one to search a configuration object for the same key in the cases when a key permits more than one value scattered across the config file. The state needs to be stored, initialized and properly cleaned.
* Definition of the new INI interfaceDmitri Pal2012-10-171-206/+789
| | | | | | | | Some time ago I started the new INI interface that exists in parallel to the existing interface but uses value object to store value rather than just strings. The header has all the details about the new interface.