Merge remote-tracking branch 'upstream/master'

Author: HoppingTappy

.github/workflows/sonarscan.yml

@@ -64,6 +64,10 @@ jobs:
     - name: Setup MSBuild
       uses: microsoft/setup-msbuild@v1
 
+    - name: Run the normal MSBuild process
+      run: build-sln.bat ${{ env.BUILD_PLATFORM }} ${{ env.BUILD_CONFIGURATION }}
+      shell: cmd
+
     - name: Fetch Build-Wrapper
       working-directory: ${{ github.workspace }}\.sonar
       run: |

README.md

@@ -123,7 +123,7 @@ vs_community__XXXXX.exe --config <.vsconfig のファイルパス>
 
 ## How to build
 
-- [7Zip](https://sevenzip.osdn.jp/) のインストールして 7z.exe へのパスを通します。
+- [7-Zip](https://7-zip.opensource.jp/) をインストールして 7z.exe へのパスを通します。
 - Visual Studio で `sakura.sln` を開いてビルドします。
 
 ### 詳細情報

build.md

@@ -24,7 +24,13 @@
     - [githash.h の更新のスキップ](#githashh-の更新のスキップ)
     - [Powershell によるZIPファイルの圧縮、解凍、内容確認の強制](#powershell-によるzipファイルの圧縮解凍内容確認の強制)
     - [CI でのビルドをスキップする方法](#ci-でのビルドをスキップする方法)
+      - [参考情報](#参考情報)
     - [MinGW w64 ビルド](#mingw-w64-ビルド)
+    - [MinGW w64 インストール方法](#mingw-w64-インストール方法)
+    - [Msys2 コンソールを開く方法](#msys2-コンソールを開く方法)
+    - [Msys2 インストール方法](#msys2-インストール方法)
+    - [PowerShell の管理者コンソールを開く方法](#powershell-の管理者コンソールを開く方法)
+    - [Chocolatey のインストール方法](#chocolatey-のインストール方法)
 
 <!-- /TOC -->
 
@@ -188,26 +194,57 @@ build-sln.bat x64   Debug
 
 生成されるバイナリは正しく動作しないが、MinGWでのビルドも可能。
 
+コマンド実行例（[MinGW w64](#mingw-w64-インストール方法) のインストールが必要。）
 
-MinGW64のビルド環境
+```cmd
+build-gnu.bat MinGW Debug
+build-gnu.bat MinGW Release
+```
 
-* [pleiades 4.6 Neon](http://mergedoc.osdn.jp/)
-* [MSYS2+MinGW-w64](https://gist.github.com/Hamayama/eb4b4824ada3ac71beee0c9bb5fa546d)
-* [MinGW-w64](https://sourceforge.net/projects/mingw-w64/files/Toolchains%20targetting%20Win64/)
+### MinGW w64 インストール方法
 
-cpu | thread モデル | 例外モデル | コメント
----- | ---- | ---- | ----
-x86_64 | win32 | seh | windows向け。外部DLL不要
-x86_64 | posix | seh | 標準。pthreadのDLLが必要
-x86_64 | win32 | sjlj | 外部DLL不要
-x86_64 | posix | sjlj | pthreadのDLLが必要
+[Msys2 コンソール](#msys2-コンソールを開く方法) で以下のコマンドを入力し `pacman` パッケージ を最新化します。
 
-標準的なMinGWセットアップでビルドしたバイナリは ```libwinpthread-1.dll``` に依存することに注意。
+```bash
+pacman -Syuu
+```
 
+[Msys2 コンソール](#msys2-コンソールを開く方法) で以下のコマンドを入力し `MinGW-w64` をインストールします。  
 
-コマンド実行例
+```bash
+pacman -S --noconfirm mingw-w64-x86_64-toolchain
+```
+
+### Msys2 コンソールを開く方法
+
+`C:\msys64\msys2.exe` を実行します。（[Msys2](#msys2-インストール方法) のインストールが必要。）
+
+### Msys2 インストール方法
+
+[PowerShell の管理者コンソール](#powershell-の管理者コンソールを開く方法)で以下のコマンドを入力し `msys2` をインストールします。（[Chocolatey](#chocolatey-のインストール方法) のインストールが必要。）
+
+```powershell
+choco install msys2 --params "/InstallDir:C:\msys64"
+```
+
+### PowerShell の管理者コンソールを開く方法
+
+Windowsタスクバーの検索窓に `powershell` と入力します。
+
+検索結果に `Windows PowerShell (x86)` が表示されるので `管理者として実行` をクリックします。
+
+### Chocolatey のインストール方法
 
+[PowerShell の管理者コンソール](#powershell-の管理者コンソールを開く方法) で以下のコマンドを実行します。
+
+```powershell
+Set-ExecutionPolicy Bypass -Scope Process -Force; [System.Net.ServicePointManager]::SecurityProtocol = [System.Net.ServicePointManager]::SecurityProtocol -bor 3072; iex ((New-Object System.Net.WebClient).DownloadString('https://chocolatey.org/install.ps1'))
 ```
-path=C:\msys64\mingw64\bin;%path%
-mingw32-make -C sakura_core -j4
+
+詳細な手順は [Chocolateyの公式サイト](https://chocolatey.org/install) で確認してください。
+
+[PowerShell の管理者コンソール](#powershell-の管理者コンソールを開く方法) で以下のコマンドを実行し `Chocolatey` のバージョンが表示されたらインストールできています。
+
+```powershell
+choco
 ```

sakura_core/apiwrap/StdControl.h

@@ -41,6 +41,7 @@ UNICODE版では問題無いが、ANSI版では設定の前にコード変換す
 
 #include <windows.h>
 #include <Commctrl.h>
+#include <shlwapi.h>
 #include "mem/CNativeW.h"
 #include <vector>
 
@@ -219,6 +220,15 @@ namespace ApiWrap{
 	{
 		::SendMessage( hwndCombo, CB_GETEDITSEL, WPARAM( &dwSelStart ), LPARAM( &dwSelEnd ) );
 	}
+	inline void Combo_SHAutoComplete( HWND hwndCombo, DWORD dwFlags )
+	{
+		COMBOBOXINFO comboInfo;
+		comboInfo.cbSize = sizeof(comboInfo);
+		if (0 != GetComboBoxInfo(hwndCombo, &comboInfo))
+		{
+			SHAutoComplete(comboInfo.hwndItem, dwFlags);
+		}
+	}
 
 	// -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- //
 	//                      リストボックス                         //

sakura_core/cmd/CViewCommander_Edit.cpp

@@ -900,7 +900,7 @@ void CViewCommander::DelCharForOverwrite( const wchar_t* pszInput, int nLen )
 					nPos += CNativeW::GetSizeOfChar(line.GetPtr(), line.GetLength(), nPos);
 					nDelLen = 1;
 					if( nKetaDiff < 0 && nPos < line.GetLength() ){
-						wchar_t c = line.At(nPos);
+						wchar_t c = line[nPos];
 						if( c != WCODE::TAB && !WCODE::IsLineDelimiter(c,
 								GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
 							nDelLen = 2;

sakura_core/dlg/CDlgGrep.cpp

@@ -323,6 +323,9 @@ BOOL CDlgGrep::OnInitDialog( HWND hwndDlg, WPARAM wParam, LPARAM lParam )
 	Combo_SetExtendedUI( GetItemHwnd( IDC_COMBO_EXCLUDE_FILE ), TRUE );
 	Combo_SetExtendedUI( GetItemHwnd( IDC_COMBO_EXCLUDE_FOLDER ), TRUE );
 
+	/* 入力補完を機能させる */
+	Combo_SHAutoComplete(GetItemHwnd( IDC_COMBO_FOLDER ), SHACF_FILESYS_DIRS|SHACF_AUTOAPPEND_FORCE_ON);
+
 	/* ダイアログのアイコン */
 //2002.02.08 Grepアイコンも大きいアイコンと小さいアイコンを別々にする。
 	HICON	hIconBig, hIconSmall;

sakura_core/doc/layout/CLayoutMgr_DoLayout.cpp

@@ -48,7 +48,7 @@ static bool _GetKeywordLength(
 	CLogicInt nWordBgn = nPos;
 	CLogicInt nWordLen = CLogicInt(0);
 	CLayoutInt nWordKetas = CLayoutInt(0);
-	while(nPos<cLineStr.GetLength() && IS_KEYWORD_CHAR(cLineStr.At(nPos))){
+	while(nPos<cLineStr.GetLength() && IS_KEYWORD_CHAR(cLineStr[nPos])){
 		CLogicXInt nCharSize = CNativeW::GetSizeOfChar( cLineStr, nPos );
 		CLayoutInt k = cLayoutMgr.GetLayoutXOfChar(cLineStr, nPos);
 
@@ -209,8 +209,8 @@ void CLayoutMgr::_DoGyotoKinsoku(SLayoutWork* pWork, PF_OnLine pfOnLine)
 
 		if( _IsKinsokuPosHead( GetMaxLineLayout() - pWork->nPosX, nCharKetas1, nCharKetas2 )
 		 && IsKinsokuHead( pWork->cLineStr.At( pWork->nPos + nCharSize ) )
-		 && !IsKinsokuHead( pWork->cLineStr.At( pWork->nPos ) )		// 1字前が行頭禁則の対象でないこと
-		 && !IsKinsokuKuto( pWork->cLineStr.At( pWork->nPos ) ) )	// 1字前が句読点ぶら下げの対象でないこと
+		 && !IsKinsokuHead( pWork->cLineStr[ pWork->nPos ] )		// 1字前が行頭禁則の対象でないこと
+		 && !IsKinsokuKuto( pWork->cLineStr[ pWork->nPos ] ) )	// 1字前が句読点ぶら下げの対象でないこと
 		{
 			pWork->nWordBgn = pWork->nPos;
 			pWork->nWordLen = nCharSize + CNativeW::GetSizeOfChar( pWork->cLineStr, pWork->nPos + nCharSize );
@@ -233,7 +233,7 @@ void CLayoutMgr::_DoGyomatsuKinsoku(SLayoutWork* pWork, PF_OnLine pfOnLine)
 		CLayoutXInt nCharKetas1 = GetLayoutXOfChar( pWork->cLineStr, pWork->nPos );
 		CLayoutXInt nCharKetas2 = GetLayoutXOfChar( pWork->cLineStr, pWork->nPos + nCharSize );
 
-		if( _IsKinsokuPosTail( GetMaxLineLayout() - pWork->nPosX, nCharKetas1, nCharKetas2 ) && IsKinsokuTail( pWork->cLineStr.At( pWork->nPos ) ) )
+		if( _IsKinsokuPosTail( GetMaxLineLayout() - pWork->nPosX, nCharKetas1, nCharKetas2 ) && IsKinsokuTail( pWork->cLineStr[ pWork->nPos ] ) )
 		{
 			pWork->nWordBgn = pWork->nPos;
 			pWork->nWordLen = nCharSize;
@@ -305,7 +305,7 @@ void CLayoutMgr::_MakeOneLine(SLayoutWork* pWork, PF_OnLine pfOnLine)
 		//@@@ 2002.09.22 YAZAKI
 		color.CheckColorMODE( &pWork->pcColorStrategy, pWork->nPos, pWork->cLineStr );
 
-		if( pWork->cLineStr.At(pWork->nPos) == WCODE::TAB ){
+		if( pWork->cLineStr[pWork->nPos] == WCODE::TAB ){
 			if(_DoTab(pWork, pfOnLine)){
 				continue;
 			}

sakura_core/env/CShareData.cpp

@@ -355,7 +355,7 @@ bool CShareData::InitShareData()
 			sEdit.m_bConvertEOLPaste = false;			/* 改行コードを変換して貼り付ける */	// 2009.02.28 salarm
 			sEdit.m_bEnableExtEol = false;
 			sEdit.m_bBoxSelectLock = true;
-			sEdit.m_bVistaStyleFileDialog = false;
+			sEdit.m_bVistaStyleFileDialog = true;
 
 			sEdit.m_bNotOverWriteCRLF = TRUE;			/* 改行は上書きしない */
 			sEdit.m_bOverWriteFixMode = false;			// 文字幅に合わせてスペースを詰める

sakura_core/mem/CNativeW.h

@@ -45,6 +45,7 @@ class CStringRef final{
 	[[nodiscard]] int GetLength() const noexcept { return static_cast<int>(m_nDataLen); }
 	[[nodiscard]] bool IsValid() const noexcept { return m_pData != nullptr; }
 	[[nodiscard]] wchar_t At( size_t nIndex ) const noexcept;
+	[[nodiscard]] wchar_t operator []( size_t nIndex ) const noexcept { return m_pData[nIndex]; }
 
 private:
 	const wchar_t*	m_pData = nullptr;

sakura_core/view/CRuler.cpp

@@ -184,7 +184,7 @@ void CRuler::DrawRulerBg(CGraphics& gr)
 
 	// 目盛り線を1本ずつ描画するのではなく後述する PolyPolyline でまとめて描画を行う
 	const int nWidth = (Int)(m_pEditView->GetTextArea().GetRightCol() - i);
-	const size_t nLinesToDraw = 1 + std::min<int>((nWidth + 1 + 1 + oneColumn - 1) / oneColumn, nMaxLineKetas - keta + 1);
+	const size_t nLinesToDraw = 1 + std::min<int>((nWidth + 1 + 1 + oneColumn - 1) / oneColumn, std::max(nMaxLineKetas - keta, 0) + 1);
 	auto& apt = m_apt;
 	auto& asz = m_asz;
 	apt.resize(nLinesToDraw * 2);

sakura_core/view/colors/CColor_Comment.cpp

@@ -51,7 +51,7 @@ bool CColor_LineComment::EndColor(const CStringRef& cStr, int nPos)
 	}
 
 	//改行
-	if( WCODE::IsLineDelimiter(cStr.At(nPos), GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
+	if( WCODE::IsLineDelimiter(cStr[nPos], GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
 		return true;
 	}
 

sakura_core/view/colors/CColor_Heredoc.cpp

@@ -78,21 +78,21 @@ bool CColor_Heredoc::BeginColor(const CStringRef& cStr, int nPos)
 		const int length = cStr.GetLength();
 		int nPosIdStart = nPos + 3;
 		for(; nPosIdStart < length; nPosIdStart++ ){
-			if(cStr.At(nPosIdStart) != L'\t' && cStr.At(nPosIdStart) != L' '){
+			if(cStr[nPosIdStart] != L'\t' && cStr[nPosIdStart] != L' '){
 				break;
 			}
 		}
 		wchar_t quote = L'\0';
 		if( !(nPosIdStart < length) ){
 			return false;
 		}
-		if( cStr.At(nPosIdStart) == L'\'' || cStr.At(nPosIdStart) == L'"' ){
-			quote = cStr.At(nPosIdStart);
+		if (cStr[nPosIdStart] == L'\'' || cStr[nPosIdStart] == L'"') {
+			quote = cStr[nPosIdStart];
 			nPosIdStart++;
 		}
 		int i = nPosIdStart;
 		for(; i < length; i++ ){
-			if( !(WCODE::IsAZ(cStr.At(i)) || WCODE::Is09(cStr.At(i)) || cStr.At(i) == L'_') ){
+			if( !(WCODE::IsAZ(cStr[i]) || WCODE::Is09(cStr[i]) || cStr[i] == L'_') ){
 				break;
 			}
 		}
@@ -101,13 +101,13 @@ bool CColor_Heredoc::BeginColor(const CStringRef& cStr, int nPos)
 		}
 		const int k = i;
 		if( quote != L'\0' ){
-			if( i < length && cStr.At(i) == quote ){
+			if( i < length && cStr[i] == quote ){
 				i++;
 			}else{
 				return false;
 			}
 		}
-		if( i < length && WCODE::IsLineDelimiter(cStr.At(i), GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
+		if( i < length && WCODE::IsLineDelimiter(cStr[i], GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
 			m_id = std::wstring(cStr.GetPtr()+nPosIdStart, k - nPosIdStart);
 			m_pszId = m_id.c_str();
 			m_nSize = m_id.size();
@@ -129,11 +129,11 @@ bool CColor_Heredoc::EndColor(const CStringRef& cStr, int nPos)
 				return false;
 			}else{
 				int i = m_nSize;
-				if( i + 1 < cStr.GetLength() && cStr.At(i) == L';' && WCODE::IsLineDelimiter(cStr.At(i+1), GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
+				if( i + 1 < cStr.GetLength() && cStr[i] == L';' && WCODE::IsLineDelimiter(cStr[i+1], GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
 					// ID;
 					this->m_nCOMMENTEND = i;
 					return false;
-				}else if( m_nSize < cStr.GetLength() && WCODE::IsLineDelimiter(cStr.At(m_nSize), GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
+				}else if( m_nSize < cStr.GetLength() && WCODE::IsLineDelimiter(cStr[m_nSize], GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
 					// ID
 					this->m_nCOMMENTEND = m_nSize;
 					return false;

sakura_core/view/colors/CColor_Quote.cpp

@@ -103,11 +103,11 @@ bool CColor_Quote::IsCppRawString(const CStringRef& cStr, int nPos)
 		// \b = ^|[\s!"#$%&'()=@{};:<>?,.*/\-\+\[\]\]
 		wchar_t c1 = L' ';
 		if( 2 <= nPos ){
-			c1 = cStr.At(nPos-2);
+			c1 = cStr[nPos-2];
 		}
 		wchar_t c2 = L' ';
 		if( 3 <= nPos ){
-			c2 = cStr.At(nPos-3);
+			c2 = cStr[nPos-3];
 		}
 		const wchar_t* pszSep = L" \t!\"#$%&'()=@{};:<>?,.*/-+[]";
 		if( (c1 == 'u' || c1 == 'U' || c1 == 'L') ){
@@ -117,7 +117,7 @@ bool CColor_Quote::IsCppRawString(const CStringRef& cStr, int nPos)
 		}else if( c1 == '8' && c2 == 'u' ){
 			wchar_t c3 = L'\0';
 			if( 4 <= nPos ){
-				c3 = cStr.At(nPos-4);
+				c3 = cStr[nPos-4];
 			}
 			if( NULL != wcschr(pszSep, c3) ){
 				return true;
@@ -142,7 +142,7 @@ bool CColor_Quote::BeginColor(const CStringRef& cStr, int nPos)
 		case STRING_LITERAL_CPP:
 			if( IsCppRawString(cStr, nPos) ){
 				for( int i = nPos + 1; i < cStr.GetLength(); i++ ){
-					if( cStr.At(i) == '(' ){
+					if( cStr[i] == '(' ){
 						if( nPos + 1 < i ){
 							m_tag = L')';
 							m_tag.append( cStr.GetPtr()+nPos+1, i - (nPos + 1) );
@@ -179,7 +179,7 @@ bool CColor_Quote::BeginColor(const CStringRef& cStr, int nPos)
 			break;
 		case STRING_LITERAL_PYTHON:
 			if( nPos + 2 < cStr.GetLength()
-			 && cStr.At(nPos+1) == m_cQuote && cStr.At(nPos+2) == m_cQuote ){
+			 && cStr[nPos+1] == m_cQuote && cStr[nPos+2] == m_cQuote ){
 				m_nCOMMENTEND = Match_QuoteStr( m_szQuote, 3, nPos + 3, cStr, true );
 				m_nColorTypeIndex = 3;
 				return true;
@@ -198,9 +198,9 @@ bool CColor_Quote::BeginColor(const CStringRef& cStr, int nPos)
 			// 終了文字列がない場合は行末までを色分け
 			if( m_pTypeData->m_bStringEndLine ){
 				// 改行コードを除く
-				if( 0 < cStr.GetLength() && WCODE::IsLineDelimiter(cStr.At(cStr.GetLength()-1), GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
-					if( 1 < cStr.GetLength() && cStr.At(cStr.GetLength()-2) == WCODE::CR
-							&& cStr.At(cStr.GetLength()-1) == WCODE::LF ){
+				if( 0 < cStr.GetLength() && WCODE::IsLineDelimiter(cStr[cStr.GetLength()-1], GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
+					if( 1 < cStr.GetLength() && cStr[cStr.GetLength()-2] == WCODE::CR
+							&& cStr[cStr.GetLength()-1] == WCODE::LF ){
 						m_nCOMMENTEND = cStr.GetLength() - 2;
 					}else{
 						m_nCOMMENTEND = cStr.GetLength() - 1;
@@ -256,29 +256,29 @@ int CColor_Quote::Match_Quote( wchar_t wcQuote, int nPos, const CStringRef& cLin
 		nCharChars = (Int)t_max(CLogicInt(1), CNativeW::GetSizeOfChar( cLineStr.GetPtr(), cLineStr.GetLength(), i ));
 		if( escapeType == STRING_LITERAL_CPP ){
 			// エスケープ \"
-			if( 1 == nCharChars && cLineStr.At(i) == L'\\' ){
+			if( 1 == nCharChars && cLineStr[i] == L'\\' ){
 				++i;
-				if( i < cLineStr.GetLength() && WCODE::IsLineDelimiter(cLineStr.At(i), GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
+				if( i < cLineStr.GetLength() && WCODE::IsLineDelimiter(cLineStr[i], GetDllShareData().m_Common.m_sEdit.m_bEnableExtEol) ){
 					if( pbEscapeEnd ){
 						*pbEscapeEnd = true;
 					}
 				}
 			}else
-			if( 1 == nCharChars && cLineStr.At(i) == wcQuote ){
+			if( 1 == nCharChars && cLineStr[i] == wcQuote ){
 				return i + 1;
 			}
 		}else if( escapeType == STRING_LITERAL_PLSQL ){
 			// エスケープ ""
-			if( 1 == nCharChars && cLineStr.At(i) == wcQuote ){
-				if( i + 1 < cLineStr.GetLength() && cLineStr.At(i + 1) == wcQuote ){
+			if( 1 == nCharChars && cLineStr[i] == wcQuote ){
+				if( i + 1 < cLineStr.GetLength() && cLineStr[i + 1] == wcQuote ){
 					++i;
 				}else{
 					return i + 1;
 				}
 			}
 		}else{
 			// エスケープなし
-			if( 1 == nCharChars && cLineStr.At(i) == wcQuote ){
+			if( 1 == nCharChars && cLineStr[i] == wcQuote ){
 				return i + 1;
 			}
 		}

sakura_core/view/figures/CFigure_Comma.cpp

@@ -90,6 +90,14 @@ void CFigure_Comma::DispSpace(CGraphics& gr, DispPos* pDispPos, CEditView* pcVie
 			if (szViewString.length() < nTabDispWidth) {
 				szViewString.append(nTabDispWidth - szViewString.length(), L' ');
 			}
+			const INT* lpDx;
+			if( szViewString.length() > 64 ) {
+				static std::vector<int> anHankakuDx; //!< 半角用文字間隔配列
+				anHankakuDx.resize(szViewString.length(), pMetrics->GetHankakuDx());
+				lpDx = &anHankakuDx[0];
+			}else {
+				lpDx = pMetrics->GetDxArray_AllHankaku();
+			}
 			::ExtTextOut(
 				gr,
 				sPos.GetDrawPos().x,
@@ -98,7 +106,7 @@ void CFigure_Comma::DispSpace(CGraphics& gr, DispPos* pDispPos, CEditView* pcVie
 				&rcClip2,
 				szViewString.c_str(),
 				static_cast<UINT>(szViewString.length()),
-				pMetrics->GetDxArray_AllHankaku()
+				lpDx
 			);
 		}
 	}