Hello, Werner and friends, I've just implemented the fix to the second issue that had been originally pointed out: Now the API uses enums to refer to rasterization errors and warnings, instead of text strings. I've also made sure to isolate all the diagnostics API inside ifdef blocks.
The example HintingDebugger C client was updated accordingly and now has a header file with all the title and description text strings for each of those enum error codes: https://gitlab.com/fsanches/HintingDebugger The API fixes are in these 3 commits: https://github.com/felipesanches/freetype2/commit/ dac77c3aeebbf2eee1647d3b95f5b5e3bf9d2db0 https://github.com/felipesanches/freetype2/commit/ 51ec98b528b395af25f03e8b3824964feaa5a89e https://github.com/felipesanches/freetype2/commit/ae9b909a327ef3877977e279eb13c80004d9c34a The remaining commits in that repo contain a few of the error/warning DIAGNOSTICS macro calls. The rest of HinTak's original patches were not published yet, but would be handy now so that we can integrate them as well. The above-mentioned code changes address all issues that were originally brought up (the couple global vars and usage of enums instead of text strings). Cheers, Felipe Sanches 2017-09-11 14:20 GMT-03:00 Felipe Sanches <[email protected]>: > All the testing of this was done with this minimalist C client for the > proposed diagnostics API: > https://github.com/felipesanches/HintingDebugger > > Happy Hacking, > Felipe Sanches > > 2017-09-11 11:57 GMT-03:00 Felipe Sanches <[email protected]>: > >> Hello, Werner, >> >> I've moved the diagnostics callback field into FT_Face_Internal. >> The current proposal to fix to the globals issue is the aggregate of the >> following 2 commits: >> >> https://github.com/felipesanches/freetype2/commit/dac77c3aee >> bbf2eee1647d3b95f5b5e3bf9d2db0 >> https://github.com/felipesanches/freetype2/commit/51ec98b528 >> b395af25f03e8b3824964feaa5a89e >> >> Please let me know if that's a good way of addressing the issue. >> >> Cheers, >> Felipe Sanches >> >> 2017-09-09 3:08 GMT-03:00 Felipe Sanches <[email protected]>: >> >>> Cool, thanks for the quick feedback! I'll see what can do about it ;-) >>> >>> 2017-09-09 3:05 GMT-03:00 Werner LEMBERG <[email protected]>: >>> >>>> >>>> > I have adapted HinTak's b06 patch to address the globals usage issue >>>> > that was originally brought up by Behdad. He had suggested moving >>>> > those globals to FT_Library, but I did place the diagnostics >>>> > callback into the FT_Face struct, instead. Let me know if that's >>>> > good. >>>> >>>> No time to look right now. However, FT_Face is a public structure and >>>> thus taboo – you *must not* modify it! Use FT_Face_Internal instead. >>>> >>>> >>>> Werner >>>> >>> >>> >> >
_______________________________________________ Freetype-devel mailing list [email protected] https://lists.nongnu.org/mailman/listinfo/freetype-devel
