Merge pull request #1692 from berryzplus/feature/remove_CharPointerW

CharPointerWを削除する

Author: berryzplus

sakura/sakura.vcxproj

@@ -289,7 +289,6 @@
     <ClInclude Include="..\sakura_core\charset\CESI.h" />
     <ClInclude Include="..\sakura_core\charset\CEuc.h" />
     <ClInclude Include="..\sakura_core\charset\charcode.h" />
-    <ClInclude Include="..\sakura_core\charset\CharPointer.h" />
     <ClInclude Include="..\sakura_core\charset\charset.h" />
     <ClInclude Include="..\sakura_core\charset\CJis.h" />
     <ClInclude Include="..\sakura_core\charset\CLatin1.h" />

sakura/sakura.vcxproj.filters

@@ -274,9 +274,6 @@
     <ClInclude Include="..\sakura_core\charset\charcode.h">
       <Filter>Cpp Source Files\charset</Filter>
     </ClInclude>
-    <ClInclude Include="..\sakura_core\charset\CharPointer.h">
-      <Filter>Cpp Source Files\charset</Filter>
-    </ClInclude>
     <ClInclude Include="..\sakura_core\charset\charset.h">
       <Filter>Cpp Source Files\charset</Filter>
     </ClInclude>

sakura_core/charset/CharPointer.h

@@ -35,8 +35,8 @@
 #include "env/CShareData.h"
 #include "typeprop/CImpExpManager.h"	// 2010/4/23 Uchi
 #include "dlg/CDlgOpenFile.h"
-#include "charset/CharPointer.h"
 #include "io/CTextStream.h"
+#include "util/file.h"
 #include "util/shell.h"
 #include "util/module.h"
 #include "apiwrap/StdApi.h"
@@ -69,7 +69,7 @@ static const DWORD p_helpids[] = {	// 2006.10.10 ryoji
 };
 
 static WCHAR* strcnv(WCHAR *str);
-static WCHAR* GetFileName(const WCHAR *fullpath);
+#define GetFileName(path) const_cast<WCHAR*>(GetFileTitlePointer(path))
 
 static int nKeyHelpRMenuType[] = {
 	STR_KEYHELP_RMENU_NONE,
@@ -672,23 +672,3 @@ static WCHAR* strcnv(WCHAR *str)
 	}
 	return str;
 }
-
-/*! フルパスからファイル名を返す
-
-	@date 2006.04.10 fon 新規作成
-	@date 2006.09.14 genta ディレクトリがない場合に最初の1文字が切れないように
-*/
-static WCHAR* GetFileName(const WCHAR* fullpath)
-{
-	const WCHAR* pszName = fullpath;
-	CharPointerT p = fullpath;
-	while( *p != L'\0'  ){
-		if( *p == L'\\' ){
-			pszName = p + 1;
-			p++;
-		}else{
-			p++;
-		}
-	}
-	return const_cast<WCHAR*>(pszName);
-}

sakura_core/typeprop/CPropTypesKeyHelp.cpp

@@ -31,7 +31,10 @@
 
 #include <Shlwapi.h>
 
-#include "charset/CharPointer.h"
+#include <regex>
+#include <string_view>
+
+#include "charset/codechecker.h"
 #include "util/module.h"
 #include "util/window.h"
 #include "env/CShareData.h"
@@ -203,22 +206,21 @@ bool IsLocalDrive( const WCHAR* pszDrive )
 	return true;
 }
 
+/*! フルパスからファイル名を返す
+
+	@date 2006.04.10 fon 新規作成
+	@date 2006.09.14 genta ディレクトリがない場合に最初の1文字が切れないように
+ */
 const WCHAR* GetFileTitlePointer(const WCHAR* pszPath)
 {
-	CharPointerT p;
-	const WCHAR* pszName;
-	p = pszName = pszPath;
-	while( *p )
-	{
-		if( *p == L'\\' ){
-			pszName = p + 1;
-			p++;
-		}
-		else{
-			p++;
+	std::wstring_view name(pszPath);
+	if (const size_t sep = name.find_last_of(L'\\'); sep != std::wstring_view::npos) {
+		if (sep + 1 == name.length()) {
+			return pszPath + name.length();
 		}
+		return pszPath + sep + 1;
 	}
-	return pszName;
+	return pszPath;
 }
 
 /*! fnameが相対パスの場合は、実行ファイルのパスからの相対パスとして開く
@@ -445,23 +447,28 @@ int CalcDirectoryDepth(
 )
 {
 	int depth = 0;
- 
+
 	//	とりあえず\の数を数える
-	for( CharPointerT p = path; *p != L'\0'; ++p ){
-		if( *p == L'\\' ){
-			++depth;
-			//	フルパスには入っていないはずだが念のため
-			//	.\はカレントディレクトリなので，深さに関係ない．
-			while( p[1] == L'.' && p[2] == L'\\' ){
-				p += 2;
-			}
+	std::wstring_view p(path);
+	std::wstring_view::size_type pos = 0;
+	do {
+		pos = p.find_first_of(L'\\', pos);
+		if (pos == std::wstring_view::npos) break;
+		++depth;
+		++pos;
+
+		//	フルパスには入っていないはずだが念のため
+		//	.\はカレントディレクトリなので，深さに関係ない．
+		while (pos + 2 <= p.length()
+			&& p[pos + 0] == L'.'
+			&& p[pos + 1] == L'\\') {
+			pos += 2;
 		}
-	}
+	} while (pos < p.length());
 
 	//	補正
 	//	ドライブ名はパスの深さに数えない
-	if( ((L'A' <= path[0] && path[0] <= L'Z') || (L'a' <= path[0] && path[0] <= L'z'))
-		&& path[1] == L':' && path[2] == L'\\' ){
+	if (std::regex_search(path, std::wregex(LR"(^[A-Z]:\\)", std::wregex::icase))) {
 		//フルパス
 		--depth; // C:\ の \ はルートの記号なので階層深さではない
 	}

sakura_core/util/file.cpp

@@ -54,7 +54,6 @@
 #include "env/CShareData.h"
 #include "env/CSakuraEnvironment.h"
 #include "print/CPrintPreview.h"	/// 2002/2/3 aroka
-#include "charset/CharPointer.h"
 #include "charset/CCodeFactory.h"
 #include "charset/CCodeBase.h"
 #include "CEditApp.h"

sakura_core/window/CEditWnd.cpp

@@ -477,6 +477,45 @@ TEST(file, GetIniFileNameForIO)
 	ASSERT_FALSE(fexist(iniPath.c_str()));
 }
 
+/*!
+ * @brief フルパスからファイル名を取り出す
+ */
+TEST(file, GetFileTitlePointer)
+{
+	// フルパスからファイル名を取得する
+	EXPECT_STREQ(L"test.txt", GetFileTitlePointer(LR"(C:\Temp\test.txt)"));
+
+	// フルパスにファイル名が含まれていない場合
+	EXPECT_STREQ(L"", GetFileTitlePointer(LR"(C:\Temp\)"));
+
+	// フルパスに\\が含まれていない場合
+	EXPECT_STREQ(L"test.txt", GetFileTitlePointer(L"test.txt"));
+
+	// 渡したパスが無効な場合は落ちます。
+	EXPECT_DEATH({ GetFileTitlePointer(nullptr); }, ".*");
+}
+
+/*!
+ * @brief ディレクトリの深さを計算する
+ */
+TEST(file, CalcDirectoryDepth)
+{
+	// ドライブ文字を含むフルパス
+	EXPECT_EQ(1, CalcDirectoryDepth(LR"(C:\Temp\test.txt)"));
+
+	// 共有フォルダを含むフルパス
+	EXPECT_EQ(1, CalcDirectoryDepth(LR"(\\host\Temp\test.txt)"));
+
+	// ドライブなしのフルパス
+	EXPECT_EQ(1, CalcDirectoryDepth(LR"(\Temp\test.txt)"));
+
+	// 相対パス（？）
+	EXPECT_EQ(1, CalcDirectoryDepth(LR"(C:\Temp\.\test.txt)"));
+
+	// 渡したパスが無効な場合は落ちます。
+	EXPECT_DEATH({ CalcDirectoryDepth(nullptr); }, ".*");
+}
+
 /*!
 	GetExtのテスト
  */