schnaader / fairytale

encode.ru community archiver
GNU Lesser General Public License v3.0
31 stars 13 forks source link

Added initial JsonParser implementation #23

Closed andrew-epstein closed 6 years ago

andrew-epstein commented 6 years ago

First attempt at writing my own parser. Feedback welcome.

andrew-epstein commented 6 years ago

@schnaader: It looks like the travis checks got stuck. Any idea what's going on?

schnaader commented 6 years ago

Yes, saw that error and fixed it. At the moment we have to add the parser to CMakeLists.txt and both Makefiles. I'll do some expirements in the travis branch, generating the Makefiles from CMake would be better, at least for Travis.

andrew-epstein commented 6 years ago

I agree with removing the Makefile(s). CMake should be able to fully replace them, at least on *nix and OSX.