blong42 / clearsilver

Official repository for the ClearSilver templating language and webkit
http://www.clearsilver.net/
Other
57 stars 25 forks source link

Support compiling 0.10.5 with gcc 5.x #12

Closed bkylerussell closed 8 years ago

bkylerussell commented 8 years ago

The main goal of this change is to support compiling 0.10.5 with gcc 5.2 (see my commit). The other changes in this pull request come from the differences between clearsilver-0.10.5 and master.

I based this off clearsilver-0.10.5 because that's the release I was using, though I can see this doesn't merge cleanly with master. It looks like 0.10.5 contains changes to util/neo_err.h that cause the incompatibility with gcc 5.x, though I don't see the changes to util/neo_err.h in any other commit on the master branch (essentially 0.10.5 seems to have been released off a point other than where it's actually tagged).

Can you help point me to where I should be pulling the latest version of the source? Is HEAD of master newer or older than 0.10.5?

bkylerussell commented 8 years ago

I'm going to close this since I just saw your message that HEAD of master is in fact newer than 0.10.5. It looks like this has been resolved differently on master by d29b6632a382d98aaf9a2e833d8bd5e26c3e60a5 which just adjusts the number of arguments in the macro to allow for the nerr_raise() usages. Thanks!