diff options
author | Eric Wong <normalperson@yhbt.net> | 2006-07-30 03:43:38 +0000 |
---|---|---|
committer | Eric Wong <normalperson@yhbt.net> | 2006-07-30 03:43:38 +0000 |
commit | 4cf5d04ca15bc28ee4636d1dccb858763513e571 (patch) | |
tree | 7898902e786f5763643a513ea96bcada8a2559cc /src/dbUtils.h | |
parent | 4d5b8509eb46cff40890b781018669233a79e414 (diff) | |
download | mpd-4cf5d04ca15bc28ee4636d1dccb858763513e571.tar.gz mpd-4cf5d04ca15bc28ee4636d1dccb858763513e571.tar.xz mpd-4cf5d04ca15bc28ee4636d1dccb858763513e571.zip |
interface/connection malloc reductions from mpd-ke
This patch massively reduces the amount of heap allocations at
the interface/command layer. Most commands with minimal output
should not allocate memory from the heap at all. Things like
repeatedly polling status, currentsong, and volume changes
should be faster as a result, and more importantly, not a source
of memory fragmentation.
These changes should be safe in that there's no way for a
remote-client to corrupt memory or otherwise do bad stuff to
MPD, but an extra set of eyes to review would be good. Of
course there's never any warranty :)
No longer do we use FILE * structures in the interface, which means
we don't have to allocate any new memory for most connections.
Now, before you go on about losing the buffering that FILE *
+implies+, remember that myfprintf() never took advantage of
any of the stdio buffering features.
To reduce the diff and make bugs easier to spot in the diff,
I've kept myfprintf in places where we write to files (and not
network interfaces). Expect myfprintf to go away entirely soon
(we'll use fprintf for writing regular files).
git-svn-id: https://svn.musicpd.org/mpd/trunk@4483 09075e82-0dd4-0310-85a5-a0d7c8717e4f
Diffstat (limited to 'src/dbUtils.h')
-rw-r--r-- | src/dbUtils.h | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/dbUtils.h b/src/dbUtils.h index e0d306fd0..7dd63a168 100644 --- a/src/dbUtils.h +++ b/src/dbUtils.h @@ -43,22 +43,22 @@ void freeLocateTagItemArray(int count, LocateTagItem * array); void freeLocateTagItem(LocateTagItem * item); -int printAllIn(FILE * fp, char *name); +int printAllIn(int fd, char *name); -int addAllIn(FILE * fp, char *name); +int addAllIn(int fd, char *name); -int printInfoForAllIn(FILE * fp, char *name); +int printInfoForAllIn(int fd, char *name); -int searchForSongsIn(FILE * fp, char *name, int numItems, +int searchForSongsIn(int fd, char *name, int numItems, LocateTagItem * items); -int findSongsIn(FILE * fp, char *name, int numItems, LocateTagItem * items); +int findSongsIn(int fd, char *name, int numItems, LocateTagItem * items); -int countSongsIn(FILE * fp, char *name); +int countSongsIn(int fd, char *name); -unsigned long sumSongTimesIn(FILE * fp, char *name); +unsigned long sumSongTimesIn(int fd, char *name); -int listAllUniqueTags(FILE * fp, int type, int numConditiionals, +int listAllUniqueTags(int fd, int type, int numConditiionals, LocateTagItem * conditionals); void printSavedMemoryFromFilenames(); |