SongUpdate: use OpenArchiveInputStream() to open archive file
Don't pass Path::c_str() to tag_stream_scan() which requires a UTF-8 string.
This commit is contained in:
parent
605af61a0f
commit
41cc31c124
@ -728,6 +728,7 @@ if ENABLE_ARCHIVE
|
|||||||
noinst_LIBRARIES += libarchive.a
|
noinst_LIBRARIES += libarchive.a
|
||||||
|
|
||||||
libmpd_a_SOURCES += \
|
libmpd_a_SOURCES += \
|
||||||
|
src/TagArchive.cxx src/TagArchive.hxx \
|
||||||
src/db/update/Archive.cxx
|
src/db/update/Archive.cxx
|
||||||
|
|
||||||
libarchive_a_SOURCES = \
|
libarchive_a_SOURCES = \
|
||||||
|
@ -37,6 +37,10 @@
|
|||||||
#include "TagFile.hxx"
|
#include "TagFile.hxx"
|
||||||
#include "TagStream.hxx"
|
#include "TagStream.hxx"
|
||||||
|
|
||||||
|
#ifdef ENABLE_ARCHIVE
|
||||||
|
#include "TagArchive.hxx"
|
||||||
|
#endif
|
||||||
|
|
||||||
#include <assert.h>
|
#include <assert.h>
|
||||||
#include <string.h>
|
#include <string.h>
|
||||||
#include <sys/stat.h>
|
#include <sys/stat.h>
|
||||||
@ -140,7 +144,7 @@ Song::UpdateFileInArchive(const Storage &storage)
|
|||||||
return false;
|
return false;
|
||||||
|
|
||||||
TagBuilder tag_builder;
|
TagBuilder tag_builder;
|
||||||
if (!tag_stream_scan(path_fs.c_str(), full_tag_handler, &tag_builder))
|
if (!tag_archive_scan(path_fs, full_tag_handler, &tag_builder))
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
tag_builder.Commit(tag);
|
tag_builder.Commit(tag);
|
||||||
|
45
src/TagArchive.cxx
Normal file
45
src/TagArchive.cxx
Normal file
@ -0,0 +1,45 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (C) 2003-2015 The Music Player Daemon Project
|
||||||
|
* http://www.musicpd.org
|
||||||
|
*
|
||||||
|
* This program is free software; you can redistribute it and/or modify
|
||||||
|
* it under the terms of the GNU General Public License as published by
|
||||||
|
* the Free Software Foundation; either version 2 of the License, or
|
||||||
|
* (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This program is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
* GNU General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU General Public License along
|
||||||
|
* with this program; if not, write to the Free Software Foundation, Inc.,
|
||||||
|
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
|
||||||
|
*/
|
||||||
|
|
||||||
|
#include "config.h"
|
||||||
|
#include "TagArchive.hxx"
|
||||||
|
#include "TagStream.hxx"
|
||||||
|
#include "fs/Path.hxx"
|
||||||
|
#include "util/Error.hxx"
|
||||||
|
#include "input/InputStream.hxx"
|
||||||
|
#include "input/plugins/ArchiveInputPlugin.hxx"
|
||||||
|
#include "thread/Cond.hxx"
|
||||||
|
|
||||||
|
#include <assert.h>
|
||||||
|
|
||||||
|
bool
|
||||||
|
tag_archive_scan(Path path, const tag_handler &handler, void *handler_ctx)
|
||||||
|
{
|
||||||
|
assert(!path.IsNull());
|
||||||
|
|
||||||
|
Mutex mutex;
|
||||||
|
Cond cond;
|
||||||
|
auto *is = OpenArchiveInputStream(path, mutex, cond, IgnoreError());
|
||||||
|
if (is == nullptr)
|
||||||
|
return false;
|
||||||
|
|
||||||
|
bool result = tag_stream_scan(*is, handler, handler_ctx);
|
||||||
|
delete is;
|
||||||
|
return result;
|
||||||
|
}
|
39
src/TagArchive.hxx
Normal file
39
src/TagArchive.hxx
Normal file
@ -0,0 +1,39 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (C) 2003-2015 The Music Player Daemon Project
|
||||||
|
* http://www.musicpd.org
|
||||||
|
*
|
||||||
|
* This program is free software; you can redistribute it and/or modify
|
||||||
|
* it under the terms of the GNU General Public License as published by
|
||||||
|
* the Free Software Foundation; either version 2 of the License, or
|
||||||
|
* (at your option) any later version.
|
||||||
|
*
|
||||||
|
* This program is distributed in the hope that it will be useful,
|
||||||
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||||
|
* GNU General Public License for more details.
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU General Public License along
|
||||||
|
* with this program; if not, write to the Free Software Foundation, Inc.,
|
||||||
|
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
|
||||||
|
*/
|
||||||
|
|
||||||
|
#ifndef MPD_TAG_ARCHIVE_HXX
|
||||||
|
#define MPD_TAG_ARCHIVE_HXX
|
||||||
|
|
||||||
|
#include "check.h"
|
||||||
|
|
||||||
|
class Path;
|
||||||
|
struct tag_handler;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Scan the tags of a song file inside an archive. Invokes matching
|
||||||
|
* decoder plugins, but does not invoke the special "APE" and "ID3"
|
||||||
|
* scanners.
|
||||||
|
*
|
||||||
|
* @return true if the file was recognized (even if no metadata was
|
||||||
|
* found)
|
||||||
|
*/
|
||||||
|
bool
|
||||||
|
tag_archive_scan(Path path, const tag_handler &handler, void *handler_ctx);
|
||||||
|
|
||||||
|
#endif
|
Loading…
Reference in New Issue
Block a user