[ccan] iniparser uploaded
Tim Post
echo at echoreply.us
Fri Apr 3 16:15:55 EST 2009
Hi Rusty,
I have uploaded iniparser. Here are some notes:
1 - Test coverage does not include functions in dictionary.c,
these are tested by extension when testing iniparser. If
tests need to go in for those, no big deal, I'll write them.
2 - The source was already commented via doxygen, so I just
left the comments in place (mostly). If someone wants to
extract the doxymentation, they can do so. If those comments
confuse ccanlint, feel free to rip them out.
3 - Tests run valgrind clean
4 - I've fixed quite a few bugs, valgrind warnings, and other nits.
I've been doing that in an existing tree using iniparser, so
if anyone wants to see the changes, they can be found at
http://echoreply.us/hg/nexgent.hg/
5 - I don't have a problem fixing bugs / maintaining the module. My
e-mail is right next to my copyright under the original author's
in the license header.
6 - It got a 5/5 lint score, the tests run just fine, it should be able
to just drop right in. If somethings broken / incorrect let me know
and I'll fix it.
7 - I have not yet a chance to add the additional directives (i.e.
include, include_if , but will in the coming weeks. It took longer
than I expected to modernize and pass its own test suite.
Cheers,
--Tim
More information about the ccan
mailing list