diff options
author | Guillaume Poussel <gpoussel@gmail.com> | 2011-01-04 22:40:36 +0100 |
---|---|---|
committer | David Tardon <dtardon@redhat.com> | 2011-01-05 09:02:33 +0100 |
commit | ed26218fdf5b0ba5a9a6c7611a5b787186185b8d (patch) | |
tree | a407b34648961b1e122faf52665975d056ca9a03 | |
parent | d7d7b15e9434301a789c386bcc4aa0e87d690e4b (diff) |
Removing few cppchecks warnings.
-rw-r--r-- | guw/guw.cc | 2 | ||||
-rw-r--r-- | soltools/checkdll/checkdll.c | 3 | ||||
-rw-r--r-- | soltools/ldump/ldump.cxx | 1 | ||||
-rw-r--r-- | soltools/mkdepend/collectdircontent.cxx | 1 | ||||
-rw-r--r-- | soltools/testSHL/inc/tstMgr.hxx | 2 | ||||
-rw-r--r-- | soltools/testSHL/util/tlog.cxx | 1 |
6 files changed, 5 insertions, 5 deletions
diff --git a/guw/guw.cc b/guw/guw.cc index ef6e5eb93f9b..a3a2f86fe69f 100644 --- a/guw/guw.cc +++ b/guw/guw.cc @@ -290,7 +290,7 @@ void init_ignorepara(string fullcommand) { bool is_ignorepara(const string ¶) { for( vector<string>::iterator it = ignorepara_vec.begin(); - it != ignorepara_vec.end(); it++ ) { + it != ignorepara_vec.end(); ++it ) { if ( para.find(*it) != string::npos ) { if ( debug ) cerr << "Found execption para: " << para << endl; diff --git a/soltools/checkdll/checkdll.c b/soltools/checkdll/checkdll.c index f1bc60ba9151..a300e0b69d67 100644 --- a/soltools/checkdll/checkdll.c +++ b/soltools/checkdll/checkdll.c @@ -51,7 +51,6 @@ void usage() int main(int argc, char *argv[]) { - int rc; int silent=0; void *phandle; char *(*pfun)(void); @@ -66,7 +65,7 @@ int main(int argc, char *argv[]) ++argv, --argc; } - if ( (rc = access( argv[1], R_OK )) == -1 ) { + if ( access( argv[1], R_OK ) == -1 ) { fprintf(stderr, "%s: ERROR: %s: %s\n", pprog_name, argv[1], strerror(errno)); return 2; diff --git a/soltools/ldump/ldump.cxx b/soltools/ldump/ldump.cxx index 0096adbb9738..cbb228e82351 100644 --- a/soltools/ldump/ldump.cxx +++ b/soltools/ldump/ldump.cxx @@ -755,6 +755,7 @@ main( int argc, char **argv ) pDump->PrintDefFile(); pDump->PrintDataBase(); delete pDump; + delete [] pLibName; return 0; } diff --git a/soltools/mkdepend/collectdircontent.cxx b/soltools/mkdepend/collectdircontent.cxx index ca73428bb922..60654bd41ce6 100644 --- a/soltools/mkdepend/collectdircontent.cxx +++ b/soltools/mkdepend/collectdircontent.cxx @@ -42,6 +42,7 @@ void IncludesCollection::add_to_collection(const string& dirPath) { while ((pent = readdir(pdir))) { dirContent.insert(pent->d_name); }; + closedir(pdir); #endif // defined( WNT ) allIncludes.insert(EntriesPair(dirPath, dirContent)); }; diff --git a/soltools/testSHL/inc/tstMgr.hxx b/soltools/testSHL/inc/tstMgr.hxx index cf59edb68f46..cbb8474b68af 100644 --- a/soltools/testSHL/inc/tstMgr.hxx +++ b/soltools/testSHL/inc/tstMgr.hxx @@ -50,7 +50,7 @@ class tstMgr { // </private_methods> public: - + tstMgr() : pImpl(0) {} // <dtor> ~tstMgr(){ cleanup(); diff --git a/soltools/testSHL/util/tlog.cxx b/soltools/testSHL/util/tlog.cxx index f3efa82e183b..9f2e86d4cc8a 100644 --- a/soltools/testSHL/util/tlog.cxx +++ b/soltools/testSHL/util/tlog.cxx @@ -93,7 +93,6 @@ void tLog::initialize( const ::rtl::OString& name ) { } sal_uInt64 uBytes=0; sal_uInt32 len = ln( buf ); - const sal_Char* ptr = buf; if ( v ) { fprintf( stderr, "%s", buf ); |