git: 5cd4319f34f1 - main - Revert "x11-toolkits/open-motif*: EditRes functionality is unreliable on 64 bit"

From: Cy Schubert <cy_at_FreeBSD.org>
Date: Tue, 05 Dec 2023 22:50:36 UTC
The branch main has been updated by cy:

URL: https://cgit.FreeBSD.org/ports/commit/?id=5cd4319f34f19678e6f8f115362d820b550d4c5b

commit 5cd4319f34f19678e6f8f115362d820b550d4c5b
Author:     Cy Schubert <cy@FreeBSD.org>
AuthorDate: 2023-12-05 22:49:10 +0000
Commit:     Cy Schubert <cy@FreeBSD.org>
CommitDate: 2023-12-05 22:49:10 +0000

    Revert "x11-toolkits/open-motif*: EditRes functionality is unreliable on 64 bit"
    
    A patch to x11-toolkits/libXmu, still in review, is required to,
    
    a) build this patch,
    b) work.
    
    This reverts commit feec232fadd3d30c62282b6dd218536b7322e761.
---
 x11-toolkits/open-motif-devel/Makefile             |   2 +-
 .../files/patch-lib_Xm_EditresCom.c                | 127 ---------------------
 x11-toolkits/open-motif/Makefile                   |   2 +-
 .../open-motif/files/patch-lib_Xm_EditresCom.c     | 127 ---------------------
 4 files changed, 2 insertions(+), 256 deletions(-)

diff --git a/x11-toolkits/open-motif-devel/Makefile b/x11-toolkits/open-motif-devel/Makefile
index d64c1dbe67d9..ec74a5e473a9 100644
--- a/x11-toolkits/open-motif-devel/Makefile
+++ b/x11-toolkits/open-motif-devel/Makefile
@@ -1,6 +1,6 @@
 PORTNAME=	open-motif
 PORTVERSION=	${COMMIT_DATE}
-PORTREVISION=	7
+PORTREVISION=	6
 CATEGORIES=	x11-toolkits
 PKGNAMESUFFIX=	-devel
 
diff --git a/x11-toolkits/open-motif-devel/files/patch-lib_Xm_EditresCom.c b/x11-toolkits/open-motif-devel/files/patch-lib_Xm_EditresCom.c
deleted file mode 100644
index 6eff286772da..000000000000
--- a/x11-toolkits/open-motif-devel/files/patch-lib_Xm_EditresCom.c
+++ /dev/null
@@ -1,127 +0,0 @@
---- lib/Xm/EditresCom.c.orig	2023-02-15 16:42:34.000000000 -0800
-+++ lib/Xm/EditresCom.c	2023-10-16 20:14:52.602297000 -0700
-@@ -183,6 +183,39 @@
- }
- #endif
- 
-+#define HAVE_XMU_GET_WIDGET_INFO_RESOLVE 1
-+#ifdef HAVE_XMU_GET_WIDGET_INFO_RESOLVE
-+static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  return _XEditResGetWidgetInfoResolve(stream, info);
-+}
-+#else
-+static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  return _XEditResGetWidgetInfo(stream, info);
-+}
-+#endif
-+
-+#define HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE 1
-+#ifdef HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE
-+static void PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  _XEditResPutWidgetInfoUnresolve(stream, info);
-+}
-+#else
-+static Boolean PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  _XEditResPutWidgetInfo(stream, info);
-+}
-+#endif
-+
-+#define HAVE_XMU_PUT_WIDGET_UNRESOLVE 1
-+#ifdef HAVE_XMU_PUT_WIDGET_UNRESOLVE
-+static void PutWidget(ProtocolStream *stream, Widget widget) {
-+  _XEditResPutWidgetUnresolve(stream, widget);
-+}
-+#else
-+static void PutWidget(ProtocolStream *stream, Widget widget) {
-+  _XEditResPut32(stream, widget);
-+}
-+#endif
-+
- /************************************************************
-  *
-  * Resource Editor Communication Code
-@@ -352,7 +385,7 @@
- 		XtCalloc(sizeof(WidgetInfo), sv_event->num_entries);
- 	    
- 	    for (i = 0; i < sv_event->num_entries; i++) {
--		if (!_XEditResGetWidgetInfo(stream, sv_event->widgets + i))
-+		if (!GetWidgetInfo(stream, sv_event->widgets + i))
- 		    goto done;
- 	    }
- 	}
-@@ -364,7 +397,7 @@
- 	    find_event->widgets = (WidgetInfo *) 
- 		                  XtCalloc(sizeof(WidgetInfo), 1);
- 
--	    if (!(_XEditResGetWidgetInfo(stream, find_event->widgets) &&
-+	    if (!(GetWidgetInfo(stream, find_event->widgets) &&
- 		  _XEditResGetSigned16(stream, &(find_event->x)) &&
- 		  _XEditResGetSigned16(stream, &(find_event->y))))
- 	    {
-@@ -384,7 +417,7 @@
- 	    get_event->widgets = (WidgetInfo *)
- 		XtCalloc(sizeof(WidgetInfo), get_event->num_entries);
- 	    for (i = 0; i < get_event->num_entries; i++) {
--		if (!_XEditResGetWidgetInfo(stream, get_event->widgets + i)) 
-+		if (!GetWidgetInfo(stream, get_event->widgets + i))
- 		    goto done;
- 	    }
- 	}
-@@ -397,7 +430,7 @@
-             _XEditResGet16(stream, &(gv_event->num_entries));
- 	    gv_event->widgets = (WidgetInfo *)
- 		XtCalloc(sizeof(WidgetInfo), gv_event->num_entries);
--            _XEditResGetWidgetInfo(stream, gv_event->widgets);
-+            GetWidgetInfo(stream, gv_event->widgets);
-         }
-         break;	
- 
-@@ -846,7 +879,7 @@
- 
-     for (i = 0 ; i < sv_event->num_entries; i++) {
- 	if ((str = VerifyWidget(w, &(sv_event->widgets[i]))) != NULL) {
--	    _XEditResPutWidgetInfo(stream, &(sv_event->widgets[i]));
-+	    PutWidgetInfo(stream, &(sv_event->widgets[i]));
- 	    _XEditResPutString8(stream, str);
- 	    XtFree(str);
- 	    count++;
-@@ -973,7 +1006,7 @@
-      */ 
- 
-     (*(info->count))++;
--    _XEditResPutWidgetInfo(info->stream, info->entry);
-+    PutWidgetInfo(info->stream, info->entry);
-     _XEditResPutString8(info->stream, pbuf);
-     if (pbuf != buf)
- 	XtFree(pbuf);
-@@ -1174,7 +1207,7 @@
- 	 * Send out the widget id. 
- 	 */
- 
--	_XEditResPutWidgetInfo(stream, &(geom_event->widgets[i]));
-+	PutWidgetInfo(stream, &(geom_event->widgets[i]));
- 	if ((str = VerifyWidget(w, &(geom_event->widgets[i]))) != NULL) {
- 	    _XEditResPutBool(stream, True); /* an error occured. */
- 	    _XEditResPutString8(stream, str);	/* set message. */
-@@ -1420,7 +1453,7 @@
- 	/* 
- 	 * Send out the widget id. 
- 	 */
--	_XEditResPutWidgetInfo(stream, &(res_event->widgets[i]));
-+	PutWidgetInfo(stream, &(res_event->widgets[i]));
- 	if ((str = VerifyWidget(w, &(res_event->widgets[i]))) != NULL) {
- 	    _XEditResPutBool(stream, True); /* an error occured. */
- 	    _XEditResPutString8(stream, str);	/* set message. */
-@@ -1766,10 +1799,11 @@
- 	
-     _XEditResPut16(stream, num_widgets);	/* insert number of widgets. */
-     for (i = 0; i < num_widgets; i++) /* insert Widgets themselves. */
--	_XEditResPut32(stream, widget_list[i]);
-+	PutWidget(stream, (Widget)widget_list[i]);
-     
-     XtFree((char *)widget_list);
- }
-+
- 
- #if 0
- /************************************************************
diff --git a/x11-toolkits/open-motif/Makefile b/x11-toolkits/open-motif/Makefile
index 91f5f39f22c9..2723ca4a7381 100644
--- a/x11-toolkits/open-motif/Makefile
+++ b/x11-toolkits/open-motif/Makefile
@@ -1,6 +1,6 @@
 PORTNAME=	open-motif
 PORTVERSION=	2.3.8
-PORTREVISION=	8
+PORTREVISION=	7
 CATEGORIES=	x11-toolkits
 # MASTER_SITES=	SF/${PORTNAME}/Motif%202.3.4%20Source%20Code/
 MASTER_SITES=	SF/motif/Motif%20${PORTVERSION}%20Source%20Code/
diff --git a/x11-toolkits/open-motif/files/patch-lib_Xm_EditresCom.c b/x11-toolkits/open-motif/files/patch-lib_Xm_EditresCom.c
deleted file mode 100644
index 6eff286772da..000000000000
--- a/x11-toolkits/open-motif/files/patch-lib_Xm_EditresCom.c
+++ /dev/null
@@ -1,127 +0,0 @@
---- lib/Xm/EditresCom.c.orig	2023-02-15 16:42:34.000000000 -0800
-+++ lib/Xm/EditresCom.c	2023-10-16 20:14:52.602297000 -0700
-@@ -183,6 +183,39 @@
- }
- #endif
- 
-+#define HAVE_XMU_GET_WIDGET_INFO_RESOLVE 1
-+#ifdef HAVE_XMU_GET_WIDGET_INFO_RESOLVE
-+static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  return _XEditResGetWidgetInfoResolve(stream, info);
-+}
-+#else
-+static Boolean GetWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  return _XEditResGetWidgetInfo(stream, info);
-+}
-+#endif
-+
-+#define HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE 1
-+#ifdef HAVE_XMU_PUT_WIDGET_INFO_UNRESOLVE
-+static void PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  _XEditResPutWidgetInfoUnresolve(stream, info);
-+}
-+#else
-+static Boolean PutWidgetInfo(ProtocolStream *stream, WidgetInfo *info) {
-+  _XEditResPutWidgetInfo(stream, info);
-+}
-+#endif
-+
-+#define HAVE_XMU_PUT_WIDGET_UNRESOLVE 1
-+#ifdef HAVE_XMU_PUT_WIDGET_UNRESOLVE
-+static void PutWidget(ProtocolStream *stream, Widget widget) {
-+  _XEditResPutWidgetUnresolve(stream, widget);
-+}
-+#else
-+static void PutWidget(ProtocolStream *stream, Widget widget) {
-+  _XEditResPut32(stream, widget);
-+}
-+#endif
-+
- /************************************************************
-  *
-  * Resource Editor Communication Code
-@@ -352,7 +385,7 @@
- 		XtCalloc(sizeof(WidgetInfo), sv_event->num_entries);
- 	    
- 	    for (i = 0; i < sv_event->num_entries; i++) {
--		if (!_XEditResGetWidgetInfo(stream, sv_event->widgets + i))
-+		if (!GetWidgetInfo(stream, sv_event->widgets + i))
- 		    goto done;
- 	    }
- 	}
-@@ -364,7 +397,7 @@
- 	    find_event->widgets = (WidgetInfo *) 
- 		                  XtCalloc(sizeof(WidgetInfo), 1);
- 
--	    if (!(_XEditResGetWidgetInfo(stream, find_event->widgets) &&
-+	    if (!(GetWidgetInfo(stream, find_event->widgets) &&
- 		  _XEditResGetSigned16(stream, &(find_event->x)) &&
- 		  _XEditResGetSigned16(stream, &(find_event->y))))
- 	    {
-@@ -384,7 +417,7 @@
- 	    get_event->widgets = (WidgetInfo *)
- 		XtCalloc(sizeof(WidgetInfo), get_event->num_entries);
- 	    for (i = 0; i < get_event->num_entries; i++) {
--		if (!_XEditResGetWidgetInfo(stream, get_event->widgets + i)) 
-+		if (!GetWidgetInfo(stream, get_event->widgets + i))
- 		    goto done;
- 	    }
- 	}
-@@ -397,7 +430,7 @@
-             _XEditResGet16(stream, &(gv_event->num_entries));
- 	    gv_event->widgets = (WidgetInfo *)
- 		XtCalloc(sizeof(WidgetInfo), gv_event->num_entries);
--            _XEditResGetWidgetInfo(stream, gv_event->widgets);
-+            GetWidgetInfo(stream, gv_event->widgets);
-         }
-         break;	
- 
-@@ -846,7 +879,7 @@
- 
-     for (i = 0 ; i < sv_event->num_entries; i++) {
- 	if ((str = VerifyWidget(w, &(sv_event->widgets[i]))) != NULL) {
--	    _XEditResPutWidgetInfo(stream, &(sv_event->widgets[i]));
-+	    PutWidgetInfo(stream, &(sv_event->widgets[i]));
- 	    _XEditResPutString8(stream, str);
- 	    XtFree(str);
- 	    count++;
-@@ -973,7 +1006,7 @@
-      */ 
- 
-     (*(info->count))++;
--    _XEditResPutWidgetInfo(info->stream, info->entry);
-+    PutWidgetInfo(info->stream, info->entry);
-     _XEditResPutString8(info->stream, pbuf);
-     if (pbuf != buf)
- 	XtFree(pbuf);
-@@ -1174,7 +1207,7 @@
- 	 * Send out the widget id. 
- 	 */
- 
--	_XEditResPutWidgetInfo(stream, &(geom_event->widgets[i]));
-+	PutWidgetInfo(stream, &(geom_event->widgets[i]));
- 	if ((str = VerifyWidget(w, &(geom_event->widgets[i]))) != NULL) {
- 	    _XEditResPutBool(stream, True); /* an error occured. */
- 	    _XEditResPutString8(stream, str);	/* set message. */
-@@ -1420,7 +1453,7 @@
- 	/* 
- 	 * Send out the widget id. 
- 	 */
--	_XEditResPutWidgetInfo(stream, &(res_event->widgets[i]));
-+	PutWidgetInfo(stream, &(res_event->widgets[i]));
- 	if ((str = VerifyWidget(w, &(res_event->widgets[i]))) != NULL) {
- 	    _XEditResPutBool(stream, True); /* an error occured. */
- 	    _XEditResPutString8(stream, str);	/* set message. */
-@@ -1766,10 +1799,11 @@
- 	
-     _XEditResPut16(stream, num_widgets);	/* insert number of widgets. */
-     for (i = 0; i < num_widgets; i++) /* insert Widgets themselves. */
--	_XEditResPut32(stream, widget_list[i]);
-+	PutWidget(stream, (Widget)widget_list[i]);
-     
-     XtFree((char *)widget_list);
- }
-+
- 
- #if 0
- /************************************************************