Fix some warnings given by clang, see issue 1718

This commit is contained in:
Philip Rinn
2013-02-12 18:23:51 +01:00
parent 153f4cb9d1
commit cbd20ac033
6 changed files with 9 additions and 7 deletions

View File

@@ -251,9 +251,9 @@ dfInfo *DFManager::addFileInfo(const Glib::ustring &filename ,bool pool )
{
Glib::RefPtr<Gio::File> file = Gio::File::create_for_path(filename);
if (!file )
return false;
return 0;
if( !file->query_exists())
return false;
return 0;
Glib::RefPtr<Gio::FileInfo> info = safe_query_file_info(file);
if (info && info->get_file_type() != Gio::FILE_TYPE_DIRECTORY && (!info->is_hidden() || !options.fbShowHidden)) {
size_t lastdot = info->get_name().find_last_of ('.');

View File

@@ -209,9 +209,9 @@ ffInfo *FFManager::addFileInfo(const Glib::ustring &filename, bool pool )
{
Glib::RefPtr<Gio::File> file = Gio::File::create_for_path(filename);
if (!file )
return false;
return 0;
if( !file->query_exists())
return false;
return 0;
Glib::RefPtr<Gio::FileInfo> info = safe_query_file_info(file);
if (info && info->get_file_type() != Gio::FILE_TYPE_DIRECTORY && (!info->is_hidden() || !options.fbShowHidden)) {
size_t lastdot = info->get_name().find_last_of ('.');

View File

@@ -558,6 +558,7 @@ Glib::ustring BatchQueue::autoCompleteFileName (const Glib::ustring& fileName, c
// separate filename and the path to the destination directory
Glib::ustring dstdir = Glib::path_get_dirname (fileName);
Glib::ustring dstfname = Glib::path_get_basename (fileName);
Glib::ustring fname;
// create directory, if does not exist
if (safe_g_mkdir_with_parents (dstdir, 0755) )
@@ -568,7 +569,6 @@ Glib::ustring BatchQueue::autoCompleteFileName (const Glib::ustring& fileName, c
bool inOverwriteMode=options.overwriteOutputFile;
for (int tries=0; tries<100; tries++) {
Glib::ustring fname;
if (tries==0)
fname = Glib::ustring::compose ("%1.%2", Glib::build_filename (dstdir, dstfname), format);
else
@@ -587,6 +587,7 @@ Glib::ustring BatchQueue::autoCompleteFileName (const Glib::ustring& fileName, c
return fname;
}
}
return "";
}
int setProgressUI (void* p) {

View File

@@ -27,7 +27,7 @@
class DFProvider {
public:
virtual rtengine::RawImage* getDF() {}
virtual rtengine::RawImage* getDF() = 0;
// add other info here
};

View File

@@ -1533,6 +1533,7 @@ bool FileCatalog::Query_key_pressed (GdkEventKey *event){
// Clear Query if the Escape character is pressed within it
FileCatalog::buttonQueryClearPressed ();
return true;
default: return false;
}
}

View File

@@ -28,7 +28,7 @@
class FFProvider {
public:
virtual rtengine::RawImage* getFF() {}
virtual rtengine::RawImage* getFF() = 0;
// add other info here
};