summaryrefslogtreecommitdiffstats
path: root/ini
Commit message (Collapse)AuthorAgeFilesLines
* Fix indentention in the switch statementDmitri Pal2012-04-051-11/+20
| | | | Spotted style violation.
* Test for all section merge modesDmitri Pal2012-04-051-13/+201
| | | | | | New test reads smerge.conf in all possible modes and compares the combined result with the sexpect.conf function.
* Test DETECT mode and use new fileDmitri Pal2012-04-051-7/+23
| | | | | | Patch adds smerge.conf file to the list of files to test and adds test for the DETECT mode for inidividual values.
* New test files for section mergeDmitri Pal2012-04-052-0/+677
| | | | | | | smerge.conf - test file sexpect.conf - file contains output of the smerge.conf processed in different merge modes for sections and values
* Return error in DETECT modeDmitri Pal2012-04-051-0/+11
| | | | | If in merge DETECT mode and there were no parsing errors return error if there were merge collisions.
* Refactor section processingDmitri Pal2012-04-051-9/+111
| | | | | | This patch adds functionality to respect merge section flags and to process section collisions differently dpending on these flags.
* Use section line numberDmitri Pal2012-04-051-2/+5
| | | | | Use section line number for error reporting about the section collisions.
* Enhance value processingDmitri Pal2012-04-051-49/+116
| | | | | | | | | This patch refactores the value processing function so that it can be used both in normal mode when velues need to be constrcuted and saved into the current section (po->sec) and in the merge mode when values are already constructed and need to be saved into a po->merge_sec.
* Preparing for merging sectionsDmitri Pal2012-04-051-0/+177
| | | | | | | | Patch implements three functions: * Function to detect a collision between two section names. * Function to empty section from all its keys * Function to add values one by one from one section to another.
* Change parse_error to use save_errorDmitri Pal2012-04-051-11/+12
| | | | | | * Changed parse_error to use save_error function created in previous commit. * Fixed comment to be more clear.
* Add save_error functionDmitri Pal2012-04-051-0/+28
| | | | Create a function to add error into error list.
* Add new vars to parse structureDmitri Pal2012-04-051-1/+13
| | | | | Adding new varibles to the internal parsing structure and initializing them.
* New merge flagsDmitri Pal2012-04-052-3/+12
| | | | | | | Adding "DETECT" merge modes. These modes would be useable for dry run cases to detect if there are any potential merge conflicts. Patch just defines new constans and adds input checks.
* New error codes and messagesDmitri Pal2012-04-052-9/+57
| | | | | | | | Patch consists of two parts: * Adding new constants for error messages to ini_configobj.h file * Making ini_print.c use this header file instead of the old one. Converting this to enums is tracked as a separate ticket.
* Fixing coverity issues 10078 & 10079Dmitri Pal2012-04-051-4/+6
| | | | | The fact that the other_test could return EOK on error was the actual problem.
* INI: Silence compilation warningsJakub Hrozek2012-03-062-1/+1
| | | | | * Do not set ext_data parameter to NULL * include a private header to avoid implicit declarations
* Coverity issue 10075Dmitri Pal2011-01-171-0/+11
| | | | | Previous fix was not complete. Secondary scan revealed other issues.
* Coverity issue 10042Dmitri Pal2011-01-171-0/+1
| | | | | | Previous fix was not complete. New scan revealed potential memory leak in case of coding error. Low severity but still a bug...
* Adding missing constDmitri Pal2011-01-031-1/+1
| | | | One liner to eliminate compilation warning.
* Freeing memory used for source dirDmitri Pal2011-01-031-2/+5
| | | | | | | | I decided not to redo the whole test. It would have been too much work. Just cleaned the code a bit and freed allocated memory. It is Coverity issue 10075.
* Free newly created value in case of errorDmitri Pal2011-01-031-0/+2
| | | | | | The newly created value object was not freed in case of an error. Coverity issue 10076.
* Initialize simple bufferDmitri Pal2011-01-031-5/+10
| | | | | | | | | Allocating simple buffer is not enough. It needs to be initialized with something. The patch does exactly this for the case when the empty comment string is represented by a NULL value in the function input variable. Coverity issue 10073.
* Leaking memory on failureDmitri Pal2011-01-031-0/+1
| | | | One liner to fix Coverity isssue 10044.
* File descritpor leakDmitri Pal2011-01-031-0/+2
| | | | | Noticed under valgrind then checked and saw that this is a coverity issue 10077.
* Coverity issue 10034Dmitri Pal2011-01-031-4/+1
| | | | | | The 'len' varible was not initialized properly. The ref_array_get function does not return error so the error check is removed.
* Additional testsDmitri Pal2010-12-212-0/+43
| | | | | | | | I investigated the coverity issue 10043. Reading code did not reveal the issue. I added the test just to be sure and ran valgrind again. It does not seem to leak despite the Coverity concerns.
* Correcting paths to test filesDmitri Pal2010-12-211-14/+6
| | | | | | | | The paths to test files did not work in parallel build or 'make diskcheck' cases. Now they are corrected. All temp files are created in the build directory and read from there.
* Fix crashes with file objectDmitri Pal2010-12-211-9/+21
| | | | | | This patch addresses the crashes with the file object when the file is not there and the file context needs to be destroyed during the open operation.
* Preparing to merge sectionsDmitri Pal2010-12-211-34/+63
| | | | | | | * Rearranged code in parser module and created a skeleton of the function that will be responcible for merging sections. * Added missing static declarations to some functions.
* Collision flag validationDmitri Pal2010-12-211-0/+45
| | | | | Added validation of the collision flags at the interface boundary.
* Allow merging valuesDmitri Pal2010-12-215-11/+410
| | | | | | | | | | | | | | | | This patch is the first pass at merging functionality. It implements merging of values that belong to the same section. Patch includes: * Definition of merge flags in doxy format * Definition of the masks in internal header * Changes to parser to handle the merging of values. * Also swithed parser to not use ini_config.h as I want to switch implementation of the current interface to new interface at some point. * New unit test was created. * New config file for this unit test was added to ini.d Main code changes are in ini_parse.c
* New copy and folding functionalityDmitri Pal2010-12-214-74/+274
| | | | | | | | | | | | | | | | * Added method to copy configuration. * Added unit test for it. * Added method to reset boandary for the whole configuration. * Added unit test for it. * Unit test now can read a file save it read again and save. Both saves produce same files! * Reworked the way the parser was dealing with empty lines. It was sutting off the value in the middle if the folded line consisted of just spaces.
* Improvements to the value objectDmitri Pal2010-12-212-27/+78
| | | | | | * Fixed different issues with folding and serialization * Added more unit testing.
* Refactoring error reportingDmitri Pal2010-12-216-28/+46
| | | | Minor cleanup in the error reporting.
* Enhancements to value objectDmitri Pal2010-12-213-1/+272
| | | | | | | * Added capability to copy value. * Added function to set alternative wrapping boundary of the value. * Created a unit test for both new functions.
* Refactoring comment objectDmitri Pal2010-12-213-142/+285
| | | | | | | | | | | | | | * Added copy coment functionality * Changed the internal implementation to use array simple buffer instead of the two arrays one for lines and one for lengths * Realigned comments in the header file * Changed the trace macros to use simplified entry and exit macros. * Corrected unit test to return error codes. * Refactored unit test to use array of test functions intead of the direct calls * Added unit test for copy functionality.
* Fix version handling of the librariesSumit Bose2010-09-231-2/+2
|
* Fix overflow in ini_parse unit testSumit Bose2010-09-231-3/+2
|
* Introducing configuration file objectDmitri Pal2010-09-229-100/+1323
| | | | | | | | The patch contains two major pieces of work: * The beginning of the new INI interface significantly re-thought and reworked. * The implementation of the file object. Parsing was adjusted to use this object.
* Starting to consolidate the new interface in one placeDmitri Pal2010-09-226-224/+180
| | | | | | | | | | I realized that the new high level interface should be defined in one file instead of being scattered in many headers. I would have to eventuallu eliminate ini_configobj.h too and move everything to the new interface in ini_config.h. This patch renames couple functions to be consistent with the naming convention and removes ini_parse.h header.
* New INI parserDmitri Pal2010-09-223-0/+1223
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The parser is added to the existing module. The old parsing functuion will be removed when we switch to the new interface. Parser logic: * There is one high level function that wraps the parser interface. It is at the bottom of the module. ini_parse_config(); * Internally it creates a perser object and then runs parser on it. * At the end parser object is destroyed. * This object stores the state of the parser. * The parser has an action queue * There are several actions that parser can perform - read line - inspect read line - record an error - process last comment in the file (POST) * Each action handler determines what to do next depending upon what has happened. * Read handler reads lines and enqueues inspection action in case of success or error action in case of failure. * Inspection action parses last read line and treats it either is a: * Comment * Section * New key + value * Continuation of the value In case of error the error action is enqueued. * Error can be fatal or non fatal. It depend on the error_level flag passed in. If the error is non fatal the read action is enqueued otherwise parser stops. * The POST action is a special action to handle comment at the bottom of the file. The comment is stored with the value it preceeds so in case there is a comment at the bottom of the file a special value needs to be created to hold just the comment.
* The beginning of the new INI interfaceDmitri Pal2010-09-224-0/+292
| | | | | | | | * ini_config_priv.h - private header * ini_configobj.h - future public header for the new interface * ini_configobj.c - just constructor/decructor for now * ini_serialize.c - code to serialize the configuration into a buffer so it can be written somewhere.
* Fixes to the value objectDmitri Pal2010-09-224-117/+98
| | | | | | | | | | | | * Removed the annoyong type casting in the serialization since there is a new function in the simplebuffer to addstring. * Changed the serialization code to not create simple buffer object but rather add to the passed in one. * Added serialization of the value that stores section * Started cleaning constants but then stopped since this effor should be done a bit later when the new high level ini config interface is better baked.
* Add error codes for the new parserDmitri Pal2010-09-222-5/+12
|
* Extend the comment interfaceDmitri Pal2010-09-222-10/+38
| | | | | Take advantage of the already known string length instead of counting bytes once more.
* More config files for testingDmitri Pal2010-09-223-7/+158
|
* Introducing Value objectDmitri Pal2010-09-223-0/+1569
| | | | | | | | | | | | | Value object is an object that combines a potentially multiline value comment and some statistical information regarding a configuration value. Patch includes: Source Header Unit test Makefile changes
* Refining comment objectDmitri Pal2010-09-223-22/+139
| | | | Comment object should store lengths of the lines.
* Build all of the ding-libs from the rootStephen Gallagher2010-09-228-1085/+4
| | | | | This changes our approach from having independent tarballs to having a single, monolithic tarball for all of the libraries
* ini_config: Adding RPM build targetsStephen Gallagher2010-08-183-2/+114
|
generated by cgit (git 2.34.1) at 2024-06-07 14:30:59 +0000