cleanup: デッドな spdlog 残骸を一掃する - #105
Merged
Merged
Conversation
#71 で spdlog submodule が除去された後に残っていたデッドコード / ドキュメント残骸を一掃する。 - include/logger.h を削除。spdlog をラップする setup_spdlog() を 定義していたが、ccbench 本体 (cc/ common/ include/) からの #include は 0 件で完全にデッド。spdlog submodule 除去済みのため #include "spdlog/spdlog.h" も解決できない壊れた状態だった。 - docs/architecture_{ja,en}.md の third_party submodule 一覧から spdlog を削除し、実態 (masstree / mimalloc / googletest) に揃える。 ハードルール#5 に従い _ja / _en を同一コミットで更新。 未追跡だった third_party/spdlog/ ディレクトリは作業ツリーからは 除去したが、元々 git に追跡されていないため commit 差分には現れない。
This was referenced May 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
#71 で spdlog submodule が除去された後に残っていた、デッドコード / ドキュメント残骸を 1 つの PR で完全に一掃する。
変更内容
include/logger.hを削除spdlog をラップする
setup_spdlog()を定義していたが、ccbench 本体 (cc/common/include/) からの#includeは 0 件で完全にデッドコード。spdlog submodule は third_party: 未使用の spdlog submodule を除去 (#64 の一部) #71 で除去済みのため、#include "spdlog/spdlog.h"が解決できない壊れた状態でもあった。docs/architecture_{ja,en}.mdの third_party submodule 一覧からspdlogを削除実態 (masstree / mimalloc / googletest) に揃える。CLAUDE.md ハードルールVldb #5 に従い
_ja/_enを同一コミットで更新。third_party/spdlog/を作業ツリーから除去third_party: 未使用の spdlog submodule を除去 (#64 の一部) #71 で submodule 登録が外れた後に残った stale なディレクトリ。
.gitmodulesにも.gitignoreにも無く git に追跡されていないため、rm -rfで作業ツリーからは除去したが、元々追跡されていないので本 PR の commit 差分には現れない。これは想定どおりの挙動。確認
git grep spdlog/git grep logger.h: 変更前はinclude/logger.h自身とdocs/architecture_{ja,en}.mdのみヒット → 変更後は 0 件（追跡ファイル）。cmake -S . -B build（configure のみ）が通ることを確認。include/logger.hはどこからも include されていないためビルドへの影響なし。#100 との関係
本 PR は #100（[P8] cleanup: 未使用の
include/logger.hと docs の spdlog 記述を除去）のスコープを完全に含む（logger.h削除 +docs/architecture_{ja,en}.mdの spdlog 記述削除）。よって #100 は本 PR で superseded となるため、別途クローズ推奨（本 PR ではClosesで自動クローズはしない）。Closes #86