Commit 68977af6 authored by Max Kellermann's avatar Max Kellermann

directory: eliminate CamelCase

CamelCase is ugly, rename the functions.
parent bb8a9533
...@@ -39,7 +39,7 @@ static time_t directory_dbModTime; ...@@ -39,7 +39,7 @@ static time_t directory_dbModTime;
void void
db_init(void) db_init(void)
{ {
music_root = newDirectory(NULL, NULL); music_root = directory_new(NULL, NULL);
updateDirectory(music_root); updateDirectory(music_root);
stats.numberOfSongs = countSongsIn(NULL); stats.numberOfSongs = countSongsIn(NULL);
stats.dbPlayTime = sumSongTimesIn(NULL); stats.dbPlayTime = sumSongTimesIn(NULL);
...@@ -48,7 +48,7 @@ db_init(void) ...@@ -48,7 +48,7 @@ db_init(void)
void void
db_finish(void) db_finish(void)
{ {
freeDirectory(music_root); directory_free(music_root);
} }
struct directory * struct directory *
...@@ -65,7 +65,7 @@ db_get_directory(const char *name) ...@@ -65,7 +65,7 @@ db_get_directory(const char *name)
if (name == NULL) if (name == NULL)
return music_root; return music_root;
return getSubDirectory(music_root, name); return directory_get_directory(music_root, name);
} }
struct song * struct song *
...@@ -113,8 +113,7 @@ db_walk(const char *name, ...@@ -113,8 +113,7 @@ db_walk(const char *name,
return -1; return -1;
} }
return traverseAllInSubDirectory(directory, forEachSong, forEachDir, return directory_walk(directory, forEachSong, forEachDir, data);
data);
} }
static char * static char *
...@@ -197,11 +196,11 @@ db_save(void) ...@@ -197,11 +196,11 @@ db_save(void)
struct stat st; struct stat st;
DEBUG("removing empty directories from DB\n"); DEBUG("removing empty directories from DB\n");
deleteEmptyDirectoriesInDirectory(music_root); directory_prune_empty(music_root);
DEBUG("sorting DB\n"); DEBUG("sorting DB\n");
sortDirectory(music_root); directory_sort(music_root);
DEBUG("writing DB\n"); DEBUG("writing DB\n");
...@@ -218,7 +217,7 @@ db_save(void) ...@@ -218,7 +217,7 @@ db_save(void)
fprintf(fp, "%s%s\n", DIRECTORY_FS_CHARSET, getFsCharset()); fprintf(fp, "%s%s\n", DIRECTORY_FS_CHARSET, getFsCharset());
fprintf(fp, "%s\n", DIRECTORY_INFO_END); fprintf(fp, "%s\n", DIRECTORY_INFO_END);
if (writeDirectoryInfo(fp, music_root) < 0) { if (directory_save(fp, music_root) < 0) {
ERROR("Failed to write to database file: %s\n", ERROR("Failed to write to database file: %s\n",
strerror(errno)); strerror(errno));
while (fclose(fp) && errno == EINTR); while (fclose(fp) && errno == EINTR);
...@@ -241,7 +240,7 @@ db_load(void) ...@@ -241,7 +240,7 @@ db_load(void)
struct stat st; struct stat st;
if (!music_root) if (!music_root)
music_root = newDirectory(NULL, NULL); music_root = directory_new(NULL, NULL);
while (!(fp = fopen(dbFile, "r")) && errno == EINTR) ; while (!(fp = fopen(dbFile, "r")) && errno == EINTR) ;
if (fp == NULL) { if (fp == NULL) {
ERROR("unable to open db file \"%s\": %s\n", ERROR("unable to open db file \"%s\": %s\n",
...@@ -302,7 +301,7 @@ db_load(void) ...@@ -302,7 +301,7 @@ db_load(void)
DEBUG("reading DB\n"); DEBUG("reading DB\n");
readDirectoryInfo(fp, music_root); directory_load(fp, music_root);
while (fclose(fp) && errno == EINTR) ; while (fclose(fp) && errno == EINTR) ;
stats.numberOfSongs = countSongsIn(NULL); stats.numberOfSongs = countSongsIn(NULL);
......
...@@ -62,7 +62,7 @@ printDirectoryInDirectory(struct directory *directory, void *data) ...@@ -62,7 +62,7 @@ printDirectoryInDirectory(struct directory *directory, void *data)
{ {
struct client *client = data; struct client *client = data;
if (directory->path) { if (directory->path) {
client_printf(client, "directory: %s\n", getDirectoryPath(directory)); client_printf(client, "directory: %s\n", directory_get_path(directory));
} }
return 0; return 0;
} }
...@@ -374,7 +374,7 @@ sumSavedFilenameMemoryInDirectory(struct directory *dir, void *data) ...@@ -374,7 +374,7 @@ sumSavedFilenameMemoryInDirectory(struct directory *dir, void *data)
if (!dir->path) if (!dir->path)
return 0; return 0;
*sum += (strlen(getDirectoryPath(dir)) + 1 *sum += (strlen(directory_get_path(dir)) + 1
- sizeof(struct directory *)) * dir->songs.nr; - sizeof(struct directory *)) * dir->songs.nr;
return 0; return 0;
......
...@@ -28,7 +28,7 @@ ...@@ -28,7 +28,7 @@
#include "dirvec.h" #include "dirvec.h"
struct directory * struct directory *
newDirectory(const char *dirname, struct directory * parent) directory_new(const char *dirname, struct directory *parent)
{ {
struct directory *directory; struct directory *directory;
...@@ -42,7 +42,7 @@ newDirectory(const char *dirname, struct directory * parent) ...@@ -42,7 +42,7 @@ newDirectory(const char *dirname, struct directory * parent)
} }
void void
freeDirectory(struct directory * directory) directory_free(struct directory *directory)
{ {
dirvec_destroy(&directory->children); dirvec_destroy(&directory->children);
songvec_destroy(&directory->songs); songvec_destroy(&directory->songs);
...@@ -50,17 +50,17 @@ freeDirectory(struct directory * directory) ...@@ -50,17 +50,17 @@ freeDirectory(struct directory * directory)
free(directory->path); free(directory->path);
free(directory); free(directory);
/* this resets last dir returned */ /* this resets last dir returned */
/*getDirectoryPath(NULL); */ /*directory_get_path(NULL); */
} }
void void
deleteEmptyDirectoriesInDirectory(struct directory *directory) directory_prune_empty(struct directory *directory)
{ {
int i; int i;
struct dirvec *dv = &directory->children; struct dirvec *dv = &directory->children;
for (i = dv->nr; --i >= 0; ) { for (i = dv->nr; --i >= 0; ) {
deleteEmptyDirectoriesInDirectory(dv->base[i]); directory_prune_empty(dv->base[i]);
if (directory_is_empty(dv->base[i])) if (directory_is_empty(dv->base[i]))
dirvec_delete(dv, dv->base[i]); dirvec_delete(dv, dv->base[i]);
} }
...@@ -69,7 +69,7 @@ deleteEmptyDirectoriesInDirectory(struct directory *directory) ...@@ -69,7 +69,7 @@ deleteEmptyDirectoriesInDirectory(struct directory *directory)
} }
struct directory * struct directory *
getSubDirectory(struct directory * directory, const char *name) directory_get_directory(struct directory *directory, const char *name)
{ {
struct directory *cur = directory; struct directory *cur = directory;
struct directory *found = NULL; struct directory *found = NULL;
...@@ -102,13 +102,13 @@ getSubDirectory(struct directory * directory, const char *name) ...@@ -102,13 +102,13 @@ getSubDirectory(struct directory * directory, const char *name)
} }
static int static int
printDirectoryList(struct client *client, const struct dirvec *dv) dirvec_print(struct client *client, const struct dirvec *dv)
{ {
size_t i; size_t i;
for (i = 0; i < dv->nr; ++i) { for (i = 0; i < dv->nr; ++i) {
client_printf(client, DIRECTORY_DIR "%s\n", client_printf(client, DIRECTORY_DIR "%s\n",
getDirectoryPath(dv->base[i])); directory_get_path(dv->base[i]));
} }
return 0; return 0;
...@@ -117,7 +117,7 @@ printDirectoryList(struct client *client, const struct dirvec *dv) ...@@ -117,7 +117,7 @@ printDirectoryList(struct client *client, const struct dirvec *dv)
int int
directory_print(struct client *client, const struct directory *directory) directory_print(struct client *client, const struct directory *directory)
{ {
printDirectoryList(client, &directory->children); dirvec_print(client, &directory->children);
songvec_print(client, &directory->songs); songvec_print(client, &directory->songs);
return 0; return 0;
...@@ -125,7 +125,7 @@ directory_print(struct client *client, const struct directory *directory) ...@@ -125,7 +125,7 @@ directory_print(struct client *client, const struct directory *directory)
/* TODO error checking */ /* TODO error checking */
int int
writeDirectoryInfo(FILE * fp, struct directory * directory) directory_save(FILE *fp, struct directory *directory)
{ {
struct dirvec *children = &directory->children; struct dirvec *children = &directory->children;
size_t i; size_t i;
...@@ -133,7 +133,7 @@ writeDirectoryInfo(FILE * fp, struct directory * directory) ...@@ -133,7 +133,7 @@ writeDirectoryInfo(FILE * fp, struct directory * directory)
if (directory->path) { if (directory->path) {
retv = fprintf(fp, "%s%s\n", DIRECTORY_BEGIN, retv = fprintf(fp, "%s%s\n", DIRECTORY_BEGIN,
getDirectoryPath(directory)); directory_get_path(directory));
if (retv < 0) if (retv < 0)
return -1; return -1;
} }
...@@ -145,7 +145,7 @@ writeDirectoryInfo(FILE * fp, struct directory * directory) ...@@ -145,7 +145,7 @@ writeDirectoryInfo(FILE * fp, struct directory * directory)
retv = fprintf(fp, DIRECTORY_DIR "%s\n", base); retv = fprintf(fp, DIRECTORY_DIR "%s\n", base);
if (retv < 0) if (retv < 0)
return -1; return -1;
if (writeDirectoryInfo(fp, cur) < 0) if (directory_save(fp, cur) < 0)
return -1; return -1;
} }
...@@ -153,13 +153,13 @@ writeDirectoryInfo(FILE * fp, struct directory * directory) ...@@ -153,13 +153,13 @@ writeDirectoryInfo(FILE * fp, struct directory * directory)
if (directory->path && if (directory->path &&
fprintf(fp, DIRECTORY_END "%s\n", fprintf(fp, DIRECTORY_END "%s\n",
getDirectoryPath(directory)) < 0) directory_get_path(directory)) < 0)
return -1; return -1;
return 0; return 0;
} }
void void
readDirectoryInfo(FILE * fp, struct directory * directory) directory_load(FILE *fp, struct directory *directory)
{ {
char buffer[MPD_PATH_MAX * 2]; char buffer[MPD_PATH_MAX * 2];
int bufferSize = MPD_PATH_MAX * 2; int bufferSize = MPD_PATH_MAX * 2;
...@@ -185,10 +185,10 @@ readDirectoryInfo(FILE * fp, struct directory * directory) ...@@ -185,10 +185,10 @@ readDirectoryInfo(FILE * fp, struct directory * directory)
if ((subdir = db_get_directory(name))) { if ((subdir = db_get_directory(name))) {
assert(subdir->parent == directory); assert(subdir->parent == directory);
} else { } else {
subdir = newDirectory(name, directory); subdir = directory_new(name, directory);
dirvec_add(&directory->children, subdir); dirvec_add(&directory->children, subdir);
} }
readDirectoryInfo(fp, subdir); directory_load(fp, subdir);
} else if (!prefixcmp(buffer, SONG_BEGIN)) { } else if (!prefixcmp(buffer, SONG_BEGIN)) {
readSongInfoIntoList(fp, &directory->songs, directory); readSongInfoIntoList(fp, &directory->songs, directory);
} else { } else {
...@@ -198,7 +198,7 @@ readDirectoryInfo(FILE * fp, struct directory * directory) ...@@ -198,7 +198,7 @@ readDirectoryInfo(FILE * fp, struct directory * directory)
} }
void void
sortDirectory(struct directory * directory) directory_sort(struct directory *directory)
{ {
int i; int i;
struct dirvec *dv = &directory->children; struct dirvec *dv = &directory->children;
...@@ -207,14 +207,14 @@ sortDirectory(struct directory * directory) ...@@ -207,14 +207,14 @@ sortDirectory(struct directory * directory)
songvec_sort(&directory->songs); songvec_sort(&directory->songs);
for (i = dv->nr; --i >= 0; ) for (i = dv->nr; --i >= 0; )
sortDirectory(dv->base[i]); directory_sort(dv->base[i]);
} }
int int
traverseAllInSubDirectory(struct directory * directory, directory_walk(struct directory *directory,
int (*forEachSong) (struct song *, void *), int (*forEachSong)(struct song *, void *),
int (*forEachDir) (struct directory *, void *), int (*forEachDir)(struct directory *, void *),
void *data) void *data)
{ {
struct dirvec *dv = &directory->children; struct dirvec *dv = &directory->children;
int err = 0; int err = 0;
...@@ -230,7 +230,7 @@ traverseAllInSubDirectory(struct directory * directory, ...@@ -230,7 +230,7 @@ traverseAllInSubDirectory(struct directory * directory,
} }
for (j = 0; err >= 0 && j < dv->nr; ++j) for (j = 0; err >= 0 && j < dv->nr; ++j)
err = traverseAllInSubDirectory(dv->base[j], forEachSong, err = directory_walk(dv->base[j], forEachSong,
forEachDir, data); forEachDir, data);
return err; return err;
......
...@@ -59,10 +59,10 @@ isRootDirectory(const char *name) ...@@ -59,10 +59,10 @@ isRootDirectory(const char *name)
} }
struct directory * struct directory *
newDirectory(const char *dirname, struct directory * parent); directory_new(const char *dirname, struct directory *parent);
void void
freeDirectory(struct directory * directory); directory_free(struct directory *directory);
static inline bool static inline bool
directory_is_empty(struct directory *directory) directory_is_empty(struct directory *directory)
...@@ -71,28 +71,29 @@ directory_is_empty(struct directory *directory) ...@@ -71,28 +71,29 @@ directory_is_empty(struct directory *directory)
} }
void void
deleteEmptyDirectoriesInDirectory(struct directory *directory); directory_prune_empty(struct directory *directory);
struct directory * struct directory *
getSubDirectory(struct directory *directory, const char *name); directory_get_directory(struct directory *directory, const char *name);
int int
directory_print(struct client *client, const struct directory *directory); directory_print(struct client *client, const struct directory *directory);
int int
writeDirectoryInfo(FILE *fp, struct directory *directory); directory_save(FILE *fp, struct directory *directory);
void void
readDirectoryInfo(FILE *fp, struct directory *directory); directory_load(FILE *fp, struct directory *directory);
void void
sortDirectory(struct directory * directory); directory_sort(struct directory *directory);
int int
traverseAllInSubDirectory(struct directory * directory, directory_walk(struct directory *directory,
int (*forEachSong) (struct song *, void *), int (*forEachSong)(struct song *, void *),
int (*forEachDir) (struct directory *, void *), int (*forEachDir)(struct directory *, void *),
void *data); void *data);
#define getDirectoryPath(dir) ((dir && dir->path) ? dir->path : "") #define directory_get_path(dir) ((dir && dir->path) ? dir->path : "")
#endif #endif
...@@ -39,7 +39,7 @@ int dirvec_delete(struct dirvec *dv, struct directory *del) ...@@ -39,7 +39,7 @@ int dirvec_delete(struct dirvec *dv, struct directory *del)
for (i = dv->nr; --i >= 0; ) { for (i = dv->nr; --i >= 0; ) {
if (dv->base[i] != del) if (dv->base[i] != del)
continue; continue;
/* we _don't_ call freeDirectory() here */ /* we _don't_ call directory_free() here */
if (!--dv->nr) { if (!--dv->nr) {
free(dv->base); free(dv->base);
dv->base = NULL; dv->base = NULL;
......
...@@ -128,7 +128,7 @@ song_get_url(struct song *song, char *path_max_tmp) ...@@ -128,7 +128,7 @@ song_get_url(struct song *song, char *path_max_tmp)
strcpy(path_max_tmp, song->url); strcpy(path_max_tmp, song->url);
else else
pfx_dir(path_max_tmp, song->url, strlen(song->url), pfx_dir(path_max_tmp, song->url, strlen(song->url),
getDirectoryPath(song->parent), directory_get_path(song->parent),
strlen(getDirectoryPath(song->parent))); strlen(directory_get_path(song->parent)));
return path_max_tmp; return path_max_tmp;
} }
...@@ -28,7 +28,7 @@ song_print_url(struct client *client, struct song *song) ...@@ -28,7 +28,7 @@ song_print_url(struct client *client, struct song *song)
{ {
if (song->parent && song->parent->path) { if (song->parent && song->parent->path) {
client_printf(client, "%s%s/%s\n", SONG_FILE, client_printf(client, "%s%s/%s\n", SONG_FILE,
getDirectoryPath(song->parent), song->url); directory_get_path(song->parent), song->url);
} else { } else {
client_printf(client, "%s%s\n", SONG_FILE, song->url); client_printf(client, "%s%s\n", SONG_FILE, song->url);
} }
......
...@@ -33,7 +33,7 @@ song_save_url(FILE *fp, struct song *song) ...@@ -33,7 +33,7 @@ song_save_url(FILE *fp, struct song *song)
{ {
if (song->parent != NULL && song->parent->path != NULL) if (song->parent != NULL && song->parent->path != NULL)
fprintf(fp, SONG_FILE "%s/%s\n", fprintf(fp, SONG_FILE "%s/%s\n",
getDirectoryPath(song->parent), song->url); directory_get_path(song->parent), song->url);
else else
fprintf(fp, SONG_FILE "%s\n", fprintf(fp, SONG_FILE "%s\n",
song->url); song->url);
......
...@@ -141,7 +141,7 @@ statDirectory(struct directory *dir) ...@@ -141,7 +141,7 @@ statDirectory(struct directory *dir)
{ {
struct stat st; struct stat st;
if (myStat(getDirectoryPath(dir), &st) < 0) if (myStat(directory_get_path(dir), &st) < 0)
return -1; return -1;
directory_set_stat(dir, &st); directory_set_stat(dir, &st);
...@@ -174,11 +174,11 @@ addSubDirectoryToDirectory(struct directory *directory, ...@@ -174,11 +174,11 @@ addSubDirectoryToDirectory(struct directory *directory,
if (inodeFoundInParent(directory, st->st_ino, st->st_dev)) if (inodeFoundInParent(directory, st->st_ino, st->st_dev))
return UPDATE_RETURN_NOUPDATE; return UPDATE_RETURN_NOUPDATE;
subDirectory = newDirectory(name, directory); subDirectory = directory_new(name, directory);
directory_set_stat(subDirectory, st); directory_set_stat(subDirectory, st);
if (updateDirectory(subDirectory) != UPDATE_RETURN_UPDATED) { if (updateDirectory(subDirectory) != UPDATE_RETURN_UPDATED) {
freeDirectory(subDirectory); directory_free(subDirectory);
return UPDATE_RETURN_NOUPDATE; return UPDATE_RETURN_NOUPDATE;
} }
...@@ -262,7 +262,7 @@ updateDirectory(struct directory *directory) ...@@ -262,7 +262,7 @@ updateDirectory(struct directory *directory)
{ {
bool was_empty = directory_is_empty(directory); bool was_empty = directory_is_empty(directory);
DIR *dir; DIR *dir;
const char *dirname = getDirectoryPath(directory); const char *dirname = directory_get_path(directory);
struct dirent *ent; struct dirent *ent;
char path_max_tmp[MPD_PATH_MAX]; char path_max_tmp[MPD_PATH_MAX];
enum update_return ret = UPDATE_RETURN_NOUPDATE; enum update_return ret = UPDATE_RETURN_NOUPDATE;
...@@ -336,7 +336,7 @@ addDirectoryPathToDB(const char *utf8path) ...@@ -336,7 +336,7 @@ addDirectoryPathToDB(const char *utf8path)
inodeFoundInParent(parentDirectory, st.st_ino, st.st_dev)) inodeFoundInParent(parentDirectory, st.st_ino, st.st_dev))
return NULL; return NULL;
else { else {
directory = newDirectory(utf8path, parentDirectory); directory = directory_new(utf8path, parentDirectory);
dirvec_add(&parentDirectory->children, directory); dirvec_add(&parentDirectory->children, directory);
} }
} }
...@@ -392,7 +392,7 @@ static enum update_return updatePath(const char *utf8path) ...@@ -392,7 +392,7 @@ static enum update_return updatePath(const char *utf8path)
ret = updateDirectory(directory); ret = updateDirectory(directory);
if (ret != UPDATE_RETURN_ERROR) { if (ret != UPDATE_RETURN_ERROR) {
free(path); free(path);
sortDirectory(directory); directory_sort(directory);
return ret; return ret;
} }
/* we don't want to delete the root directory */ /* we don't want to delete the root directory */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment