<feed xmlns='http://www.w3.org/2005/Atom'>
<title>ding-libs.git, branch master-wip</title>
<subtitle>ding-libs wip repository</subtitle>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/'/>
<entry>
<title>ini_comment.h needs to be installed as well for the new ini interface.</title>
<updated>2013-01-28T13:53:48+00:00</updated>
<author>
<name>Günther Deschner</name>
<email>gdeschner@redhat.com</email>
</author>
<published>2012-10-24T12:48:27+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=6faf2530ee561ee09a3e47365ad456ae4eec23e4'/>
<id>6faf2530ee561ee09a3e47365ad456ae4eec23e4</id>
<content type='text'>
Guenther
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Guenther
</pre>
</div>
</content>
</entry>
<entry>
<title>Unit test for parsing flags.</title>
<updated>2013-01-24T07:34:56+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-25T14:31:07+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=66e30c2f3862093505a8497744207d2812c95be3'/>
<id>66e30c2f3862093505a8497744207d2812c95be3</id>
<content type='text'>
Added unit test.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Added unit test.
</pre>
</div>
</content>
</entry>
<entry>
<title>More interface refactoring</title>
<updated>2013-01-24T07:34:51+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-24T22:45:00+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=6823d7104cca0f8ac22ed9432ee8a2f1a0d9124a'/>
<id>6823d7104cca0f8ac22ed9432ee8a2f1a0d9124a</id>
<content type='text'>
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.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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.
</pre>
</div>
</content>
</entry>
<entry>
<title>Introducing parsing flags</title>
<updated>2013-01-24T07:34:48+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-24T18:33:44+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=fa00e2e16e5c1d3ac56f1bbd3a84875642d0bb39'/>
<id>fa00e2e16e5c1d3ac56f1bbd3a84875642d0bb39</id>
<content type='text'>
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.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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.
</pre>
</div>
</content>
</entry>
<entry>
<title>Refactor interface a bit</title>
<updated>2013-01-24T07:34:43+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-24T16:31:24+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=3b9baa61367f2b651c8068efc99ce017331622e0'/>
<id>3b9baa61367f2b651c8068efc99ce017331622e0</id>
<content type='text'>
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.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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.
</pre>
</div>
</content>
</entry>
<entry>
<title>Replacing sprintf with snprintf</title>
<updated>2013-01-24T07:34:40+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-23T18:49:56+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=833a46e384828be48c27898e755d6215eb5c4bb8'/>
<id>833a46e384828be48c27898e755d6215eb5c4bb8</id>
<content type='text'>
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.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fixing headers</title>
<updated>2013-01-24T07:34:37+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-23T18:03:00+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=57faa64667411a3d8eeeeff62b1e659a94ed29cf'/>
<id>57faa64667411a3d8eeeeff62b1e659a94ed29cf</id>
<content type='text'>
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.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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.
</pre>
</div>
</content>
</entry>
<entry>
<title>Use ENOMEM instead of errno</title>
<updated>2013-01-24T07:34:35+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-23T16:59:48+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=360710d1a6c0c24d00a915a630884281b0b9e665'/>
<id>360710d1a6c0c24d00a915a630884281b0b9e665</id>
<content type='text'>
Reviewed all uses or errno. Cleaned places that needed cleaning.
Did not remove &lt;errno.h&gt; yet where it is not needed.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewed all uses or errno. Cleaned places that needed cleaning.
Did not remove &lt;errno.h&gt; yet where it is not needed.
</pre>
</div>
</content>
</entry>
<entry>
<title>Converting errors to enum</title>
<updated>2013-01-24T07:34:32+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-23T16:07:43+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=9b368a8652b595528dcf85c9ecfeba0d93550ee1'/>
<id>9b368a8652b595528dcf85c9ecfeba0d93550ee1</id>
<content type='text'>
Patch converts error and error level
to enum instead of defines.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Patch converts error and error level
to enum instead of defines.
</pre>
</div>
</content>
</entry>
<entry>
<title>Add INI_GET_LAST_VALUE</title>
<updated>2013-01-07T08:25:44+00:00</updated>
<author>
<name>Dmitri Pal</name>
<email>dpal@redhat.com</email>
</author>
<published>2012-12-23T21:54:22+00:00</published>
<link rel='alternate' type='text/html' href='https://fedorapeople.org/cgit/gd/public_git/ding-libs.git/commit/?id=6c3a184c9345ab35bb64002530dc68074cc3a10b'/>
<id>6c3a184c9345ab35bb64002530dc68074cc3a10b</id>
<content type='text'>
Refactored the defines to enum.
Added support for fetching last value from the section.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Refactored the defines to enum.
Added support for fetching last value from the section.
</pre>
</div>
</content>
</entry>
</feed>
