search

michaelrsweet / mxml

Tiny XML library.
https://www.msweet.org/mxml
Apache License 2.0
428 stars 157 forks source link

When I make this project ,There are errors. #191

Closed huanghaoa closed 7 years ago

huanghaoa commented 7 years ago

I download this project,and configure,make,there will be errors,like this mxmldoc.c:3981:37: error: ‘AE_IFREG’ undeclared (first use in this function) archive_entry_set_filetype(entry, AE_IFREG); Did it lack the libarchive or something?

greendow commented 7 years ago

I compiled mini-XML 2.10 on 64-bit CentOS with GCC 4.4.7. No errors occurred.

huanghaoa commented 7 years ago

@greendow Thanks you,man.I mean I complied mini-xml code in the github now.Maybe I should wait for mini-XML 2.11.And mini-XML 2.10 had the memory leak in the mxmlNodeDelete,Author has fixed it in project now,look out.Forgive my English.

greendow commented 7 years ago

I also hope memory leak can be fixed in the next version. I used many open-source XML library developed with C, including mini-XML, libroxml, etc. Almost every one has memory leak problems more or less. Mini-XML does well and I hope it can go further.

huanghaoa commented 7 years ago

@greendow Author has fixed the memory leak. detail at https://github.com/michaelrsweet/mxml/commit/1272c3dfdb967f49ebb502f28ba3eb56e2360f54

michaelrsweet commented 7 years ago

Sorry, I pushed some changes to mxmldoc so that it can generate ePub books but haven't added all of the conditional code yet for systems without libarchive... it's coming...

michaelrsweet commented 7 years ago

The build issues should now be resolved.