Is there a memory leakage in this simple code snippet

Quote:

---

Originally Posted by medievalelks

Here's a version that doesn't depend on strdup or the POSIX call:

Code:

---

std::string dirname(const std::string & path)
{
    std::string dir = path.substr(0, path.find_last_of("/"));

    return ((dir != path) ? dir : std::string("."));
}

---

---

Although that wouldn't work on a system that has different directory separators than '/', e.g a Windows system which uses : and \, or one of them systems that uses : only.

But the idea is good, and it's definitely simple.

--
Mats

Here's a more portable version:

Code:

---

std::string dirname(const std::string & path, const std::string & sep)
{
    std::string dir = path.substr(0, path.find_last_of(sep));

    return ((dir != path) ? dir : std::string("."));
}

---