Commit message (Collapse) | Author | Files | Lines | ||
---|---|---|---|---|---|
2014-01-21 | db/upnp: use iterator in Namei() | Max Kellermann | 1 | -4/+5 | |
2014-01-21 | db/upnp: move "objid" declaration down | Max Kellermann | 1 | -2/+2 | |
Don't allocate the object when it's not needed. | |||||
2014-01-21 | db/upnp: eliminate redundant Namei() parameter "oobjid" | Max Kellermann | 1 | -12/+7 | |
The caller should better obtain the object id from the returned UPnPDirObject instance. | |||||
2014-01-21 | db/upnp: remove redundant objid.empty() check from VisitServer() | Max Kellermann | 1 | -4/+0 | |
If Namei() succeeds, the object id is guaranteed to be set. | |||||
2014-01-21 | db/upnp: use std::move() to extract server name | Max Kellermann | 1 | -1/+1 | |
Reduce runtime overhead. | |||||
2014-01-21 | db/upnp: use vector::front() instead of [0] | Max Kellermann | 1 | -5/+5 | |
2014-01-21 | db/upnp: don't use stringToTokens() in ParseDuration() | Max Kellermann | 1 | -4/+20 | |
Reduce bloat. | |||||
2014-01-21 | db/upnp: don't construct redundant empty vector<string> in Visit() | Max Kellermann | 1 | -1/+1 | |
We know already that vpath is empty, let's use this instance. | |||||
2014-01-21 | db/upnp: simplify GetSong() | Max Kellermann | 1 | -20/+18 | |
2014-01-21 | LightSong: add attribute "real_uri" | Max Kellermann | 4 | -25/+30 | |
The UPnP database plugin can now show relative song URIs for remote songs. | |||||
2014-01-21 | DetachedSong: add attribute "real_uri" | Max Kellermann | 7 | -11/+62 | |
Prepare for UPnP songs that retain there database identity. | |||||
2014-01-21 | DetachedSong: remove parameter names from "defaulted" constructors | Max Kellermann | 1 | -2/+2 | |
Suppresses gcc warnings. | |||||
2014-01-21 | db/upnp: fix empty song URI | Max Kellermann | 1 | -3/+6 | |
Fall back to UPnPDirObject::url if there is no override. | |||||
2014-01-21 | DetachedSong: use "default" copy/move constructors | Max Kellermann | 1 | -10/+2 | |
Remove unnecessary explicit code. | |||||
2014-01-21 | pcm/SoxrResampler: Add configurable quality levels | Chase Geigle | 3 | -3/+73 | |
2014-01-21 | PlaylistSong: modify the given song object in-place | Max Kellermann | 4 | -74/+55 | |
Reduce bloat. | |||||
2014-01-21 | PlaylistSong: remove redundant "secure" check | Max Kellermann | 1 | -3/+3 | |
This has been verified already in the caller. | |||||
2014-01-20 | PlaylistSong: recurse after applying base_uri | Max Kellermann | 1 | -3/+3 | |
Allows applying map_to_relative_path() with base_uri. | |||||
2014-01-20 | PlaylistSong: remove the IsInDatabase() check | Max Kellermann | 1 | -4/+0 | |
No caller passes a database song to this function, but if he does, this check doesn't make sense. | |||||
2014-01-20 | PlaylistSong: don't map relative URI to absolute path | Max Kellermann | 1 | -18/+2 | |
This didn't make sense. | |||||
2014-01-20 | CommandLine: show the current git tag and commit id | Max Kellermann | 1 | -1/+5 | |
2014-01-20 | PlaylistSong: pass reference variables internally | Max Kellermann | 1 | -18/+17 | |
2014-01-20 | Queue: rename struct queue to Queue | Max Kellermann | 10 | -53/+53 | |
Works around a build failure on Solaris because annoyingly, Solaris reserves the name "queue". This rename was pending anyway. | |||||
2014-01-20 | Queue: make the constructor "explicit" | Max Kellermann | 1 | -1/+1 | |
2014-01-19 | Song: make the "parent" attribute mandatory | Max Kellermann | 6 | -21/+16 | |
The Song class is only used for database songs now. A Song without a Directory is not possible anymore. | |||||
2014-01-19 | Directory: use Song::Export() | Max Kellermann | 1 | -15/+1 | |
2014-01-19 | LightSong: new class to be used by DatabasePlugin callbacks | Max Kellermann | 42 | -223/+404 | |
Detach the Song class completely from the public API, only to be used by SimpleDatabase and the update thread. | |||||
2014-01-19 | db/proxy: simplify error handling in GetSong() | Max Kellermann | 1 | -11/+9 | |
Check mpd_response_finish() before using mpd_song. Don't skip this check even if the mpd_song is non-nullptr. | |||||
2014-01-19 | db/simple: make borrowed_song_count "mutable" | Max Kellermann | 2 | -3/+3 | |
Simpler to use than const_cast. | |||||
2014-01-19 | Tag: pack attributes tighter | Max Kellermann | 2 | -7/+7 | |
This saves another 3% memory. | |||||
2014-01-19 | Song: embed the Tag object statically into class Song | Max Kellermann | 14 | -47/+26 | |
Reduces overhead because we need to manage only one memory allocation. According to valgrind/massif, we save 7%. | |||||
2014-01-18 | Song: use the VarSize library | Max Kellermann | 2 | -13/+19 | |
2014-01-18 | db/upnp: move Tag from UPnPDirObject to Song | Max Kellermann | 1 | -11/+14 | |
The UPnPDirObject is a temporary object, we can move its contents. This reduces runtime overhead. | |||||
2014-01-18 | db/upnp/Discovery: eliminate attribute "last_seen" | Max Kellermann | 2 | -6/+4 | |
2014-01-18 | db/upnp/Discovery: use monotonic clock instead of time() | Max Kellermann | 2 | -10/+18 | |
2014-01-18 | db/upnp/Discovery: eliminate full copy in getServer() | Max Kellermann | 2 | -9/+17 | |
2014-01-18 | db/upnp: use getServer() in Visit() | Max Kellermann | 1 | -17/+9 | |
Reduce bloat. | |||||
2014-01-18 | db/upnp/Discovery: eliminate Error attribute | Max Kellermann | 3 | -51/+32 | |
Move code to method Start() and add Error& parameters to several methods. | |||||
2014-01-18 | db/upnp: remove unnecessary m_superdir nullptr checks | Max Kellermann | 1 | -3/+3 | |
2014-01-18 | db/upnp: pass ContentDirectoryService references | Max Kellermann | 1 | -27/+27 | |
2014-01-18 | db/upnp: cache m_path.back() | Max Kellermann | 2 | -16/+18 | |
2014-01-18 | db/upnp: add UpnpMakeAction() helper function | Max Kellermann | 2 | -30/+78 | |
2014-01-18 | db/upnp/ixmlwrap: getFirstElementValue() returns const char * | Max Kellermann | 5 | -27/+33 | |
Eliminate the std::string bloat. | |||||
2014-01-18 | db/upnp: merge duplicate nullptr check | Max Kellermann | 1 | -2/+2 | |
2014-01-18 | db/upnp: free IXML_Document *request manually | Max Kellermann | 1 | -38/+40 | |
2014-01-18 | db/upnp/Device: move code to method Parse() | Max Kellermann | 4 | -31/+39 | |
Forward the Error to the caller. | |||||
2014-01-18 | db/upnp/Device: use ContentDirectoryDescriptor's move operator | Max Kellermann | 3 | -7/+7 | |
Reduce bloat. | |||||
2014-01-18 | db/upnp/Discovery: don't copy XML to std::string | Max Kellermann | 3 | -4/+4 | |
Pass "const char *" to Expat. | |||||
2014-01-18 | db/upnp/Discovery: fix memory leak | Max Kellermann | 1 | -0/+1 | |
2014-01-18 | db/upnp: remove unused typedef DevServIt | Max Kellermann | 1 | -2/+0 | |