Hi Hin-Tak, These macros were never used before. I fixed them. Now I think they made the code less readable and I might revert to the old code.
Thanks, Alexei On Wed, May 22, 2024 at 6:12 PM Hin-Tak Leung <ht...@users.sourceforge.net> wrote: > > Actually it might be a good idea to stick CC=g++/clang++ as an additional job > in .gitlab-ci.yml ? I mean, it already does gcc and clang. > > On Wednesday 22 May 2024 at 23:05:47 BST, Hin-Tak Leung > <ht...@users.sourceforge.net> wrote: > > > Should be obvious - needs casting from "void *" to "unsigned char *" and > etc... > > Shouldn't be too hard to see yourself with CC=c++ when building... > > In file included from > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/src/truetype/truetype.c:22: > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/src/truetype/ttgload.c:421:14: > error: assigning to 'FT_Byte *' (aka 'unsigned char *') from incompatible > type 'FT_Pointer' (aka 'void *') > 421 | if ( FT_DUP( exec->glyphIns, p, n_ins ) ) > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/include/freetype/internal/ftmemory.h:378:29: > note: expanded from macro 'FT_DUP' > 378 | FT_MEM_SET_ERROR( FT_MEM_DUP( dst, address, size ) ) > | ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/include/freetype/internal/ftmemory.h:375:19: > note: expanded from macro 'FT_MEM_DUP' > 375 | (dst) = ft_mem_dup( memory, (address), (FT_ULong)(size), > &error ) > | ^ > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/include/freetype/internal/ftmemory.h:231:38: > note: expanded from macro 'FT_MEM_SET_ERROR' > 231 | #define FT_MEM_SET_ERROR( cond ) ( (cond), error != 0 ) > | ^~~~ > In file included from > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/src/truetype/truetype.c:23: > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/src/truetype/ttgxvar.c:2718:12: > error: assigning to 'FT_MM_Var *' (aka 'FT_MM_Var_ *') from incompatible > type 'FT_Pointer' (aka 'void *') > 2718 | if ( FT_DUP( mmvar, ttface->blend->mmvar, > ttface->blend->mmvar_len ) ) > | > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/include/freetype/internal/ftmemory.h:378:29: > note: expanded from macro 'FT_DUP' > 378 | FT_MEM_SET_ERROR( FT_MEM_DUP( dst, address, size ) ) > | ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/include/freetype/internal/ftmemory.h:375:19: > note: expanded from macro 'FT_MEM_DUP' > 375 | (dst) = ft_mem_dup( memory, (address), (FT_ULong)(size), > &error ) > | ^ > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/include/freetype/internal/ftmemory.h:231:38: > note: expanded from macro 'FT_MEM_SET_ERROR' > 231 | #define FT_MEM_SET_ERROR( cond ) ( (cond), error != 0 ) > | ^~~~ > 2 errors generated. > make: *** > [/__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/src/truetype/rules.mk:60: > /__w/freetype2-demos-skia/freetype2-demos-skia/freetype2/objs/truetype.lo] > Error 1 > /__w/freetype2-demos-skia/freetype2-demos-skia > Error: Process completed with exit code 1. -- Alexei A. Podtelezhnikov, PhD
