Skip to content

Commit

Permalink
Make fs::path::string() always return utf-8 string
Browse files Browse the repository at this point in the history
  • Loading branch information
ken2812221 committed Sep 22, 2018
1 parent 920c090 commit 2c3eade
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
1 change: 0 additions & 1 deletion src/fs.h
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@

#include <boost/filesystem.hpp>
#include <boost/filesystem/fstream.hpp>
#include <boost/filesystem/detail/utf8_codecvt_facet.hpp>

/** Filesystem operations and types */
namespace fs = boost::filesystem;
Expand Down
6 changes: 2 additions & 4 deletions src/qt/guiutil.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,6 @@
#include <QFontDatabase>
#endif

static fs::detail::utf8_codecvt_facet utf8;

namespace GUIUtil {

QString dateTimeStr(const QDateTime &date)
Expand Down Expand Up @@ -764,12 +762,12 @@ void setClipboard(const QString& str)

fs::path qstringToBoostPath(const QString &path)
{
return fs::path(path.toStdString(), utf8);
return fs::path(path.toStdString());
}

QString boostPathToQString(const fs::path &path)
{
return QString::fromStdString(path.string(utf8));
return QString::fromStdString(path.string());
}

QString formatDurationStr(int secs)
Expand Down
4 changes: 4 additions & 0 deletions src/util.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1206,7 +1206,11 @@ void SetupEnvironment()
// A dummy locale is used to extract the internal default locale, used by
// fs::path, which is then used to explicitly imbue the path.
std::locale loc = fs::path::imbue(std::locale::classic());
#ifndef WIN32
fs::path::imbue(loc);
#else
fs::path::imbue(std::locale(loc, new std::codecvt_utf8_utf16<wchar_t>()));
#endif
}

bool SetupNetworking()
Expand Down
1 change: 0 additions & 1 deletion test/lint/lint-includes.sh
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@ EXPECTED_BOOST_INCLUDES=(
boost/chrono/chrono.hpp
boost/date_time/posix_time/posix_time.hpp
boost/filesystem.hpp
boost/filesystem/detail/utf8_codecvt_facet.hpp
boost/filesystem/fstream.hpp
boost/multi_index/hashed_index.hpp
boost/multi_index/ordered_index.hpp
Expand Down

0 comments on commit 2c3eade

Please sign in to comment.