aboutsummaryrefslogtreecommitdiffstats
path: root/src/tag
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2013-09-27 22:31:24 +0200
committerMax Kellermann <max@duempel.org>2013-10-02 08:57:55 +0200
commit060814daa83f6a94f5934464ae42a406c5c7e947 (patch)
treef636ec6cdbb8e52fda6db987d2a28fc73c7b94b4 /src/tag
parentc53492a76a8a05825e1c7f699c05645eee891199 (diff)
downloadmpd-060814daa83f6a94f5934464ae42a406c5c7e947.tar.gz
mpd-060814daa83f6a94f5934464ae42a406c5c7e947.tar.xz
mpd-060814daa83f6a94f5934464ae42a406c5c7e947.zip
Log: new logging library API
Prepare to migrate away from GLib. Currently, we're still using GLib as a backend.
Diffstat (limited to 'src/tag')
-rw-r--r--src/tag/Aiff.cxx11
-rw-r--r--src/tag/Riff.cxx11
-rw-r--r--src/tag/TagId3.cxx16
3 files changed, 19 insertions, 19 deletions
diff --git a/src/tag/Aiff.cxx b/src/tag/Aiff.cxx
index 09d107d5b..51622fec7 100644
--- a/src/tag/Aiff.cxx
+++ b/src/tag/Aiff.cxx
@@ -19,6 +19,8 @@
#include "config.h" /* must be first for large file support */
#include "Aiff.hxx"
+#include "util/Domain.hxx"
+#include "Log.hxx"
#include <glib.h>
@@ -26,11 +28,9 @@
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>
-#include <errno.h>
#include <string.h>
-#undef G_LOG_DOMAIN
-#define G_LOG_DOMAIN "aiff"
+static constexpr Domain aiff_domain("aiff");
struct aiff_header {
char id[4];
@@ -50,15 +50,14 @@ aiff_seek_id3(FILE *file)
struct stat st;
if (fstat(fileno(file), &st) < 0) {
- g_warning("Failed to stat file descriptor: %s",
- g_strerror(errno));
+ LogErrno(aiff_domain, "Failed to stat file descriptor");
return 0;
}
/* seek to the beginning and read the AIFF header */
if (fseek(file, 0, SEEK_SET) != 0) {
- g_warning("Failed to seek: %s", g_strerror(errno));
+ LogErrno(aiff_domain, "Failed to seek");
return 0;
}
diff --git a/src/tag/Riff.cxx b/src/tag/Riff.cxx
index 3d022e661..d756ebc30 100644
--- a/src/tag/Riff.cxx
+++ b/src/tag/Riff.cxx
@@ -19,6 +19,8 @@
#include "config.h" /* must be first for large file support */
#include "Riff.hxx"
+#include "util/Domain.hxx"
+#include "Log.hxx"
#include <glib.h>
@@ -26,11 +28,9 @@
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>
-#include <errno.h>
#include <string.h>
-#undef G_LOG_DOMAIN
-#define G_LOG_DOMAIN "riff"
+static constexpr Domain riff_domain("riff");
struct riff_header {
char id[4];
@@ -50,15 +50,14 @@ riff_seek_id3(FILE *file)
struct stat st;
if (fstat(fileno(file), &st) < 0) {
- g_warning("Failed to stat file descriptor: %s",
- g_strerror(errno));
+ LogErrno(riff_domain, "Failed to stat file descriptor");
return 0;
}
/* seek to the beginning and read the RIFF header */
if (fseek(file, 0, SEEK_SET) != 0) {
- g_warning("Failed to seek: %s", g_strerror(errno));
+ LogErrno(riff_domain, "Failed to seek");
return 0;
}
diff --git a/src/tag/TagId3.cxx b/src/tag/TagId3.cxx
index 2a217dc47..2f32ef6e5 100644
--- a/src/tag/TagId3.cxx
+++ b/src/tag/TagId3.cxx
@@ -24,6 +24,8 @@
#include "Tag.hxx"
#include "TagBuilder.hxx"
#include "util/Error.hxx"
+#include "util/Domain.hxx"
+#include "Log.hxx"
#include "ConfigGlobal.hxx"
#include "Riff.hxx"
#include "Aiff.hxx"
@@ -36,9 +38,6 @@
#include <errno.h>
#include <string.h>
-#undef G_LOG_DOMAIN
-#define G_LOG_DOMAIN "id3"
-
# ifndef ID3_FRAME_COMPOSER
# define ID3_FRAME_COMPOSER "TCOM"
# endif
@@ -58,6 +57,8 @@
#define ID3_FRAME_ALBUM_ARTIST "TPE2"
#endif
+static constexpr Domain id3_domain("id3");
+
static inline bool
tag_is_id3v1(struct id3_tag *tag)
{
@@ -104,8 +105,9 @@ import_id3_string(bool is_id3v1, const id3_ucs4_t *ucs4)
nullptr, nullptr,
nullptr, nullptr);
if (utf8 == nullptr) {
- g_debug("Unable to convert %s string to UTF-8: '%s'",
- encoding, isostr);
+ FormatWarning(id3_domain,
+ "Unable to convert %s string to UTF-8: '%s'",
+ encoding, isostr);
g_free(isostr);
return nullptr;
}
@@ -526,7 +528,7 @@ tag_id3_riff_aiff_load(FILE *file)
id3_byte_t *buffer = (id3_byte_t *)g_malloc(size);
size_t ret = fread(buffer, size, 1, file);
if (ret != 1) {
- g_warning("Failed to read RIFF chunk");
+ LogWarning(id3_domain, "Failed to read RIFF chunk");
g_free(buffer);
return nullptr;
}
@@ -564,7 +566,7 @@ tag_id3_scan(const char *path_fs,
struct id3_tag *tag = tag_id3_load(path_fs, error);
if (tag == nullptr) {
if (error.IsDefined())
- g_warning("%s", error.GetMessage());
+ LogError(error);
return false;
}