Skip to content

Commit

Permalink
Enable std GNU99
Browse files Browse the repository at this point in the history
  • Loading branch information
hughbe committed Sep 30, 2017
1 parent 61a3dd1 commit 8ad8abc
Show file tree
Hide file tree
Showing 5 changed files with 32 additions and 28 deletions.
2 changes: 1 addition & 1 deletion configure.ac
Expand Up @@ -31,7 +31,7 @@ PKG_CHECK_MODULES(CAIRO, cairo >= $CAIRO_REQUIRED_VERSION)
# Optional use (experimental and unsupported) of Pango's text rendering on top of Cairo
AC_ARG_WITH(pango, [ --with-pango],[text_v=pango],[text_v=cairo])

GDIPLUS_CFLAGS="$GDIPLUS_CFLAGS -Wall -Wextra -Wno-unused-parameter -Wno-sign-compare"
GDIPLUS_CFLAGS="$GDIPLUS_CFLAGS -Wall -Wextra -Wno-unused-parameter -Wno-sign-compare -std=gnu99"

AC_ARG_ENABLE(warnaserror, AS_HELP_STRING([--enable-warnaserror],[Enable treating warnings as errors.]),[warnaserror=yes],[warnaserror=no])

Expand Down
6 changes: 6 additions & 0 deletions src/carbon-private.c
Expand Up @@ -40,4 +40,10 @@ void gdip_get_display_dpi_carbon (float *h_dpi, float *v_dpi) {
}
}
}
#else
#include <glib.h>

void gdip_get_display_dpi_carbon (float *h_dpi, float *v_dpi) {
g_assert_not_reached ();
}
#endif
2 changes: 0 additions & 2 deletions src/carbon-private.h
Expand Up @@ -23,8 +23,6 @@
#ifndef __CARBON_PRIVATE_H__
#define __CARBON_PRIVATE_H__

#ifdef __APPLE__
void gdip_get_display_dpi_carbon (float *h_dpi, float *v_dpi);
#endif

#endif
44 changes: 22 additions & 22 deletions src/metafile.c
Expand Up @@ -983,12 +983,12 @@ gdip_metafile_play_setup (GpMetafile *metafile, GpGraphics *graphics, int x, int
switch (context->metafile->metafile_header.Type) {
case METAFILETYPE_WMFPLACEABLE:
case METAFILETYPE_WMF:
context->objects_count = metafile->metafile_header.WmfHeader.mtNoObjects;
context->objects_count = metafile->metafile_header.Header.Wmf.mtNoObjects;
break;
case METAFILETYPE_EMF:
case METAFILETYPE_EMFPLUSONLY:
case METAFILETYPE_EMFPLUSDUAL:
context->objects_count = metafile->metafile_header.EmfHeader.nHandles + 1; /* 0 is reserved */
context->objects_count = metafile->metafile_header.Header.Emf.nHandles + 1; /* 0 is reserved */
break;
default:
GdipFree (context);
Expand Down Expand Up @@ -1107,13 +1107,13 @@ static void
MetafileHeaderLE (MetafileHeader *header)
{
#if G_BYTE_ORDER != G_LITTLE_ENDIAN
header->WmfHeader.mtType = GUINT16_FROM_LE (header->WmfHeader.mtType);
header->WmfHeader.mtHeaderSize = GUINT16_FROM_LE (header->WmfHeader.mtHeaderSize);
header->WmfHeader.mtVersion = GUINT16_FROM_LE (header->WmfHeader.mtVersion);
header->WmfHeader.mtSize = GUINT32_FROM_LE (header->WmfHeader.mtSize);
header->WmfHeader.mtNoObjects = GUINT16_FROM_LE (header->WmfHeader.mtNoObjects);
header->WmfHeader.mtMaxRecord = GUINT32_FROM_LE (header->WmfHeader.mtMaxRecord);
header->WmfHeader.mtNoParameters = GUINT16_FROM_LE (header->WmfHeader.mtNoParameters);
header->Header.Wmf.mtType = GUINT16_FROM_LE (header->Header.Wmf.mtType);
header->Header.Wmf.mtHeaderSize = GUINT16_FROM_LE (header->Header.Wmf.mtHeaderSize);
header->Header.Wmf.mtVersion = GUINT16_FROM_LE (header->Header.Wmf.mtVersion);
header->Header.Wmf.mtSize = GUINT32_FROM_LE (header->Header.Wmf.mtSize);
header->Header.Wmf.mtNoObjects = GUINT16_FROM_LE (header->Header.Wmf.mtNoObjects);
header->Header.Wmf.mtMaxRecord = GUINT32_FROM_LE (header->Header.Wmf.mtMaxRecord);
header->Header.Wmf.mtNoParameters = GUINT16_FROM_LE (header->Header.Wmf.mtNoParameters);
#endif
}

Expand Down Expand Up @@ -1169,8 +1169,8 @@ combine_headers (GDIPCONST WmfPlaceableFileHeader *wmfPlaceableFileHeader, Metaf
header->DpiX = gdip_get_display_dpi ();
header->DpiY = header->DpiX;
}
header->Size = header->WmfHeader.mtSize * 2;
header->Version = header->WmfHeader.mtVersion;
header->Size = header->Header.Wmf.mtSize * 2;
header->Version = header->Header.Wmf.mtVersion;
header->EmfPlusFlags = 0;
header->EmfPlusHeaderSize = 0;
header->LogicalDpiX = 0;
Expand Down Expand Up @@ -1237,30 +1237,30 @@ g_warning ("ALDUS_PLACEABLE_METAFILE key %d, hmf %d, L %d, T %d, R %d, B %d, inc
aldus_header.Checksum);
#endif
size = sizeof (METAHEADER);
if (gdip_read_wmf_data (pointer, (BYTE*)&header->WmfHeader, size, source) != size)
if (gdip_read_wmf_data (pointer, (BYTE*)&header->Header.Wmf, size, source) != size)
return InvalidParameter;

WmfPlaceableFileHeaderLE (&aldus_header);
MetafileHeaderLE (header);
status = combine_headers (&aldus_header, header);
break;
case WMF_TYPE_AND_HEADERSIZE_KEY:
memcpy (&header->WmfHeader, &key, size);
memcpy (&header->Header.Wmf, &key, size);

size = sizeof (METAHEADER) - size;
if (gdip_read_wmf_data (pointer, (BYTE*)(&header->WmfHeader) + sizeof(DWORD), size, source) != size)
if (gdip_read_wmf_data (pointer, (BYTE*)(&header->Header.Wmf) + sizeof(DWORD), size, source) != size)
return InvalidParameter;

MetafileHeaderLE (header);
status = combine_headers (NULL, header);
break;
case EMF_EMR_HEADER_KEY:
emf = &(header->EmfHeader);
emf = &(header->Header.Emf);
emf->iType = key;
size = sizeof (ENHMETAHEADER3) - size;
if (gdip_read_emf_data (pointer, (BYTE*)(&header->EmfHeader) + sizeof (DWORD), size, source) != size)
if (gdip_read_emf_data (pointer, (BYTE*)(&header->Header.Emf) + sizeof (DWORD), size, source) != size)
return InvalidParameter;
EnhMetaHeaderLE (&header->EmfHeader);
EnhMetaHeaderLE (&header->Header.Emf);
#if FALSE
g_warning ("EMF HEADER iType %d, nSize %d, Bounds L %d, T %d, R %d, B %d, Frame L %d, T %d, R %d, B %d, signature %X, version %d, bytes %d, records %d, handles %d, reserved %d, description %d, %d, palentries %d, device %d, %d, millimeters %d, %d",
emf->iType, emf->nSize,
Expand Down Expand Up @@ -1321,9 +1321,9 @@ g_warning ("EMF HEADER iType %d, nSize %d, Bounds L %d, T %d, R %d, B %d, Frame

#if FALSE
g_warning ("METAHEADER type %d, header %d, version %d, size %d, #obj %d, max rec %d, #params %d",
header->WmfHeader.mtType, header->WmfHeader.mtHeaderSize, header->WmfHeader.mtVersion,
header->WmfHeader.mtSize, header->WmfHeader.mtNoObjects, header->WmfHeader.mtMaxRecord,
header->WmfHeader.mtNoParameters);
header->Header.Wmf.mtType, header->Header.Wmf.mtHeaderSize, header->Header.Wmf.mtVersion,
header->Header.Wmf.mtSize, header->Header.Wmf.mtNoObjects, header->Header.Wmf.mtMaxRecord,
header->Header.Wmf.mtNoParameters);
#endif
return status;
}
Expand All @@ -1347,13 +1347,13 @@ gdip_get_metafile_from (void *pointer, GpMetafile **metafile, ImageSource source
case METAFILETYPE_WMF:
mf->base.image_format = WMF;
/* note: mtSize is in WORD, mtSize contains the METAHEADER, mf->length is in bytes */
mf->length = mf->metafile_header.WmfHeader.mtSize * 2 - sizeof (METAHEADER);
mf->length = mf->metafile_header.Header.Wmf.mtSize * 2 - sizeof (METAHEADER);
break;
case METAFILETYPE_EMF:
case METAFILETYPE_EMFPLUSONLY:
case METAFILETYPE_EMFPLUSDUAL:
mf->base.image_format = EMF;
mf->length = mf->metafile_header.EmfHeader.nBytes - mf->metafile_header.EmfHeader.nSize;
mf->length = mf->metafile_header.Header.Emf.nBytes - mf->metafile_header.Header.Emf.nSize;
adjust_emf_headers = TRUE;
break;
}
Expand Down
6 changes: 3 additions & 3 deletions src/win32structs.h
Expand Up @@ -308,9 +308,9 @@ typedef struct {
int Width;
int Height;
union {
METAHEADER WmfHeader;
ENHMETAHEADER3 EmfHeader;
};
METAHEADER Wmf;
ENHMETAHEADER3 Emf;
} Header;
int EmfPlusHeaderSize;
int LogicalDpiX;
int LogicalDpiY;
Expand Down

0 comments on commit 8ad8abc

Please sign in to comment.