| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
For accessing the child of a directory, one needs X_OK on the
directory.
|
|
|
|
|
|
| |
MPD doesn't have child processes anymore, and thus we're not expecting
to receive SIGCHLD very often. Since hard disk access isn't
interrupted by signals anyway, we don't need those excessive checks.
|
| |
|
| |
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
Check ferror() instead of the fprintf() return value.
|
|
|
|
|
|
| |
Store a list of supported tag items in the database. When loading a
database which does not have a matching list, we must rescan in order
to get the missing information.
|
|
|
|
|
| |
Changed songvec_load() to song_load(). Added start and end markers
for each song. Removed the "key" line, it's redundant.
|
| |
|
| |
|
|
|
|
| |
The "music_root" global variable is allocated by db_init().
|
|
|
|
|
|
|
| |
Use a single GString buffer object in all functions loading the
database. Enlarge it automatically for long lines. This eliminates
the maximum line length for tag values. There is still an upper limit
of 512 kB to prevent denial of service, but that's reasonable I guess.
|
|
|
|
|
|
|
| |
When the filesystem_charset is changed in mpd.conf, MPD should discard
the old database. In this error branch, MPD did not fill the GError
object properly, and logged a warning message instead, which caused a
segmentation fault.
|
|
|
|
| |
Moved code from db_get_song().
|
|
|
|
|
| |
Renamed directory_get_directory() to directory_lookup_directory().
Added API documentation.
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
Do error reporting with GLib's GError library in this library, too.
|
|
|
|
| |
Do error reporting with GLib's GError library.
|
|
|
|
| |
Renamed a bunch of variables.
|
|
|
|
| |
Moved some of them to to directory_save.c, and others to database.c.
|
|
|
|
|
|
| |
When you change the filesystem charset, discard the old database file
and create a new one. The old database file will most likely contain
stale or invalid information.
|
|
|
|
| |
http://xkcd.com/292/
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
Without a music_directory, MPD is an excellent streaming client.
|
|
|
|
| |
Don't include conf.h in database.c.
|
|
|
|
| |
Don't return a writable pointer.
|
|
|
|
| |
Renamed functions, types, variables.
|
|
|
|
| |
Make db_load(), db_save() and db_check() return bool instead of int.
|
|
|
|
|
| |
For updating the database, directory_update_init() should be called
explicitly.
|
| |
|
|
|
|
|
| |
Start the daemon after --create-db. This makes --create-db a flag
which discards the old database and starts with a fresh one.
|
| |
|
|
|
|
|
|
|
|
|
| |
Make the event_pipe (formerly main_notify) send/receive a set of
events, with a callback for each one.
The default event PIPE_EVENT_SIGNAL does not have a callback. It
is still there for waking up the main thread, when it is waiting for
the player thread.
|
|
|
|
| |
Continuing the previous patch.
|
|
|
|
|
|
| |
We are going to migrate away from the concept of notifying the main
thread. There should be events sent to it instead. This patch starts
a series to implement that.
|
| |
|
|
|
|
|
| |
Only include headers which are really needed. os_compat.h aimed to
make MPD easily portable, but was never actually made portable.
|
|
|
|
| |
Replaced myFgets() with fgets() + g_strchomp().
|
|
|
|
| |
Rename variables and functions.
|
|
|
|
| |
Remove duplicated code from MPD.
|
|
|
|
| |
Again, GLib's version is more robust than ours.
|
|
|
|
| |
Since the return value cannot be -1 anymore, we can make it unsigned.
|
|
|
|
|
|
|
| |
The documentation for directory_update_init() was incorrect: a job ID
must be positive, not non-negative. If the update queue is full and
no job was created, it makes more sense to return 0 instead of -1,
because it is more consistent with the return value of isUpdatingDB().
|
|
|
|
|
|
|
|
| |
If the user requests database update during startup, call
directory_update_init(). This should be changed to fully asynchronous
update later.
For this to work, main_notify has to be initialized before db_init().
|
|
|
|
| |
Search'n'replace typo..
|
|
|
|
|
|
| |
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.
|
|
|
|
| |
Use sizeof(buffer) instead.
|
|
|
|
| |
Removed a superfluous closure.
|
|
|
|
|
| |
For the root directory, let's set path to an empty string. This saves
a few checks.
|
|
|
|
| |
CamelCase is ugly, rename the functions.
|