It doesn't appear that patch 1/3 of my recent series fixing some
compiling issues with the WINGs examples [1] made it in to this last
update.  Was this an oversight?  I've attached the patch to this email.

Thanks!
Doug

[1] http://lists.windowmaker.org/dev/msg07388.html

On 11/02/2014 06:27 AM, crmafra wrote:
> - Log -----------------------------------------------------------------
> http://repo.or.cz/w/wmaker-crm.git/commit/4a2e202af8c0d9872ea8a7a66aa0f0f72fc40db5
>
> commit 4a2e202af8c0d9872ea8a7a66aa0f0f72fc40db5
> Author: Doug Torrance <dtorra...@monmouthcollege.edu>
> Date:   Thu Oct 30 16:24:47 2014 -0500
>
>     WINGs: Avoid cast from pointer to integer of different size compiler 
> warnings.
>
> diff --git a/WINGs/Examples/fontl.c b/WINGs/Examples/fontl.c
> index 4f4eaabb..a440a22c 100644
> --- a/WINGs/Examples/fontl.c
> +++ b/WINGs/Examples/fontl.c
> @@ -23,6 +23,7 @@
>  #include <stdint.h>
>  #include <WINGs/WINGs.h>
>  #include <WINGs/WUtil.h>
> +#include <inttypes.h>
>  
>  void wAbort()
>  {
> @@ -35,7 +36,7 @@ void show(WMWidget * self, void *data)
>       void *d;
>       WMLabel *l = (WMLabel *) data;
>       d = WMGetHangedData(self);
> -     sprintf(buf, "%i -  0x%x - 0%o", (int)d, (int)d, (int)d);
> +     sprintf(buf, "%"PRIiPTR" -  0x%"PRIxPTR" - 0%"PRIoPTR, (intptr_t) d, 
> (intptr_t) d, (intptr_t) d);
>       WMSetLabelText(l, buf);
>  }
>  
>
> http://repo.or.cz/w/wmaker-crm.git/commit/a47f07cfc30e87fd9ed542ad940a6c46624a5703
>
> commit a47f07cfc30e87fd9ed542ad940a6c46624a5703
> Author: Doug Torrance <dtorra...@monmouthcollege.edu>
> Date:   Thu Oct 30 16:24:46 2014 -0500
>
>     WINGs: Fix unused parameter compiler warnings in examples.
>
> diff --git a/WINGs/Examples/colorpick.c b/WINGs/Examples/colorpick.c
> index 24daa2ce..3d6a2dc4 100644
> --- a/WINGs/Examples/colorpick.c
> +++ b/WINGs/Examples/colorpick.c
> @@ -7,6 +7,7 @@ void showSelectedColor(void *self, void *cdata)
>  {
>       WMColorPanel *panel = (WMColorPanel *) self;
>  
> +     (void) cdata;
>       printf("Selected Color: %sn", 
> WMGetColorRGBDescription(WMGetColorPanelColor(panel)));
>  }
>  
> diff --git a/WINGs/Examples/fontl.c b/WINGs/Examples/fontl.c
> index 6e08a90b..4f4eaabb 100644
> --- a/WINGs/Examples/fontl.c
> +++ b/WINGs/Examples/fontl.c
> @@ -41,6 +41,8 @@ void show(WMWidget * self, void *data)
>  
>  void quit(WMWidget * self, void *data)
>  {
> +     (void) self;
> +     (void) data;
>       exit(0);
>  }
>  
> diff --git a/WINGs/Examples/puzzle.c b/WINGs/Examples/puzzle.c
> index 0c50b75b..ca1af101 100644
> --- a/WINGs/Examples/puzzle.c
> +++ b/WINGs/Examples/puzzle.c
> @@ -155,6 +155,9 @@ static void resizeObserver(void *self, WMNotification * 
> notif)
>       WMSize size = WMGetViewSize(WMWidgetView(win));
>       int x, y;
>  
> +     (void) self;
> +     (void) notif;
> +
>       WinSize = size.width;
>       for (y = 0; y < Size; y++) {
>               for (x = 0; x < Size; x++) {
>

From 36318e1081c3780453fbebfafb9b9001d8154d8f Mon Sep 17 00:00:00 2001
From: Doug Torrance <dtorra...@monmouthcollege.edu>
Date: Thu, 30 Oct 2014 12:27:23 -0500
Subject: [PATCH 1/3] WINGs: Link examples against Xlib.

The font lister WINGs example program was failing to build because it uses
the XOpenDisplay() function from Xlib.
---
 WINGs/Examples/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/WINGs/Examples/Makefile.am b/WINGs/Examples/Makefile.am
index 4ddaaf2..cfab01f 100644
--- a/WINGs/Examples/Makefile.am
+++ b/WINGs/Examples/Makefile.am
@@ -8,7 +8,7 @@ noinst_PROGRAMS = fontl puzzle colorpick
 
 LDADD= $(top_builddir)/WINGs/libWINGs.la $(top_builddir)/wrlib/libwraster.la \
 	$(top_builddir)/WINGs/libWUtil.la \
-	@XFTLIBS@ @INTLIBS@
+	@XFTLIBS@ @INTLIBS@ @XLIBS@
 
 colorpick_DEPENDENCIES = $(top_builddir)/WINGs/libWINGs.la
 
-- 
1.9.1

Reply via email to