File nan_xpt2d_add_return_for_BE.patch of Package octave-forge-nan

Overview Repositories Revisions Requests Users Attributes Meta

File nan_xpt2d_add_return_for_BE.patch of Package octave-forge-nan (Revision d405450b1a8b2272180209bcd1814fcc)

Currently displaying revision d405450b1a8b2272180209bcd1814fcc , Show latest

From: Michel Normand <normand@linux.vnet.ibm.com>
Subject: nan xpt2d add return for BE
Date: Tue, 12 Apr 2016 12:02:48 +0200

for xpt2d and d2xpt  add return for BE, to avoid compilation warning
treated as error by OBS build process:
===
I: Program returns random data in a function
E: octave-forge-nan no-return-in-nonvoid-function xptopen.cpp:1062, 1112

I: Program returns random data in a function
E: octave-forge-nan no-return-in-nonvoid-function xptopen.cpp:1062, 1112
===

Signed-off-by: Michel Normand <normand@linux.vnet.ibm.com>
---
 NaN/src/xptopen.cpp |   17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

Index: octave-forge-nan-2.8.1/NaN/src/xptopen.cpp
===================================================================
--- octave-forge-nan-2.8.1.orig/NaN/src/xptopen.cpp
+++ octave-forge-nan-2.8.1/NaN/src/xptopen.cpp
@@ -1027,15 +1027,16 @@ if present.
 
 double xpt2d(uint64_t x) {
 	// x is little-endian 64bit IBM floating point format
-	char c = *((char*)&x+7) & 0x7f;
-	uint64_t u = x;
-	*((char*)&u+7)=0;
-
 #if __BYTE_ORDER == __BIG_ENDIAN
 
 	mexErrMsgTxt("IEEE-to-IBM conversion on big-endian platform not supported, yet");
+	return(NaN);
 
 #elif __BYTE_ORDER==__LITTLE_ENDIAN
+	char c = *((char*)&x+7) & 0x7f;
+	uint64_t u = x;
+	*((char*)&u+7)=0;
+
 
 #if DEBUG
 	mexPrintf("xpt2d(%016Lx): [0x%x]\n",x,c);
@@ -1067,15 +1068,15 @@ double xpt2d(uint64_t x) {
 */
 
 uint64_t d2xpt(double x) {
-	uint64_t s,m;
-	int e;
-
 #if __BYTE_ORDER == __BIG_ENDIAN
 
 	mexErrMsgTxt("IEEE-to-IBM conversion on big-endian platform not supported, yet");
-
+	return(0x2eLL << 56);	// NaN - not a number
 
 #elif __BYTE_ORDER==__LITTLE_ENDIAN
+	uint64_t s,m;
+	int e;
+
 
 	if (x != x) return(0x2eLL << 56);	// NaN - not a number

Places

File nan_xpt2d_add_return_for_BE.patch of Package octave-forge-nan (Revision d405450b1a8b2272180209bcd1814fcc)

Places