Re: [ft-devel] Upstreaming freetype diagnostics patches

[Felipe Sanches]

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 <address@hidden>:

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 <address@hidden>:

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/dac77c3aeebbf2eee1647d3b95f5b5e3bf9d2db0
https://github.com/felipesanches/freetype2/commit/51ec98b528b395af25f03e8b3824964feaa5a89e

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 <address@hidden>:

Cool, thanks for the quick feedback! I'll see what  can do about it ;-)

2017-09-09 3:05 GMT-03:00 Werner LEMBERG <address@hidden>:

> 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