時間型の fmt 対応

[Raclamusi]

時間型 ( std::chrono::duration ) を fmt に対応させることを提案します。

# include <Siv3D.hpp> // OpenSiv3D v0.6.5

void Main()
{
	Print << 1h + 3s;
	//Print << U"{}"_fmt(1h + 3s);  // error!

	while (System::Update());
}

実装案1

• s3d::Format を使う
• 実装が簡単

# include <Siv3D.hpp> // OpenSiv3D v0.6.5

template <class Rep, class Period>
struct SIV3D_HIDDEN fmt::formatter<std::chrono::duration<Rep, Period>, s3d::char32>
{
	std::u32string tag;

	auto parse(basic_format_parse_context<s3d::char32>& ctx)
	{
		return s3d::detail::GetFormatTag(tag, ctx);
	}

	template <class FormatContext>
	auto format(const std::chrono::duration<Rep, Period>& value, FormatContext& ctx)
	{
		const s3d::String s = s3d::Format(value);
		const basic_string_view<s3d::char32> sv(s.data(), s.size());

		if (tag.empty())
		{
			return format_to(ctx.out(), U"{}", sv);
		}
		else
		{
			const std::u32string format = (U"{:" + tag + U'}');
			return format_to(ctx.out(), format, sv);
		}
	}
};

void Main()
{
	Print << 1h + 3s;
	Print << U"{}"_fmt(1h + 3s);

	while (System::Update());
}

実装案2

• fmt::format を使う
• s3d::Literals::FormatLiterals::operator ""_fmt の結果は fmt::format と同じ方が自然？
• 標準の出力と同じになるので、より多くの時間型に対応できる
• s3d::Microseconds などの出力の単位が処理系定義になる ( μs or us )

# include <Siv3D.hpp> // OpenSiv3D v0.6.5

template <class Rep, class Period>
struct SIV3D_HIDDEN fmt::formatter<std::chrono::duration<Rep, Period>, s3d::char32>
{
	std::u32string tag;

	auto parse(basic_format_parse_context<s3d::char32>& ctx)
	{
		return s3d::detail::GetFormatTag(tag, ctx);
	}

	template <class FormatContext>
	auto format(const std::chrono::duration<Rep, Period>& value, FormatContext& ctx)
	{
		const s3d::String s = s3d::Unicode::Widen(fmt::format("{}", value));
		const basic_string_view<s3d::char32> sv(s.data(), s.size());

		if (tag.empty())
		{
			return format_to(ctx.out(), U"{}", sv);
		}
		else
		{
			const std::u32string format = (U"{:" + tag + U'}');
			return format_to(ctx.out(), format, sv);
		}
	}
};

void Main()
{
	using Picoseconds = std::chrono::duration<int64, std::pico>;
	using FemtosecondsF = std::chrono::duration<double, std::femto>;

	//Print << Picoseconds{ 42 };  // error!
	Print << U"{}"_fmt(Picoseconds{ 42 });
	Print << U"{}"_fmt(FemtosecondsF{ 42 });
	Print << U"{}"_fmt(std::chrono::years{ 42 });

	{
		// 標準の出力と同じ
		std::ostringstream oss;
		oss << Picoseconds{ 42 };
		assert(U"{}"_fmt(Picoseconds{ 42 }) == Unicode::Widen(oss.str()));
	}

	while (System::Update());
}

[Reputeless]

詳しい報告ありがとうございます！
μs/us の差異は好ましくない（Siv3D は μs us で統一したい）ので、v0.6 世代では実装案 1 が良いです。
pull-request と、コメントで assert で良いので簡単なテストを合わせて送っていただければマージします。

対応する時間型を増やせる件については、v0.8 世代で調査・検討したいと思います。

[Raclamusi]

Siv3D って us じゃないですか？

OpenSiv3D/Siv3D/src/Siv3D/Duration/SivDuration.cpp

Lines 248 to 251 in 4247cab

	void Formatter(FormatData& formatData, const Microseconds& microseconds)
	{
	detail::FormatDuration(formatData, microseconds, U"us");
	}

[Reputeless]

私の記憶違いでした。
us で一貫させたいです。