Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Warning fixes, IsEmpty->empty.

Browse Source 
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@30392 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Włodzimierz Skiba
2004-11-09 18:48:38 +00:00
parent 56eee37fc8
commit bb8a847812
2 changed files with 25 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/common/datetime.cpp
View File

@@ -500,7 +500,7 @@ static bool GetNumericToken(size_t len, const wxChar*& p, unsigned long *number)
break; break;
} }
return !s.IsEmpty() && s.ToULong(number); return !s.empty() && s.ToULong(number);
} }
// scans all alphabetic characters and returns the resulting string // scans all alphabetic characters and returns the resulting string
@@ -542,8 +542,8 @@ wxDateTime::Tm::Tm(const struct tm& tm, const TimeZone& tz)
mday = (wxDateTime::wxDateTime_t)tm.tm_mday; mday = (wxDateTime::wxDateTime_t)tm.tm_mday;
mon = (wxDateTime::Month)tm.tm_mon; mon = (wxDateTime::Month)tm.tm_mon;
year = 1900 + tm.tm_year; year = 1900 + tm.tm_year;
wday = tm.tm_wday; wday = (wxDateTime::wxDateTime_t)tm.tm_wday;
yday = tm.tm_yday; yday = (wxDateTime::wxDateTime_t)tm.tm_yday;
} }
bool wxDateTime::Tm::IsValid() const bool wxDateTime::Tm::IsValid() const
@@ -559,7 +559,7 @@ void wxDateTime::Tm::ComputeWeekDay()
// compute the week day from day/month/year: we use the dumbest algorithm // compute the week day from day/month/year: we use the dumbest algorithm
// possible: just compute our JDN and then use the (simple to derive) // possible: just compute our JDN and then use the (simple to derive)
// formula: weekday = (JDN + 1.5) % 7 // formula: weekday = (JDN + 1.5) % 7
wday = (wxDateTime::WeekDay)(GetTruncatedJDN(mday, mon, year) + 2) % 7; wday = (wxDateTime::wxDateTime_t)((wxDateTime::WeekDay)(GetTruncatedJDN(mday, mon, year) + 2) % 7);
} }
void wxDateTime::Tm::AddMonths(int monDiff) void wxDateTime::Tm::AddMonths(int monDiff)
@@ -2392,7 +2392,7 @@ wxString wxDateTime::Format(const wxChar *format, const TimeZone& tz) const
fmt += *p; fmt += *p;
} }
if ( !fmt.IsEmpty() ) if ( !fmt.empty() )
{ {
// we've only got the flags and width so far in fmt // we've only got the flags and width so far in fmt
fmt.Prepend(_T('%')); fmt.Prepend(_T('%'));
@@ -2964,7 +2964,7 @@ const wxChar *wxDateTime::ParseFormat(const wxChar *date,
wxString am, pm, token = GetAlphaToken(input); wxString am, pm, token = GetAlphaToken(input);
GetAmPmStrings(&am, &pm); GetAmPmStrings(&am, &pm);
if (am.IsEmpty() && pm.IsEmpty()) if (am.empty() && pm.empty())
return (wxChar *)NULL; // no am/pm strings defined return (wxChar *)NULL; // no am/pm strings defined
if ( token.CmpNoCase(pm) == 0 ) if ( token.CmpNoCase(pm) == 0 )
{ {

36
src/common/db.cpp
View File

@@ -1013,8 +1013,8 @@ bool wxDb::setConnectionOptions(void)
if (retcode != SQL_SUCCESS && retcode != SQL_SUCCESS_WITH_INFO) if (retcode != SQL_SUCCESS && retcode != SQL_SUCCESS_WITH_INFO)
return(DispAllErrors(henv, hdbc)); return(DispAllErrors(henv, hdbc));
retcode = SQLSetConnectOption(hdbc, SQL_AUTOCOMMIT, SQL_AUTOCOMMIT_OFF); /* retcode = */ SQLSetConnectOption(hdbc, SQL_AUTOCOMMIT, SQL_AUTOCOMMIT_OFF);
retcode = SQLSetConnectOption(hdbc, SQL_OPT_TRACE, SQL_OPT_TRACE_OFF); /* retcode = */ SQLSetConnectOption(hdbc, SQL_OPT_TRACE, SQL_OPT_TRACE_OFF);
// SQLSetConnectOption(hdbc, SQL_TXN_ISOLATION, SQL_TXN_READ_COMMITTED); // No dirty reads // SQLSetConnectOption(hdbc, SQL_TXN_ISOLATION, SQL_TXN_READ_COMMITTED); // No dirty reads
// By default, MS Sql Server closes cursors on commit and rollback. The following // By default, MS Sql Server closes cursors on commit and rollback. The following
@@ -1026,7 +1026,7 @@ bool wxDb::setConnectionOptions(void)
{ {
const long SQL_PRESERVE_CURSORS = 1204L; const long SQL_PRESERVE_CURSORS = 1204L;
const long SQL_PC_ON = 1L; const long SQL_PC_ON = 1L;
retcode = SQLSetConnectOption(hdbc, SQL_PRESERVE_CURSORS, SQL_PC_ON); /* retcode = */ SQLSetConnectOption(hdbc, SQL_PRESERVE_CURSORS, SQL_PC_ON);
} }
// Display the connection options to verify them // Display the connection options to verify them
@@ -2192,7 +2192,7 @@ bool wxDb::ExecSql(const wxString &pSqlStmt, wxDbColInf** columns, short& numcol
//fill in column information (name, datatype) //fill in column information (name, datatype)
for (colNum = 0; colNum < noCols; colNum++) for (colNum = 0; colNum < noCols; colNum++)
{ {
if (SQLColAttributes(hstmt, colNum+1, SQL_COLUMN_NAME, if (SQLColAttributes(hstmt, (UWORD)(colNum+1), SQL_COLUMN_NAME,
name, sizeof(name), name, sizeof(name),
&Sword, &Sdword) != SQL_SUCCESS) &Sword, &Sdword) != SQL_SUCCESS)
{ {
@@ -2203,7 +2203,7 @@ bool wxDb::ExecSql(const wxString &pSqlStmt, wxDbColInf** columns, short& numcol
wxStrncpy(pColInf[colNum].colName, name, DB_MAX_COLUMN_NAME_LEN); wxStrncpy(pColInf[colNum].colName, name, DB_MAX_COLUMN_NAME_LEN);
if (SQLColAttributes(hstmt, colNum+1, SQL_COLUMN_TYPE, if (SQLColAttributes(hstmt, (UWORD)(colNum+1), SQL_COLUMN_TYPE,
NULL, 0, &Sword, &Sdword) != SQL_SUCCESS) NULL, 0, &Sword, &Sdword) != SQL_SUCCESS)
{ {
DispAllErrors(henv, hdbc, hstmt); DispAllErrors(henv, hdbc, hstmt);
@@ -2240,7 +2240,7 @@ bool wxDb::ExecSql(const wxString &pSqlStmt, wxDbColInf** columns, short& numcol
#ifdef __WXDEBUG__ #ifdef __WXDEBUG__
default: default:
wxString errMsg; wxString errMsg;
errMsg.Printf(wxT("SQL Data type %d currently not supported by wxWidgets"), Sdword); errMsg.Printf(wxT("SQL Data type %ld currently not supported by wxWidgets"), (long)Sdword);
wxLogDebug(errMsg,wxT("ODBC DEBUG MESSAGE")); wxLogDebug(errMsg,wxT("ODBC DEBUG MESSAGE"));
#endif #endif
} }
@@ -2371,7 +2371,7 @@ int wxDb::GetKeyFields(const wxString &tableName, wxDbColInf* colInf, UWORD noCo
} // while } // while
tempStr.Trim(); // Get rid of any unneeded blanks tempStr.Trim(); // Get rid of any unneeded blanks
if (!tempStr.IsEmpty()) if (!tempStr.empty())
{ {
for (i=0; i<noCols; i++) for (i=0; i<noCols; i++)
{ // Find the Column name { // Find the Column name
@@ -2500,7 +2500,7 @@ wxDbColInf *wxDb::GetColumns(wxChar *tableName[], const wxChar *userID)
// MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we // MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we
// use the call below that leaves out the user name // use the call below that leaves out the user name
if (!UserID.IsEmpty() && if (!UserID.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsMS_SQL_SERVER) Dbms() != dbmsMS_SQL_SERVER)
@@ -2655,7 +2655,7 @@ wxDbColInf *wxDb::GetColumns(const wxString &tableName, UWORD *numCols, const wx
// MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we // MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we
// use the call below that leaves out the user name // use the call below that leaves out the user name
if (!UserID.IsEmpty() && if (!UserID.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsMS_SQL_SERVER) Dbms() != dbmsMS_SQL_SERVER)
@@ -2910,7 +2910,7 @@ wxDbColInf *wxDb::GetColumns(const wxString &tableName, int *numCols, const wxCh
// MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we // MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we
// use the call below that leaves out the user name // use the call below that leaves out the user name
if (!UserID.IsEmpty() && if (!UserID.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsMS_SQL_SERVER) Dbms() != dbmsMS_SQL_SERVER)
@@ -3149,7 +3149,7 @@ int wxDb::GetColumnCount(const wxString &tableName, const wxChar *userID)
// MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we // MySQL, SQLServer, and Access cannot accept a user name when looking up column names, so we
// use the call below that leaves out the user name // use the call below that leaves out the user name
if (!UserID.IsEmpty() && if (!UserID.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsMS_SQL_SERVER) Dbms() != dbmsMS_SQL_SERVER)
@@ -3242,7 +3242,7 @@ wxDbInf *wxDb::GetCatalog(const wxChar *userID)
SQLFreeStmt(hstmt, SQL_CLOSE); // Close if Open SQLFreeStmt(hstmt, SQL_CLOSE); // Close if Open
tblNameSave.Empty(); tblNameSave.Empty();
if (!UserID.IsEmpty() && if (!UserID.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsMS_SQL_SERVER) Dbms() != dbmsMS_SQL_SERVER)
@@ -3347,7 +3347,7 @@ bool wxDb::Catalog(const wxChar *userID, const wxString &fileName)
wxString UserID; wxString UserID;
convertUserID(userID,UserID); convertUserID(userID,UserID);
if (!UserID.IsEmpty() && if (!UserID.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsINTERBASE && Dbms() != dbmsINTERBASE &&
@@ -3478,7 +3478,7 @@ bool wxDb::TableExists(const wxString &tableName, const wxChar *userID, const wx
// Some databases cannot accept a user name when looking up table names, // Some databases cannot accept a user name when looking up table names,
// so we use the call below that leaves out the user name // so we use the call below that leaves out the user name
if (!UserID.IsEmpty() && if (!UserID.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsMS_SQL_SERVER && Dbms() != dbmsMS_SQL_SERVER &&
@@ -3548,7 +3548,7 @@ bool wxDb::TablePrivileges(const wxString &tableName, const wxString &priv, cons
// Some databases cannot accept a user name when looking up table names, // Some databases cannot accept a user name when looking up table names,
// so we use the call below that leaves out the user name // so we use the call below that leaves out the user name
if (!Schema.IsEmpty() && if (!Schema.empty() &&
Dbms() != dbmsMY_SQL && Dbms() != dbmsMY_SQL &&
Dbms() != dbmsACCESS && Dbms() != dbmsACCESS &&
Dbms() != dbmsMS_SQL_SERVER) Dbms() != dbmsMS_SQL_SERVER)
@@ -4292,9 +4292,11 @@ bool wxDbGetDataSource(HENV henv, wxChar *Dsn, SWORD DsnMaxLength, wxChar *DsDes
*/ */
{ {
SWORD cb1,cb2; SWORD cb1,cb2;
SWORD lengthDsn = (SWORD)(DsnMaxLength*sizeof(wxChar));
SWORD lengthDsDesc = (SWORD)(DsDescMaxLength*sizeof(wxChar));
if (SQLDataSources(henv, direction, (SQLTCHAR FAR *) Dsn, DsnMaxLength*sizeof(wxChar), &cb1, if (SQLDataSources(henv, direction, (SQLTCHAR FAR *) Dsn, lengthDsn, &cb1,
(SQLTCHAR FAR *) DsDesc, DsDescMaxLength*sizeof(wxChar), &cb2) == SQL_SUCCESS) (SQLTCHAR FAR *) DsDesc, lengthDsDesc, &cb2) == SQL_SUCCESS)
return true; return true;
else else
return false; return false;