cleanup GUID/ClsId/CLSID types

- rename GUID to SvGUID so we don't need an #ifdef WIN32
- drop ClsId struct, since it is used interchangeably with GUID and has
the same structure

Change-Id: Idf5c14c82a6861ef585fb57896a9b12cfe40374c

9 files changed, 112 insertions, 190 deletions

diff --git a/filter/source/msfilter/svdfppt.cxx b/filter/source/msfilter/svdfppt.cxx
index 197b8696ea84..0d99c9d57418 100644
--- a/filter/source/msfilter/svdfppt.cxx
+++ b/filter/source/msfilter/svdfppt.cxx
@@ -1822,9 +1822,7 @@ SdrObject* SdrPowerPointImport::ImportOLE( long nOLEId,
                     {
                         if ( xObjStor->GetClassName() == SvGlobalName() )
                         {
-                            ClsId aId( pObjStor->GetClassId() );
-                            xObjStor->SetClass( SvGlobalName( aId.n1, aId.n2, aId.n3, aId.n4, aId.n5, aId.n6, aId.n7, aId.n8, aId.n9, aId.n10, aId.n11 ),
-                                pObjStor->GetFormat(), pObjStor->GetUserName() );
+                            xObjStor->SetClass( SvGlobalName( pObjStor->GetClassId() ), pObjStor->GetFormat(), pObjStor->GetUserName() );
                         }
                         SotStorageStreamRef xSrcTst = xObjStor->OpenSotStream( "\1Ole" );
                         if ( xSrcTst.Is() )
diff --git a/include/sot/stg.hxx b/include/sot/stg.hxx
index b42ed4c7f590..2d8b1d1d1186 100644
--- a/include/sot/stg.hxx
+++ b/include/sot/stg.hxx
@@ -43,12 +43,7 @@ class StgIo;
 class StgDirEntry;
 class StgStrm;
 class SvGlobalName;
-struct ClsId
-{
-    sal_Int32   n1;
-    sal_Int16   n2, n3;
-    sal_uInt8   n4, n5, n6, n7, n8, n9, n10, n11;
-};
+typedef struct SvGUID ClsId;
 
 class SOT_DLLPUBLIC StorageBase : public SvRefBase
 {
diff --git a/include/tools/globname.hxx b/include/tools/globname.hxx
index 8bd996f779ff..4defc52f201f 100644
--- a/include/tools/globname.hxx
+++ b/include/tools/globname.hxx
@@ -24,9 +24,17 @@
 #include <tools/toolsdllapi.h>
 #include <com/sun/star/uno/Sequence.hxx>
 
+struct SvGUID
+{
+    sal_uInt32 Data1;
+    sal_uInt16 Data2;
+    sal_uInt16 Data3;
+    sal_uInt8  Data4[8];
+};
+
 struct ImpSvGlobalName
 {
-    sal_uInt8       szData[ 16 ];
+    struct SvGUID   szData;
     sal_uInt16      nRefCount;
 
     enum Empty { EMPTY };
@@ -41,14 +49,6 @@ struct ImpSvGlobalName
     bool        operator == ( const ImpSvGlobalName & rObj ) const;
 };
 
-#ifdef WNT
-struct _GUID;
-typedef struct _GUID GUID;
-#else
-struct GUID;
-#endif
-
-typedef GUID CLSID;
 class SvStream;
 
 class TOOLS_DLLPUBLIC SvGlobalName
@@ -93,9 +93,8 @@ public:
     bool          MakeId( const OUString & rId );
     OUString      GetHexName() const;
 
-                  SvGlobalName( const CLSID & rId );
-    const CLSID & GetCLSID() const { return *(CLSID *)pImp->szData; }
-    const sal_uInt8* GetBytes() const { return pImp->szData; }
+                  SvGlobalName( const SvGUID & rId );
+    const SvGUID& GetCLSID() const { return pImp->szData; }
 
     // platform independent representation of a "GlobalName"
     // maybe transported remotely
diff --git a/oox/source/ole/olehelper.cxx b/oox/source/ole/olehelper.cxx
index 1be5d6f24116..409eb4a8ebcc 100644
--- a/oox/source/ole/olehelper.cxx
+++ b/oox/source/ole/olehelper.cxx
@@ -259,20 +259,12 @@ sal_uInt32 OleHelper::encodeOleColor( sal_Int32 nRgbColor )
 
 void OleHelper::exportGuid( BinaryOutputStream& rOStr, const SvGlobalName& rId )
 {
-    const sal_uInt8* pBytes = rId.GetBytes();
-    sal_uInt32 a;
-    memcpy(&a, pBytes, sizeof(sal_uInt32));
-    rOStr<< a;
-
-    sal_uInt16 b;
-    memcpy(&b, pBytes+4, sizeof(sal_uInt16));
-    rOStr << b;
-
-    memcpy(&b, pBytes+6, sizeof(sal_uInt16));
-    rOStr << b;
-
-    rOStr.writeArray( (sal_uInt8 *)&pBytes[ 8 ], 8 );
+    rOStr << rId.GetCLSID().Data1;
+    rOStr << rId.GetCLSID().Data2;
+    rOStr << rId.GetCLSID().Data3;
+    rOStr.writeArray( &rId.GetCLSID().Data4, 8 );
 }
+
 OUString OleHelper::importGuid( BinaryInputStream& rInStrm )
 {
     OUStringBuffer aBuffer;
diff --git a/sfx2/source/doc/objitem.cxx b/sfx2/source/doc/objitem.cxx
index b5c8fa022633..86b9d48030fc 100644
--- a/sfx2/source/doc/objitem.cxx
+++ b/sfx2/source/doc/objitem.cxx
@@ -81,7 +81,7 @@ bool SfxObjectShellItem::PutValue( const com::sun::star::uno::Any& rVal, sal_uIn
                 xModel, ::com::sun::star::uno::UNO_QUERY );
             if ( xTunnel.is() )
             {
-                ::com::sun::star::uno::Sequence < sal_Int8 > aSeq( (sal_Int8*) SvGlobalName( SFX_GLOBAL_CLASSID ).GetBytes(), 16 );
+                ::com::sun::star::uno::Sequence < sal_Int8 > aSeq = SvGlobalName( SFX_GLOBAL_CLASSID ).GetByteSequence();
                 sal_Int64 nHandle = xTunnel->getSomething( aSeq );
                 if ( nHandle )
                 {
diff --git a/sot/source/sdstor/stg.cxx b/sot/source/sdstor/stg.cxx
index d6ae01ea7cea..84219df91e69 100644
--- a/sot/source/sdstor/stg.cxx
+++ b/sot/source/sdstor/stg.cxx
@@ -977,11 +977,11 @@ SvGlobalName Storage::GetClassName()
 {
     StgCompObjStream aCompObj( *this, false );
     if( aCompObj.Load() )
-        return SvGlobalName( (const CLSID&) aCompObj.GetClsId() );
+        return SvGlobalName( aCompObj.GetClsId() );
     pIo->ResetError();
 
     if ( pEntry )
-        return SvGlobalName( (const CLSID&) pEntry->aEntry.GetClassId() );
+        return SvGlobalName( pEntry->aEntry.GetClassId() );
 
     return SvGlobalName();
 }
@@ -1041,7 +1041,7 @@ const ClsId& Storage::GetClassId() const
     if ( pEntry )
         return pEntry->aEntry.GetClassId();
 
-    static ClsId aDummyId = {0,0,0,0,0,0,0,0,0,0,0};
+    static ClsId aDummyId = {0,0,0,{0,0,0,0,0,0,0,0}};
     return aDummyId;
 }
 
diff --git a/sot/source/sdstor/stgelem.cxx b/sot/source/sdstor/stgelem.cxx
index 1a50c1e20de8..8cf9239f5f97 100644
--- a/sot/source/sdstor/stgelem.cxx
+++ b/sot/source/sdstor/stgelem.cxx
@@ -37,34 +37,34 @@ static const sal_uInt8 cStgSignature[ 8 ] = { 0xD0,0xCF,0x11,0xE0,0xA1,0xB1,0x1A
 
 SvStream& ReadClsId( SvStream& r, ClsId& rId )
 {
-    r.ReadInt32( rId.n1 )
-     .ReadInt16( rId.n2 )
-     .ReadInt16( rId.n3 )
-     .ReadUChar( rId.n4 )
-     .ReadUChar( rId.n5 )
-     .ReadUChar( rId.n6 )
-     .ReadUChar( rId.n7 )
-     .ReadUChar( rId.n8 )
-     .ReadUChar( rId.n9 )
-     .ReadUChar( rId.n10 )
-     .ReadUChar( rId.n11 );
+    r.ReadUInt32( rId.Data1 )
+     .ReadUInt16( rId.Data2 )
+     .ReadUInt16( rId.Data3 )
+     .ReadUChar( rId.Data4[0] )
+     .ReadUChar( rId.Data4[1] )
+     .ReadUChar( rId.Data4[2] )
+     .ReadUChar( rId.Data4[3] )
+     .ReadUChar( rId.Data4[4] )
+     .ReadUChar( rId.Data4[5] )
+     .ReadUChar( rId.Data4[6] )
+     .ReadUChar( rId.Data4[7] );
     return r;
 }
 
 SvStream& WriteClsId( SvStream& r, const ClsId& rId )
 {
     return
-       r .WriteInt32( (sal_Int32) rId.n1 )
-         .WriteInt16( (sal_Int16) rId.n2 )
-         .WriteInt16( (sal_Int16) rId.n3 )
-         .WriteUChar( (sal_uInt8) rId.n4 )
-         .WriteUChar( (sal_uInt8) rId.n5 )
-         .WriteUChar( (sal_uInt8) rId.n6 )
-         .WriteUChar( (sal_uInt8) rId.n7 )
-         .WriteUChar( (sal_uInt8) rId.n8 )
-         .WriteUChar( (sal_uInt8) rId.n9 )
-         .WriteUChar( (sal_uInt8) rId.n10 )
-         .WriteUChar( (sal_uInt8) rId.n11 );
+       r .WriteUInt32( rId.Data1 )
+         .WriteUInt16( rId.Data2 )
+         .WriteUInt16( rId.Data3 )
+         .WriteUChar( rId.Data4[0] )
+         .WriteUChar( rId.Data4[1] )
+         .WriteUChar( rId.Data4[2] )
+         .WriteUChar( rId.Data4[3] )
+         .WriteUChar( rId.Data4[4] )
+         .WriteUChar( rId.Data4[5] )
+         .WriteUChar( rId.Data4[6] )
+         .WriteUChar( rId.Data4[7] );
 }
 
 ///////////////////////////// class StgHeader
diff --git a/sot/source/sdstor/ucbstorage.cxx b/sot/source/sdstor/ucbstorage.cxx
index e95a9b4fe007..c6a5e8b2c318 100644
--- a/sot/source/sdstor/ucbstorage.cxx
+++ b/sot/source/sdstor/ucbstorage.cxx
@@ -2454,7 +2454,7 @@ void UCBStorage::SetClass( const SvGlobalName & rClass, sal_uLong nOriginalClipF
 
 void UCBStorage::SetClassId( const ClsId& rClsId )
 {
-    pImp->m_aClassId = SvGlobalName( (const CLSID&) rClsId );
+    pImp->m_aClassId = SvGlobalName( rClsId );
     if ( pImp->m_aClassId == SvGlobalName() )
         return;
 
diff --git a/tools/source/ref/globname.cxx b/tools/source/ref/globname.cxx
index 3fe206497344..aaa8b3fc2d0e 100644
--- a/tools/source/ref/globname.cxx
+++ b/tools/source/ref/globname.cxx
@@ -31,18 +31,18 @@
 ImpSvGlobalName::ImpSvGlobalName( const ImpSvGlobalName & rObj )
 {
     nRefCount = 0;
-    memcpy( szData, rObj.szData, sizeof( szData ) );
+    szData = rObj.szData;
 }
 
 ImpSvGlobalName::ImpSvGlobalName( Empty )
 {
     nRefCount = 1;
-    memset( szData, 0, sizeof( szData ) );
+    memset( &szData, 0, sizeof( szData ) );
 }
 
 bool ImpSvGlobalName::operator == ( const ImpSvGlobalName & rObj ) const
 {
-    return !memcmp( szData, rObj.szData, sizeof( szData ) );
+    return !memcmp( &szData, &rObj.szData, sizeof( szData ) );
 }
 
 // SvGlobalName ----------------------------------------------------------------
@@ -55,22 +55,11 @@ SvGlobalName::SvGlobalName()
     pImp->nRefCount++;
 }
 
-#ifdef WNT
-struct _GUID
-#else
-struct GUID
-#endif
-{
-    sal_uInt32 Data1;
-    sal_uInt16 Data2;
-    sal_uInt16 Data3;
-    sal_uInt8  Data4[8];
-};
-SvGlobalName::SvGlobalName( const CLSID & rId )
+SvGlobalName::SvGlobalName( const SvGUID & rId )
 {
     pImp = new ImpSvGlobalName();
     pImp->nRefCount++;
-    memcpy( pImp->szData, &rId, sizeof( pImp->szData ) );
+    pImp->szData = rId;
 }
 
 SvGlobalName::SvGlobalName( sal_uInt32 n1, sal_uInt16 n2, sal_uInt16 n3,
@@ -79,18 +68,17 @@ SvGlobalName::SvGlobalName( sal_uInt32 n1, sal_uInt16 n2, sal_uInt16 n3,
 {
     pImp = new ImpSvGlobalName();
     pImp->nRefCount++;
-
-    memcpy(pImp->szData, &n1, sizeof(n1));
-    memcpy(pImp->szData+4, &n2, sizeof(n2));
-    memcpy(pImp->szData+6, &n3, sizeof(n3));
-    pImp->szData[ 8  ] = b8;
-    pImp->szData[ 9  ] = b9;
-    pImp->szData[ 10 ] = b10;
-    pImp->szData[ 11 ] = b11;
-    pImp->szData[ 12 ] = b12;
-    pImp->szData[ 13 ] = b13;
-    pImp->szData[ 14 ] = b14;
-    pImp->szData[ 15 ] = b15;
+    pImp->szData.Data1 = n1;
+    pImp->szData.Data2 = n2;
+    pImp->szData.Data3 = n3;
+    pImp->szData.Data4[0] = b8;
+    pImp->szData.Data4[1] = b9;
+    pImp->szData.Data4[2] = b10;
+    pImp->szData.Data4[3] = b11;
+    pImp->szData.Data4[4] = b12;
+    pImp->szData.Data4[5] = b13;
+    pImp->szData.Data4[6] = b14;
+    pImp->szData.Data4[7] = b15;
 }
 
 SvGlobalName::~SvGlobalName()
@@ -122,89 +110,50 @@ void SvGlobalName::NewImp()
 
 SvStream& WriteSvGlobalName( SvStream& rOStr, const SvGlobalName & rObj )
 {
-    sal_uInt32 a;
-    memcpy(&a, rObj.pImp->szData, sizeof(sal_uInt32));
-    rOStr.WriteUInt32( a );
-
-    sal_uInt16 b;
-    memcpy(&b, rObj.pImp->szData+4, sizeof(sal_uInt16));
-    rOStr.WriteUInt16( b );
-
-    memcpy(&b, rObj.pImp->szData+6, sizeof(sal_uInt16));
-    rOStr.WriteUInt16( b );
-
-    rOStr.Write( (sal_Char *)&rObj.pImp->szData[ 8 ], 8 );
+    rOStr.WriteUInt32( rObj.pImp->szData.Data1 );
+    rOStr.WriteUInt16( rObj.pImp->szData.Data2 );
+    rOStr.WriteUInt16( rObj.pImp->szData.Data3 );
+    rOStr.Write( (sal_Char *)&rObj.pImp->szData.Data4, 8 );
     return rOStr;
 }
 
 SvStream& operator >> ( SvStream& rStr, SvGlobalName & rObj )
 {
     rObj.NewImp(); // copy if necessary
-
-    sal_uInt32 a;
-    rStr.ReadUInt32( a );
-    memcpy(rObj.pImp->szData, &a, sizeof(sal_uInt32));
-
-    sal_uInt16 b;
-    rStr.ReadUInt16( b );
-    memcpy(rObj.pImp->szData+4, &b, sizeof(sal_uInt16));
-
-    rStr.ReadUInt16( b );
-    memcpy(rObj.pImp->szData+6, &b, sizeof(sal_uInt16));
-
-    rStr.Read( (sal_Char *)&rObj.pImp->szData[ 8 ], 8 );
+    rStr.ReadUInt32( rObj.pImp->szData.Data1 );
+    rStr.ReadUInt16( rObj.pImp->szData.Data2 );
+    rStr.ReadUInt16( rObj.pImp->szData.Data3 );
+    rStr.Read( (sal_Char *)&rObj.pImp->szData.Data4, 8 );
     return rStr;
 }
 
 
 bool SvGlobalName::operator < ( const SvGlobalName & rObj ) const
 {
-    int n = memcmp( pImp->szData +6, rObj.pImp->szData +6,
-                    sizeof( pImp->szData ) -6);
-    if( n < 0 )
+    if( pImp->szData.Data3 < rObj.pImp->szData.Data3 )
         return true;
-    else if( n > 0 )
+    else if( pImp->szData.Data3 > rObj.pImp->szData.Data3 )
         return false;
 
-    sal_uInt16 Data2_a;
-    memcpy(&Data2_a, pImp->szData+4, sizeof(sal_uInt16));
-
-    sal_uInt16 Data2_b;
-    memcpy(&Data2_b, rObj.pImp->szData+4, sizeof(sal_uInt16));
-
-    if( Data2_a < Data2_b )
+    if( pImp->szData.Data2 < rObj.pImp->szData.Data2 )
         return true;
-    else if( Data2_a == Data2_b )
-    {
-        sal_uInt32 Data1_a;
-        memcpy(&Data1_a, pImp->szData, sizeof(sal_uInt32));
-
-        sal_uInt32 Data1_b;
-        memcpy(&Data1_b, rObj.pImp->szData, sizeof(sal_uInt32));
-
-        return Data1_a  < Data1_b;
-    }
-    else
+    else if( pImp->szData.Data2 > rObj.pImp->szData.Data2 )
         return false;
 
+    return pImp->szData.Data1 < rObj.pImp->szData.Data1;
 }
 
 SvGlobalName & SvGlobalName::operator += ( sal_uInt32 n )
 {
     NewImp();
 
-    sal_uInt32 nOld;
-    memcpy(&nOld, pImp->szData, sizeof(sal_uInt32));
-    sal_uInt32 nNew = nOld + n;
-    memcpy(pImp->szData, &nNew, sizeof(sal_uInt32));
+    sal_uInt32 nOld = pImp->szData.Data1;
+    pImp->szData.Data1 += n;
 
-    if( nOld > nNew )
+    if( nOld > pImp->szData.Data1 )
     {
         // overflow
-        sal_uInt16 Data2;
-        memcpy(&Data2, pImp->szData + 4, sizeof(sal_uInt16));
-        ++Data2;
-        memcpy(pImp->szData + 4, &Data2, sizeof(sal_uInt16));
+        pImp->szData.Data2++;
     }
     return *this;
 }
@@ -217,7 +166,7 @@ bool SvGlobalName::operator == ( const SvGlobalName & rObj ) const
 void SvGlobalName::MakeFromMemory( void * pData )
 {
     NewImp();
-    memcpy( pImp->szData, pData, sizeof( pImp->szData ) );
+    memcpy( &pImp->szData, pData, sizeof( pImp->szData ) );
 }
 
 bool SvGlobalName::MakeId( const OUString & rIdStr )
@@ -289,10 +238,10 @@ bool SvGlobalName::MakeId( const OUString & rIdStr )
         }
 
         NewImp();
-        memcpy(&pImp->szData[0], &nFirst, sizeof(nFirst));
-        memcpy(&pImp->szData[4], &nSec, sizeof(nSec));
-        memcpy(&pImp->szData[6], &nThird, sizeof(nThird));
-        memcpy(&pImp->szData[ 8 ], szRemain, 8);
+        memcpy(&pImp->szData.Data1, &nFirst, sizeof(nFirst));
+        memcpy(&pImp->szData.Data2, &nSec, sizeof(nSec));
+        memcpy(&pImp->szData.Data3, &nThird, sizeof(nThird));
+        memcpy(&pImp->szData.Data4, szRemain, 8);
         return true;
     }
     return false;
@@ -303,29 +252,24 @@ OUString SvGlobalName::GetHexName() const
     OStringBuffer aHexBuffer;
 
     sal_Char buf[ 10 ];
-    sal_uInt32 Data1;
-    memcpy(&Data1, pImp->szData, sizeof(sal_uInt32));
-    sprintf( buf, "%8.8" SAL_PRIXUINT32, Data1 );
+    sprintf( buf, "%8.8" SAL_PRIXUINT32, pImp->szData.Data1 );
     aHexBuffer.append(buf);
     aHexBuffer.append('-');
-    sal_uInt16 i ;
-    for( i = 4; i < 8; i += 2 )
-    {
-        sal_uInt16 Data2;
-        memcpy(&Data2, pImp->szData+i, sizeof(sal_uInt16));
-        sprintf( buf, "%4.4X", Data2 );
-        aHexBuffer.append(buf);
-        aHexBuffer.append('-');
-    }
-    for( i = 8; i < 10; i++ )
+    sprintf( buf, "%4.4X", pImp->szData.Data2 );
+    aHexBuffer.append(buf);
+    aHexBuffer.append('-');
+    sprintf( buf, "%4.4X", pImp->szData.Data3 );
+    aHexBuffer.append(buf);
+    aHexBuffer.append('-');
+    for( int i = 0; i < 2; i++ )
     {
-        sprintf( buf, "%2.2x", pImp->szData[ i ] );
+        sprintf( buf, "%2.2x", pImp->szData.Data4[ i ] );
         aHexBuffer.append(buf);
     }
     aHexBuffer.append('-');
-    for( i = 10; i < 16; i++ )
+    for( int i = 2; i < 8; i++ )
     {
-        sprintf( buf, "%2.2x", pImp->szData[ i ] );
+        sprintf( buf, "%2.2x", pImp->szData.Data4[ i ] );
         aHexBuffer.append(buf);
     }
     return OStringToOUString(aHexBuffer.makeStringAndClear(), RTL_TEXTENCODING_ASCII_US);
@@ -337,28 +281,22 @@ com::sun::star::uno::Sequence < sal_Int8 > SvGlobalName::GetByteSequence() const
     // maybe transported remotely
     com::sun::star::uno::Sequence< sal_Int8 > aResult( 16 );
 
-    sal_uInt32 Data1;
-    memcpy(&Data1, pImp->szData, sizeof(sal_uInt32));
-    aResult[0] = (sal_Int8) (Data1 >> 24);
-    aResult[1] = (sal_Int8) ((Data1 << 8 ) >> 24);
-    aResult[2] = (sal_Int8) ((Data1 << 16 ) >> 24);
-    aResult[3] = (sal_Int8) ((Data1 << 24 ) >> 24);
-    sal_uInt16 Data2;
-    memcpy(&Data2, pImp->szData+4, sizeof(sal_uInt16));
-    aResult[4] = (sal_Int8) (Data2 >> 8);
-    aResult[5] = (sal_Int8) ((Data2 << 8 ) >> 8);
-    sal_uInt16 Data3;
-    memcpy(&Data3, pImp->szData+6, sizeof(sal_uInt16));
-    aResult[6] = (sal_Int8) (Data3 >> 8);
-    aResult[7] = (sal_Int8) ((Data3 << 8 ) >> 8);
-    aResult[8] = pImp->szData[ 8 ];
-    aResult[9] = pImp->szData[ 9 ];
-    aResult[10] = pImp->szData[ 10 ];
-    aResult[11] = pImp->szData[ 11 ];
-    aResult[12] = pImp->szData[ 12 ];
-    aResult[13] = pImp->szData[ 13 ];
-    aResult[14] = pImp->szData[ 14 ];
-    aResult[15] = pImp->szData[ 15 ];
+    aResult[ 0] = (sal_Int8) (pImp->szData.Data1 >> 24);
+    aResult[ 1] = (sal_Int8) ((pImp->szData.Data1 << 8 ) >> 24);
+    aResult[ 2] = (sal_Int8) ((pImp->szData.Data1 << 16 ) >> 24);
+    aResult[ 3] = (sal_Int8) ((pImp->szData.Data1 << 24 ) >> 24);
+    aResult[ 4] = (sal_Int8) (pImp->szData.Data2 >> 8);
+    aResult[ 5] = (sal_Int8) ((pImp->szData.Data2 << 8 ) >> 8);
+    aResult[ 6] = (sal_Int8) (pImp->szData.Data3 >> 8);
+    aResult[ 7] = (sal_Int8) ((pImp->szData.Data3 << 8 ) >> 8);
+    aResult[ 8] = pImp->szData.Data4[ 0 ];
+    aResult[ 9] = pImp->szData.Data4[ 1 ];
+    aResult[10] = pImp->szData.Data4[ 2 ];
+    aResult[11] = pImp->szData.Data4[ 3 ];
+    aResult[12] = pImp->szData.Data4[ 4 ];
+    aResult[13] = pImp->szData.Data4[ 5 ];
+    aResult[14] = pImp->szData.Data4[ 6 ];
+    aResult[15] = pImp->szData.Data4[ 7 ];
 
     return aResult;
 }
@@ -366,7 +304,7 @@ com::sun::star::uno::Sequence < sal_Int8 > SvGlobalName::GetByteSequence() const
 SvGlobalName::SvGlobalName( const com::sun::star::uno::Sequence < sal_Int8 >& aSeq )
 {
     // create SvGlobalName from a platform independent representation
-    GUID aResult;
+    SvGUID aResult;
     memset( &aResult, 0, sizeof( aResult ) );
     if ( aSeq.getLength() == 16 )
     {
@@ -379,7 +317,7 @@ SvGlobalName::SvGlobalName( const com::sun::star::uno::Sequence < sal_Int8 >& aS
 
     pImp = new ImpSvGlobalName();
     pImp->nRefCount++;
-    memcpy( pImp->szData, &aResult, sizeof( pImp->szData ) );
+    pImp->szData = aResult;
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */