'make all' fails (was: Re: revised T42 support)

[John Mandereau]

Le mardi 08 mai 2007 à 15:18 +0200, Werner LEMBERG a écrit :
> > The attached file, ttf.cc, contains completely revised support for
> > conversion of TTFs to T42 fonts.
> 
> Here an updated version which handles the case where a single glyph
> has more than 64k data.

Hi,

'make all' fails with current Git on my i686 Fedora Core box with GCC
4.1.1 20070105 (Red Hat 4.1.1-51):

rm -f ./out/ttf.dep; DEPENDENCIES_OUTPUT="./out/ttf.dep ./out/ttf.o" g++
-c -Woverloaded-virtual -DHAVE_CONFIG_H  -DNDEBUG -I./include -I./out
-I../flower/include -I../flower/./out -I../flower/include  -O2
-finline-functions -g -pipe -pthread -I/usr/include/freetype2
-I/usr/include/pango-1.0 -I/usr/include/freetype2
-I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include
-Wno-pmf-conversions  -W -Wall -Wconversion -o out/ttf.o ttf.cc
ttf.cc: In function 'void t42_write_table(void*, FT_FaceRec_*, const
unsigned char*, size_t, bool, FT_ULong, FT_ULong)':
ttf.cc:185: error: expected `;' before 'delete'
make[1]: *** [out/ttf.o] Error 1
make[1]: Leaving directory `/home/lilycvs/git/lily/m-trans/lily'
make: *** [all] Erreur 2


My C++ skills are very limited: the most straightforward fix is adding a
semicolon at end of line 183 in ttf.cc, then 'make' all succeeds, but
I'm not confident enough to push any C++ change to Git.

Cheers
John