Re: Gitlab-ci? Re: Another c++ compile failure...

[Vincent Torri]

hello

I don't understand the interest of compiling Freetype with a c++
compiler. The source code is written in C. By convention, CC is
usually for C compilers. To compile C++ code, CXX env var should be
used, not CC

best regards

Vincent Torri

On Thu, May 23, 2024 at 3:16 AM Alexei Podtelezhnikov
<apodtele@gmail.com> wrote:
>
> 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
> <htl10@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 
> > <htl10@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
>