roelj / inklingreader

A GNU/Linux-friendly version of the Wacom Inkling SketchManager.
GNU General Public License v3.0
50 stars 16 forks source link

CLANG support #10

Closed roelj closed 10 years ago

roelj commented 10 years ago

I see @su-v has made the code compatible with CLANG. As far as I can see it doesn't conflict with the recent changes. Could you (@su-v) send a pull request so I can easily merge your changes into here?

su-v commented 10 years ago

Hi!

Could you (…) send a pull request so I can easily merge your changes into here?

Yes, of course - I do have caveats though:

roelj commented 10 years ago

I think that's fine. As far as I can see merging will go smoothly, even if it's behind the current commit. It's definitely useful! :)

su-v commented 10 years ago

Here's a screenshot on OS X Lion with sample WPI file in inklingreader (in the background) and the exported SVG file opened in Inkscape: https://www.dropbox.com/s/56n6n2ef5qwz5mu/inklingreader-osx-1.png

roelj commented 10 years ago

After merging I produced the same with CLANG on Debian 7.4 (including the compiler warning). Thanks for the pull request and the screenshot.