Author: hdu
Date: Thu Apr 18 13:44:11 2013
New Revision: 1469324
URL: http://svn.apache.org/r1469324
Log:
WaE: fix suspicious assignments in if-conditions
Modified:
openoffice/trunk/main/dbaccess/source/ui/misc/UITools.cxx
Modified: openoffice/trunk/main/dbaccess/source/ui/misc/UITools.cxx
URL:
http://svn.apache.org/viewvc/openoffice/trunk/main/dbaccess/source/ui/misc/UITools.cxx?rev=1469324&r1=1469323&r2=1469324&view=diff
==============================================================================
--- openoffice/trunk/main/dbaccess/source/ui/misc/UITools.cxx (original)
+++ openoffice/trunk/main/dbaccess/source/ui/misc/UITools.cxx Thu Apr 18
13:44:11 2013
@@ -1576,54 +1576,65 @@ TOTypeInfoSP queryTypeInfoByType(sal_Int
switch(_nDataType)
{
case DataType::TINYINT:
- if( pTypeInfo =
queryTypeInfoByType(DataType::SMALLINT,_rTypeInfo))
+ pTypeInfo =
queryTypeInfoByType(DataType::SMALLINT,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
// run through
case DataType::SMALLINT:
- if( pTypeInfo =
queryTypeInfoByType(DataType::INTEGER,_rTypeInfo))
+ pTypeInfo =
queryTypeInfoByType(DataType::INTEGER,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
// run through
case DataType::INTEGER:
- if( pTypeInfo =
queryTypeInfoByType(DataType::FLOAT,_rTypeInfo))
+ pTypeInfo =
queryTypeInfoByType(DataType::FLOAT,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
// run through
case DataType::FLOAT:
- if( pTypeInfo =
queryTypeInfoByType(DataType::REAL,_rTypeInfo))
+ pTypeInfo =
queryTypeInfoByType(DataType::REAL,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
// run through
case DataType::DATE:
case DataType::TIME:
if( DataType::DATE == _nDataType || DataType::TIME ==
_nDataType )
{
- if( pTypeInfo =
queryTypeInfoByType(DataType::TIMESTAMP,_rTypeInfo))
+ pTypeInfo =
queryTypeInfoByType(DataType::TIMESTAMP,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
}
// run through
case DataType::TIMESTAMP:
case DataType::REAL:
case DataType::BIGINT:
- if ( pTypeInfo =
queryTypeInfoByType(DataType::DOUBLE,_rTypeInfo) )
+ pTypeInfo =
queryTypeInfoByType(DataType::DOUBLE,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
// run through
case DataType::DOUBLE:
- if ( pTypeInfo =
queryTypeInfoByType(DataType::NUMERIC,_rTypeInfo) )
+ pTypeInfo =
queryTypeInfoByType(DataType::NUMERIC,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
// run through
case DataType::NUMERIC:
- pTypeInfo =
queryTypeInfoByType(DataType::DECIMAL,_rTypeInfo);
+ pTypeInfo =
queryTypeInfoByType(DataType::DECIMAL,_rTypeInfo);
break;
case DataType::DECIMAL:
- if ( pTypeInfo =
queryTypeInfoByType(DataType::NUMERIC,_rTypeInfo) )
+ pTypeInfo =
queryTypeInfoByType(DataType::NUMERIC,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
- if ( pTypeInfo =
queryTypeInfoByType(DataType::DOUBLE,_rTypeInfo) )
+ pTypeInfo =
queryTypeInfoByType(DataType::DOUBLE,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
break;
case DataType::VARCHAR:
- if ( pTypeInfo =
queryTypeInfoByType(DataType::LONGVARCHAR,_rTypeInfo) )
+ pTypeInfo =
queryTypeInfoByType(DataType::LONGVARCHAR,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
break;
case DataType::LONGVARCHAR:
- if ( pTypeInfo =
queryTypeInfoByType(DataType::CLOB,_rTypeInfo) )
+ pTypeInfo =
queryTypeInfoByType(DataType::CLOB,_rTypeInfo);
+ if( pTypeInfo != NULL)
break;
break;
default:
