Salastil/ncmpcpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

replace all string::find_last_of() with string::rfind()

Browse Source 
find_last_of was working properly only because it was searching
only for one character, string::rfind() is appropriate function
for this job.
This commit is contained in:
Andrzej Rybczak
2009-02-05 21:52:31 +01:00
parent e637d8f627
commit d68824f0f2
4 changed files with 13 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/browser.cpp
View File

@@ -51,7 +51,7 @@ namespace
bool hasSupportedExtension(const string &file)
{
unsigned int last_dot = file.find_last_of(".");
size_t last_dot = file.rfind(".");
if (last_dot > file.length())
return false;
@@ -138,7 +138,7 @@ void DisplayItem(const Item &item, void *, Menu<Item> *menu)
*menu << "[..]";
return;
}
size_t slash = item.name.find_last_of("/");
size_t slash = item.name.rfind("/");
*menu << "[" << (slash != string::npos ? item.name.substr(slash+1) : item.name) << "]";
return;
}
@@ -180,7 +180,7 @@ void GetDirectory(string dir, string subdir)
if (dir != "/")
{
Item parent;
size_t slash = dir.find_last_of("/");
size_t slash = dir.rfind("/");
parent.song = (Song *) 1; // in that way we assume that's really parent dir
parent.name = slash != string::npos ? dir.substr(0, slash) : "/";
parent.type = itDirectory;