summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWerner Lemberg <wl@gnu.org>2023-01-16 16:38:56 +0100
committerWerner Lemberg <wl@gnu.org>2023-01-16 16:38:56 +0100
commit6a179ff7d55714501c5efce85e2a47de6055a07a (patch)
tree66e3e0651ac7d9358e9eed50e7a7fe7126af5a74
parent2692b3215be4f106b714974c55f4ab80da25189c (diff)
downloadfreetype2-6a179ff7d55714501c5efce85e2a47de6055a07a.tar.gz
sr/*.c: Various minor fixes.
* src/autofit/ft-hb.c (_hb_ft_reference_table): Call `FT_UNUSED` after variable declarations. * src/gxvalid/gxvjust.c (gxv_just_widthDeltaClusters_validate): Eliminate unused variable. * src/gzip/ftgzip.c: Don't call GCC '-Wstrict-prototypes' pragma for C++ compiler. * src/sfnt/ttcolr.c (ENSURE_READ_BYTES): Remove final semicolon to avoid compiler warning. * src/sfnt/ttsvg.c (tt_face_load_svg_doc): Fix signedness warning.
-rw-r--r--src/autofit/ft-hb.c3
-rw-r--r--src/gxvalid/gxvjust.c3
-rw-r--r--src/gzip/ftgzip.c2
-rw-r--r--src/sfnt/ttcolr.c2
-rw-r--r--src/sfnt/ttsvg.c3
5 files changed, 8 insertions, 5 deletions
diff --git a/src/autofit/ft-hb.c b/src/autofit/ft-hb.c
index 1958f9c36..7ba1bfa55 100644
--- a/src/autofit/ft-hb.c
+++ b/src/autofit/ft-hb.c
@@ -38,12 +38,13 @@
static hb_blob_t *
_hb_ft_reference_table (hb_face_t *face, hb_tag_t tag, void *user_data)
{
- FT_UNUSED (face);
FT_Face ft_face = (FT_Face) user_data;
FT_Byte *buffer;
FT_ULong length = 0;
FT_Error error;
+ FT_UNUSED (face);
+
/* Note: FreeType like HarfBuzz uses the NONE tag for fetching the entire blob */
error = FT_Load_Sfnt_Table (ft_face, tag, 0, NULL, &length);
diff --git a/src/gxvalid/gxvjust.c b/src/gxvalid/gxvjust.c
index 6af2c79c8..389a15516 100644
--- a/src/gxvalid/gxvjust.c
+++ b/src/gxvalid/gxvjust.c
@@ -156,7 +156,6 @@
{
FT_Bytes p = table;
FT_Bytes wdc_end = table + GXV_JUST_DATA( wdc_offset_max );
- FT_UInt i;
GXV_NAME_ENTER( "just justDeltaClusters" );
@@ -164,7 +163,7 @@
if ( limit <= wdc_end )
FT_INVALID_OFFSET;
- for ( i = 0; p <= wdc_end; i++ )
+ while ( p <= wdc_end )
{
gxv_just_wdc_entry_validate( p, limit, gxvalid );
p += gxvalid->subtable_length;
diff --git a/src/gzip/ftgzip.c b/src/gzip/ftgzip.c
index b415e9d6c..f7ffebdcb 100644
--- a/src/gzip/ftgzip.c
+++ b/src/gzip/ftgzip.c
@@ -90,7 +90,9 @@
#if defined( __GNUC__ )
#pragma GCC diagnostic push
+#ifndef __cplusplus
#pragma GCC diagnostic ignored "-Wstrict-prototypes"
+#endif
#pragma GCC diagnostic ignored "-Wimplicit-fallthrough"
#pragma GCC diagnostic ignored "-Wredundant-decls"
#endif
diff --git a/src/sfnt/ttcolr.c b/src/sfnt/ttcolr.c
index d6693d773..710357c60 100644
--- a/src/sfnt/ttcolr.c
+++ b/src/sfnt/ttcolr.c
@@ -61,7 +61,7 @@
#define ENSURE_READ_BYTES( byte_size ) \
if ( p < colr->paints_start_v1 || \
p > (FT_Byte*)colr->table + colr->table_size - byte_size ) \
- return 0;
+ return 0
typedef enum FT_PaintFormat_Internal_
diff --git a/src/sfnt/ttsvg.c b/src/sfnt/ttsvg.c
index 411813a98..22b9a4bb1 100644
--- a/src/sfnt/ttsvg.c
+++ b/src/sfnt/ttsvg.c
@@ -310,7 +310,8 @@
if ( error != FT_Err_Ok )
goto Exit;
- doc_limit = svg->table_size - ( doc_list - (FT_Byte*)svg->table );
+ doc_limit = svg->table_size -
+ (FT_ULong)( doc_list - (FT_Byte*)svg->table );
if ( doc_offset > doc_limit ||
doc_length > doc_limit - doc_offset )
{