--- nono/wx/wxlogmonitor.cpp	2026/04/29 17:04:56	1.1.1.7
+++ nono/wx/wxlogmonitor.cpp	2026/04/29 17:05:16	1.1.1.11
@@ -4,84 +4,67 @@
 // Licensed under nono-license.txt
 //
 
-#include "wxlogmonitor.h"
-#include "logger.h"
-#include "sjis.h"
-
 //
 // ログモニター
 //
 
+#include "wxlogmonitor.h"
+#include "logger.h"
+#include "mainapp.h"
+#include "sjis.h"
+
 // イベントテーブル
 wxBEGIN_EVENT_TABLE(WXLogMonitor, inherited)
 	EVT_SIZE(WXLogMonitor::OnSize)
-	EVT_SCROLL(WXLogMonitor::OnScroll)
 wxEND_EVENT_TABLE()
 
 // コンストラクタ
 WXLogMonitor::WXLogMonitor(wxWindow *parent)
-	: inherited(parent, wxID_ANY, _("Log"))
+	: inherited(parent, wxID_ANY, _("Log"), DEFAULT_STYLE | wxRESIZE_BORDER)
 {
 	col = 80;
 	row = 40;
 
 	monitor.obj = &log;
-	monitor.func = (MonitorCallback_t)&LogMonitor::MonitorUpdate;
+	monitor.func = ToMonitorCallback(&LogMonitor::MonitorUpdate);
 	monitor.SetSize(col, row);
 	// プライベート利用なので Regist() は不要
 
 	log.Init(this);
 
 	// →
-	// +--+-----------------------+--+--+
-	// |  |↓+------------------+ |  |  |
-	// |  |  | 上余白(padding1) | |  |  |
-	// |左|  +------------------+ |右|  |
-	// |余|  | TextScreen       | |余| <- VScroll
-	// |白|  +------------------+ |白|  |
-	// |  |  | 下余白(padding2) | |  |  |
-	// |  |  +------------------+ |  |  |
-	// +--+-----------------------+--+--+
-
-	topsizer = new wxBoxSizer(wxHORIZONTAL);
-	// 左余白
-	topsizer->Add(new WXPaddingPanel(this), 0, wxEXPAND);
-	// 中列の縦 sizer
-	auto *vbox = new wxBoxSizer(wxVERTICAL);
-	topsizer->Add(vbox, 0, wxEXPAND);
-
-	// パディングパネル(上下)
-	// ウィンドウを1行ずつ伸縮させるためには、クライアント領域がフォント高さの
-	// 整数倍になってないといけない。TextScreen はそれ自身が行の整数倍の大きさ
-	// だが、上下の余白はその時点のフォントサイズによって変更する必要がある。
-	// 先にサイズ不定のパネルだけ置いといて、フォントサイズ確定後に大きさを
-	// 確定させる。DoSize() 参照。
-	padding1 = new WXPaddingPanel(this);
-	padding2 = new WXPaddingPanel(this);
-	screen = new WXTextScreen(this, monitor);
-	vbox->Add(padding1, 0, wxEXPAND);
-	vbox->Add(screen, 1, wxEXPAND);
-	vbox->Add(padding2, 0, wxEXPAND);
+	// +------------+---------+
+	// | TextScreen | VScroll |
+	// +------------+---------+
+
+	wxBoxSizer *topsizer = new wxBoxSizer(wxHORIZONTAL);
+
+	// テキストスクリーン
+	screen = new WXMonitorPanel(this, monitor);
+	topsizer->Add(screen, 1, wxEXPAND);
 
-	// 右余白
-	topsizer->Add(new WXPaddingPanel(this), 0, wxEXPAND);
 	// スクロールバー
-	vscroll = new wxScrollBar(this, wxID_ANY, wxDefaultPosition,
-		wxDefaultSize, wxSB_VERTICAL);
+	vscroll = new WXScrollBar(this, wxID_ANY, wxSB_VERTICAL);
 	topsizer->Add(vscroll, 0, wxEXPAND);
 
 	SetSizer(topsizer);
-
 	// ウィンドウサイズを確定させる
 	DoSize();
 
 	// スクロールのために (パネル領域での) MouseWheel イベントもここで
-	// 受け取りたい。スクロールバー上のマウスホイール操作は最初から Scroll
-	// イベントとして処理されているので問題ないが、パネル領域でのマウス
-	// ホイール操作はここ(wxFrame)ではなくパネルに対して飛んでくる。
+	// 受け取りたい。スクロールバー上のマウスホイール操作はスクロールバーが
+	// 処理しているので問題ないが、パネル領域でのマウスホイール操作はここ
+	// (wxFrame)ではなくパネルに対して飛んでくる。
 	// ここで一緒に処理したほうが楽なので、こちらに回す。
 	screen->Connect(wxEVT_MOUSEWHEEL,
 		wxMouseEventHandler(WXLogMonitor::OnMouseWheel), NULL, this);
+
+	// スクロールバーからの位置変更通知
+	vscroll->Connect(NONO_EVT_SCROLL,
+		wxScrollEventHandler(WXLogMonitor::OnScroll), NULL, this);
+
+	// XXX ログウィンドウのコピーはまだ動いてない
+	screen->DisconnectContextMenu();
 }
 
 // デストラクタ
@@ -89,11 +72,11 @@ WXLogMonitor::~WXLogMonitor()
 {
 }
 
-// フォントサイズが確定したら、それによってウィンドウサイズを再計算する。
-void
-WXLogMonitor::DoSize()
+bool
+WXLogMonitor::Layout()
 {
-	CalcVSize(0);
+	// 縦リサイズ可能レイアウト
+	return LayoutTextVResize(screen);
 }
 
 // サイズ変更イベント
@@ -126,7 +109,7 @@ WXLogMonitor::SetScroll()
 	}
 	thumbsize = row;
 	pagesize = thumbsize - 1;
-	vscroll->SetScrollbar(pos, thumbsize, range, pagesize);
+	vscroll->SetScrollParam(pos, thumbsize, range, pagesize);
 }
 
 // マウスホイールイベント (WXTextScreen 上で起きたやつをこっちに回してある)
@@ -146,13 +129,14 @@ WXLogMonitor::OnMouseWheel(wxMouseEvent&
 		pos = maxpos;
 
 	DoScroll(pos);
+	// スクロールバーの位置を追従
+	vscroll->SetThumbPosition(pos);
 }
 
-// スクロールイベント (全部入り)
+// スクロールバーからの通知イベント
 void
 WXLogMonitor::OnScroll(wxScrollEvent& event)
 {
-	// Thumb の上端の位置 (0 なら Thumb が上端に接している)
 	DoScroll(event.GetPosition());
 }
 
@@ -182,18 +166,16 @@ WXLogMonitor::DoScroll(int pos)
 	// パネル最下行に表示する行の最新行からのオフセット
 	// (0 なら Thumb が下端に接している)
 	vpos = vscroll->GetRange() - vscroll->GetThumbSize() - pos;
-
-	// Thumb の位置を更新
-	vscroll->SetThumbPosition(pos);
 }
 
+
 //
 // モニターオブジェクト
 //
 
 // コンストラクタ
 LogMonitor::LogMonitor()
-	: inherited("LogMonitor")
+	: inherited(OBJ_LOGMONITOR)
 {
 }
 
@@ -207,6 +189,8 @@ LogMonitor::Init(WXLogMonitor *parent_)
 {
 	parent = parent_;
 
+	logger = gMainApp.GetLogger();
+
 	// バックログは今の所固定長
 	logbuf.Init(parent->col, backlog);
 	logbuf.Mode = TextScreen::Ring;
@@ -230,9 +214,9 @@ LogMonitor::MonitorUpdate(Monitor *, Tex
 	// 流速が早すぎた場合にこのループから抜けるタイミングがなくなる可能性が
 	// (可能性としては) あるので、適当に1画面分くらいで打ち切る。
 	// 取り込み残した分は次回取り込まれるし、その流速も上回るようだと
-	// どのみち gLogger の固定長キューで落ちる。
+	// どのみち Logger の固定長キューで落ちる。
 	for (int i = 0; i < parent->row; i++) {
-		if (gLogger.Read(buf, sizeof(buf)) == false)
+		if (logger->Read(buf, sizeof(buf)) == false)
 			break;
 		Append(buf);
 	}
@@ -303,17 +287,17 @@ LogMonitor::IncLines()
 void
 LogMonitor::Append(const char *utfbuf)
 {
-	const char *sjis;
 	const char *s;
 	int x;
 
 	// 文字コードの変換
 	wxString utfstr(utfbuf, wxConvUTF8);
-	sjis = (const char *)utfstr.mb_str(conv);
+	// mb_str() をキャストせずに一度保持しておく必要がある (scan-build 対策)
+	auto sjis = utfstr.mb_str(conv);
 
 	logbuf.Locate(0, cursor);
 
-	for (s = sjis; *s; ) {
+	for (s = (const char *)sjis; *s; ) {
 		// 最終桁で2バイト文字の1バイト目が来たら
 		// 代わりに1文字空白を入れる (それによってこの後改行が起きる)
 		if (SJIS::IsZenkaku(*s)) {