aboutsummaryrefslogtreecommitdiffstats
path: root/src/directory.c (follow)
Commit message (Collapse)AuthorAgeFilesLines
* directory: require the caller to lock the db_mutexMax Kellermann2012-02-021-20/+15
| | | | | Reduce the number of lock/unlock cycles, and make database handling safer.
* directory: add new objects to the end of the linked listMax Kellermann2012-02-021-2/+2
| | | | | This was the cause of the reversed ordering when loading a database file.
* directory: fix deep path lookupMax Kellermann2012-01-291-16/+14
| | | | | This commit fixes a major regression in directory_lookup_directory(), which broke the deep lookup of directories.
* directory: mark unused parameterMax Kellermann2012-01-241-1/+2
|
* directory: replace songvec with doubly linked listMax Kellermann2012-01-241-13/+24
|
* directory: add function directory_get_song(), ...Max Kellermann2012-01-241-2/+32
| | | | Wrap songvec_find() and other songvec methods.
* directory: replace dirvec with doubly linked listMax Kellermann2012-01-241-22/+49
| | | | | Random access is not needed, and a linked list is easier to manage: we don't need to (re-)allocate the pointer array.
* directory: simplify constructors and clarify API documentationMax Kellermann2012-01-241-4/+37
| | | | | Pass only the "name" to a directory, instead of the full (relative) path.
* db_print: print absolute URI of playlist entriesMax Kellermann2011-10-081-1/+1
| | | | | | | The protocol mandates that playlist URIs are absolute (i.e. full URI relative to the music directory, not relative to the parent directory). This adds the parameter "directory" to the "playlist" visitor method.
* db_visitor: add method playlist()Max Kellermann2011-09-131-0/+8
|
* db_plugin: add method visit()Max Kellermann2011-09-131-3/+4
|
* directory: don't visit "self" in _walk()Max Kellermann2011-09-131-4/+4
| | | | Let the caller decide if the current directory should be visited.
* directory: don't require a "directory" visitorMax Kellermann2011-09-131-6/+6
| | | | | .. to recurse. A visitor may not have one, but still be interested in a recursive search.
* database: add struct db_visitorMax Kellermann2011-09-131-19/+25
| | | | Use this struct for db_walk().
* copyright year 2011Max Kellermann2011-01-291-1/+1
|
* directory: free playlist vector in directory_free()Max Kellermann2010-07-251-0/+2
|
* update: store playlist files in databaseMax Kellermann2010-07-211-0/+2
| | | | | Don't open the music directory for each "lsinfo" call. Get the list of playlist files from the memory database.
* Update copyright notices.Avuton Olrich2009-12-311-1/+1
|
* include config.h in all sourcesMax Kellermann2009-11-121-0/+1
| | | | | | After we've been hit by Large File Support problems several times in the past week (which only occur on 32 bit platforms, which I don't have), this is yet another attempt to fix the issue.
* directory: free empty directories after removing them (memleak)Max Kellermann2009-08-141-3/+8
| | | | | dirvec_delete() does not free the object, we have to call directory_free() afterwards.
* directory: added directory_lookup_song()Max Kellermann2009-04-011-0/+30
| | | | Moved code from db_get_song().
* directory: renamed directory_get_directory()Max Kellermann2009-04-011-4/+4
| | | | | Renamed directory_get_directory() to directory_lookup_directory(). Added API documentation.
* all: Update copyright header.Avuton Olrich2009-03-131-6/+7
| | | | | | | | This updates the copyright header to all be the same, which is pretty much an update of where to mail request for a copy of the GPL and the years of the MPD project. This also puts all committers under 'The Music Player Project' umbrella. These entries should go individually in the AUTHORS file, for consistancy.
* use g_free() instead of free()Max Kellermann2009-01-251-2/+2
| | | | | | On some platforms, g_free() must be used for memory allocated by GLib. This patch intends to correct a lot of occurrences, but is probably not complete.
* directory: use GLib instead of utils.hMax Kellermann2009-01-021-5/+6
|
* directory: directory_free() frees childrenMax Kellermann2008-10-311-0/+7
| | | | | directory_free() should free all of its children (subdirectories and songs). This way, db_finish() properly frees all allocated memory.
* path: replaced mpd_basename() with g_path_get_basename()Max Kellermann2008-10-201-1/+2
| | | | | | GLib's g_path_get_basename() is much more reliable than mpd_basename(). The latter could be tricked into an assertion failure.
* directory: added directory_get_name()Max Kellermann2008-10-131-0/+7
| | | | directory_get_name() returns the base name of the directory.
* directory: use mpd_sizeof_str_flex_array for path, tooEric Wong2008-10-131-6/+7
| | | | This way we avoid unnecessary heap allocations.
* directory: added inline wrappers for accessing childrenMax Kellermann2008-10-091-1/+1
| | | | | Some tiny utilities... wrappers like these may become helpful when we introduce locking.
* directory: moved dirvec struct declaration to dirvec.hMax Kellermann2008-10-091-1/+0
| | | | | No idea why it was created in directory.h, but it should be in dirvec.h.
* diretory: moved code to directory_save.c, directory_print.cMax Kellermann2008-10-091-103/+3
| | | | | | Remove clutter from directory.c. Everything which saves or loads to/from the hard disk goes to directory_save.c, and code which sends directory information to the client is moved into directory_print.c.
* directory: path must not be NULLMax Kellermann2008-10-081-7/+7
| | | | | For the root directory, let's set path to an empty string. This saves a few checks.
* directory: eliminate CamelCaseMax Kellermann2008-10-081-23/+23
| | | | CamelCase is ugly, rename the functions.
* database: renamed functions, "db_" prefix and no CamelCaseMax Kellermann2008-10-081-1/+1
| | | | Yet another CamelCase removal patch.
* directory: moved code to database.cMax Kellermann2008-10-081-308/+10
| | | | | | Taming the directory.c monster, part II: move the database management stuff to database. directory.c should only contain code which works on directory objects.
* directory: converted isRootDirectory() to an inline functionMax Kellermann2008-10-081-5/+5
| | | | | The function isRootDirectory() is tiny and can be converted to an inline function. Don't allow name==NULL.
* song: removed CamelCaseMax Kellermann2008-10-081-1/+1
| | | | CamelCase is ugly... rename all functions.
* song: converted typedef Song to struct songMax Kellermann2008-10-081-6/+7
| | | | Again, a data type which can be forward-declared.
* directory: converted typedef Directory to struct directoryMax Kellermann2008-10-081-28/+33
| | | | | The struct can be forward-declared by other headers, which relaxes the header dependencies.
* update: merged exploreDirectory() into updateDirectory()Max Kellermann2008-10-081-1/+1
| | | | | | exploreDirectory() duplicates some code in updateDirectory(). Merge both functions, and use directory_is_empty() to determine whether update or explore mode should be used.
* directory: added directory_is_empty()Max Kellermann2008-10-081-1/+1
| | | | | directory_is_empty() is a tiny inline function which determine if a directory has any child objects (sub directories or songs).
* directory: moved code to update.cMax Kellermann2008-10-081-575/+17
| | | | | | The source directory.c mixes several libraries: directory object management, database management and database update, resulting in a 1000+ line monster. Move the whole database update code to update.c.
* directory: fix return value in removeDeletedFromDirectoryEric Wong2008-10-071-1/+1
| | | | oops :x
* directory: serialize song deletes from playlist during updateEric Wong2008-10-071-3/+28
| | | | | | | | | | | | | | | | This makes the update code thread-safe and doesn't penalize the playlist code by complicating it with complicated and error-prone locks (and the associated overhead, not everybody has a thread-implementation as good as NPTL). The update task blocks during the delete; but the update task is a slow task anyways so we can block w/o people caring too much. This was also our only freeSong call site, so remove that function. Note that deleting entire directories is not fully thread-safe, yet; as their traversals are not yet locked.
* directory: use songvec_for_each for iteratorsEric Wong2008-10-071-28/+30
| | | | Get rid of songvec_write so we can enforce proper locking
* dbUtils/directory: traverseAllIn forEachSong returns -1 on errorEric Wong2008-10-071-12/+9
| | | | Being consistent with most UNIX functions...
* Assert if we don't have song or song->url setEric Wong2008-10-071-2/+2
| | | | song objects cannot exist without a path or URL
* song: stop storing song_typeEric Wong2008-10-061-1/+1
| | | | | | | We already know if a song is a URL or not based on whether it has parentDir defined or not. Hopefully one day in the future we can drop HTTP support from MPD entirely when an HTTP filesystem comes along and we can access streams via open(2).
* song: use flex arrays to store song->urlEric Wong2008-10-061-1/+1
| | | | | | Reduce the number of allocations we make, so there's less pressure on the allocator and less overhead to keep track of the allocations in.