The following code gives me an uninitialised warning on nCol. There is no possible default clause for this one. switch( nXIndex & 3 ) { case 0 : nCol = *pTmp >> 6; break; case 1 : nCol = ( *pTmp >> 4 ) & 0x03 ; break; case 2 : nCol = ( *pTmp >> 2 ) & 0x03; break; case 3 : nCol = ( *pTmp++ ) & 0x03; break; } -- Thanks KenF OpenOffice.org developer