[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