From 9ff0d883fcc564ad91221953049a47e0937063fa Mon Sep 17 00:00:00 2001 From: Martin Ebourne Date: Sun, 12 Feb 2006 03:16:57 +0000 Subject: This ppc problem is clearly a common big-endian platform issue. Remove ppc hack and configure option. Replace dubious casts with clearer use of memcpy to make this work on all platforms all of the time. Note that there isn't really a call to memcpy, the compiler inlines it as 2 int copies which is an insignificant overhead (even true with -O0). --- lib/raidfile/RaidFileWrite.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'lib/raidfile') diff --git a/lib/raidfile/RaidFileWrite.cpp b/lib/raidfile/RaidFileWrite.cpp index c9963f05..285dbbc3 100644 --- a/lib/raidfile/RaidFileWrite.cpp +++ b/lib/raidfile/RaidFileWrite.cpp @@ -462,10 +462,8 @@ void RaidFileWrite::TransformToRaidStorage() ASSERT(sizeof(RaidFileRead::FileSizeType) >= sizeof(off_t)); int sizePos = (blockSize/sizeof(unsigned int)) - 2; RaidFileRead::FileSizeType sw = box_hton64(writeFileStat.st_size); -#ifdef HAVE_PPC_WORKAROUND - static RaidFileRead::FileSizeType ppcWorkaround = sw; -#endif - unsigned int *psize = (unsigned int *)(&sw); + unsigned int psize[2]; + ::memcpy(psize, &sw, sizeof(RaidFileRead::FileSizeType)); pparity[sizePos+0] = pstripe1[sizePos+0] ^ psize[0]; pparity[sizePos+1] = pstripe1[sizePos+1] ^ psize[1]; } -- cgit v1.2.3