diff options
author | Lennart Poettering <lennart@poettering.net> | 2018-04-05 18:00:39 +0200 |
---|---|---|
committer | Sven Eden <yamakuzure@gmx.net> | 2018-08-24 16:47:08 +0200 |
commit | 97ac04b502a8bbe11b8a58adcecaf7d7bc3f478e (patch) | |
tree | ab5b4656a95680096e669fca92bc90c0bb3e6e40 | |
parent | b628c4dded8d514451cee188245aec42aea0707e (diff) |
path-util: document a few other special cases for last_path_component()
-rw-r--r-- | src/basic/path-util.c | 15 | ||||
-rw-r--r-- | src/test/test-path-util.c | 4 |
2 files changed, 15 insertions, 4 deletions
diff --git a/src/basic/path-util.c b/src/basic/path-util.c index 4c5f984bd..addd39373 100644 --- a/src/basic/path-util.c +++ b/src/basic/path-util.c @@ -723,16 +723,23 @@ char* dirname_malloc(const char *path) { } const char *last_path_component(const char *path) { - /* Finds the last component of the path, preserving the - * optional trailing slash that signifies a directory. + + /* Finds the last component of the path, preserving the optional trailing slash that signifies a directory. + * * a/b/c → c * a/b/c/ → c/ + * x → x + * x/ → x/ + * /y → y + * /y/ → y/ * / → / * // → / * /foo/a → a * /foo/a/ → a/ - * This is different than basename, which returns "" when - * a trailing slash is present. + * + * Also, the empty string is mapped to itself. + * + * This is different than basename(), which returns "" when a trailing slash is present. */ unsigned l, k; diff --git a/src/test/test-path-util.c b/src/test/test-path-util.c index 057c06e8c..f7e96283f 100644 --- a/src/test/test-path-util.c +++ b/src/test/test-path-util.c @@ -401,6 +401,10 @@ static void test_last_path_component(void) { assert_se(streq(last_path_component("/foo/a"), "a")); assert_se(streq(last_path_component("/foo/a/"), "a/")); assert_se(streq(last_path_component(""), "")); + assert_se(streq(last_path_component("a"), "a")); + assert_se(streq(last_path_component("a/"), "a/")); + assert_se(streq(last_path_component("/a"), "a")); + assert_se(streq(last_path_component("/a/"), "a/")); } static void test_filename_is_valid(void) { |