Skip to content

NULLマクロの再定義をやめる #2104

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
berryzplus merged 4 commits into from
Jul 21, 2025
Merged

NULLマクロの再定義をやめる #2104

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
4fe1df6
NULLマクロの再定義をやめる
berryzplus Jun 29, 2025
7ed994d
Merge branch 'master' into feature/revert_null_redefine
berryzplus Jul 6, 2025
9abd971
Merge branch 'master' into feature/revert_null_redefine
berryzplus Jul 10, 2025
b439070
Merge branch 'master' into feature/revert_null_redefine
berryzplus Jul 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 0 additions & 13 deletions sakura_core/StdAfx.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,19 +114,6 @@
#include <vsstyle.h>
#include <wrl.h>

// Windows SDKのマクロ定数「NULL」を訂正する。
// マクロ定数「NULL」は、省略可能なポインタ型パラメータに「省略」を指定するために使う。
// オリジナルでは「#define NULL 0」と定義されている。
// C++ではC++11からnullptrキーワードが導入されており、
// ポインタ型に0を渡すのは「不適切」になっている。
// 従来通りマクロ定数「NULL」を書けるようにするため、独自に上書き定義してしまう。
#ifdef __cplusplus
# pragma warning( push )
# pragma warning( disable : 4005 )
# define NULL nullptr
# pragma warning( pop )
#endif // end of #ifdef __cplusplus

// プロジェクト内のファイルだがプリコンパイル対象とする。
// プリコンパイルの有無がビルドパフォーマンスに大きく影響するため。
#include "env/DLLSHAREDATA.h"
Expand Down
50 changes: 25 additions & 25 deletions sakura_core/env/CShareData.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,9 @@ const unsigned int uShareDataVersion = N_SHAREDATA_VERSION;
//@@@ 2002.01.03 YAZAKI m_tbMyButtonなどをCShareDataからCMenuDrawerへ移動
CShareData::CShareData()
{
m_hFileMap = NULL;
m_pShareData = NULL;
m_pvTypeSettings = NULL;
m_hFileMap = nullptr;
m_pShareData = nullptr;
m_pvTypeSettings = nullptr;
}

/*!
Expand All @@ -72,20 +72,20 @@ CShareData::~CShareData()
{
if( m_pShareData ){
/* プロセスのアドレス空間から､ すでにマップされているファイル ビューをアンマップします */
SetDllShareData( NULL );
SetDllShareData(nullptr);
::UnmapViewOfFile( m_pShareData );
m_pShareData = NULL;
m_pShareData = nullptr;
}
if( m_hFileMap ){
CloseHandle( m_hFileMap );
}
if( m_pvTypeSettings ){
for( int i = 0; i < (int)m_pvTypeSettings->size(); i++ ){
delete (*m_pvTypeSettings)[i];
(*m_pvTypeSettings)[i] = NULL;
(*m_pvTypeSettings)[i] = nullptr;
}
delete m_pvTypeSettings;
m_pvTypeSettings = NULL;
m_pvTypeSettings = nullptr;
}
}

Expand All @@ -112,7 +112,7 @@ bool CShareData::InitShareData()
{
MY_RUNNINGTIMER(cRunningTimer,L"CShareData::InitShareData" );

m_hwndTraceOutSource = NULL; // 2006.06.26 ryoji
m_hwndTraceOutSource = nullptr; // 2006.06.26 ryoji

/* ファイルマッピングオブジェクト */
{
Expand All @@ -121,16 +121,16 @@ bool CShareData::InitShareData()
strShareDataName += pszProfileName;
m_hFileMap = ::CreateFileMapping(
INVALID_HANDLE_VALUE, // Sep. 6, 2003 wmlhq
NULL,
nullptr,
PAGE_READWRITE | SEC_COMMIT,
0,
sizeof( DLLSHAREDATA ),
strShareDataName.c_str()
);
}
if( NULL == m_hFileMap ){
if( nullptr == m_hFileMap ){
::MessageBox(
NULL,
nullptr,
L"CreateFileMapping()に失敗しました",
L"予期せぬエラー",
MB_OK | MB_APPLMODAL | MB_ICONSTOP
Expand All @@ -155,20 +155,20 @@ bool CShareData::InitShareData()
m_pShareData->m_nSize = sizeof(*m_pShareData);

// 2004.05.13 Moca リソースから製品バージョンの取得
GetAppVersionInfo( NULL, VS_VERSION_INFO,
GetAppVersionInfo( nullptr, VS_VERSION_INFO,
&m_pShareData->m_sVersion.m_dwProductVersionMS, &m_pShareData->m_sVersion.m_dwProductVersionLS );

m_pShareData->m_sFlags.m_bEditWndChanging = FALSE; // 編集ウィンドウ切替中 // 2007.04.03 ryoji
m_pShareData->m_sFlags.m_bRecordingKeyMacro = FALSE; /* キーボードマクロの記録中 */
m_pShareData->m_sFlags.m_hwndRecordingKeyMacro = NULL; /* キーボードマクロを記録中のウィンドウ */
m_pShareData->m_sFlags.m_hwndRecordingKeyMacro = nullptr; /* キーボードマクロを記録中のウィンドウ */

m_pShareData->m_sNodes.m_nSequences = 0; /* ウィンドウ連番 */
m_pShareData->m_sNodes.m_nNonameSequences = 0;
m_pShareData->m_sNodes.m_nGroupSequences = 0; /* タブグループ連番 */ // 2007.06.20 ryoji
m_pShareData->m_sNodes.m_nEditArrNum = 0;

m_pShareData->m_sHandles.m_hwndTray = NULL;
m_pShareData->m_sHandles.m_hwndDebug = NULL;
m_pShareData->m_sHandles.m_hwndTray = nullptr;
m_pShareData->m_sHandles.m_hwndDebug = nullptr;

for( int i = 0; i < _countof(m_pShareData->m_dwCustColors); i++ ){
m_pShareData->m_dwCustColors[i] = RGB( 255, 255, 255 );
Expand Down Expand Up @@ -744,9 +744,9 @@ bool CShareData::InitShareData()
m_pShareData->m_nSize != sizeof(*m_pShareData) ){
// この共有データ領域は使えない．
// ハンドルを解放する
SetDllShareData( NULL );
SetDllShareData( nullptr );
::UnmapViewOfFile( m_pShareData );
m_pShareData = NULL;
m_pShareData = nullptr;
return false;
}
// To Here Oct. 27, 2000 genta
Expand Down Expand Up @@ -837,7 +837,7 @@ void CShareData::ConvertLangValues(std::vector<std::wstring>& values, bool bSetV
break;
}
}
assert( m_pvTypeSettings != NULL );
assert( m_pvTypeSettings != nullptr );
indexBackup = index;
ConvertLangValue( shareData.m_TypeBasis.m_szTypeName, STR_TYPE_NAME_BASIS );
for( i = 0; i < (int)GetTypeSettings().size(); i++ ){
Expand Down Expand Up @@ -873,7 +873,7 @@ void CShareData::ConvertLangValues(std::vector<std::wstring>& values, bool bSetV
BOOL CShareData::IsPathOpened( const WCHAR* pszPath, HWND* phwndOwner )
{
EditInfo* pfi;
*phwndOwner = NULL;
*phwndOwner = nullptr;

// 2007.10.01 genta 相対パスを絶対パスに変換
// 変換しないとIsPathOpenedで正しい結果が得られず，
Expand Down Expand Up @@ -1035,7 +1035,7 @@ void CShareData::TraceOutString( const wchar_t* pStr, int len )
bool CShareData::OpenDebugWindow( HWND hwnd, bool bAllwaysActive )
{
bool ret = true;
if( NULL == m_pShareData->m_sHandles.m_hwndDebug
if( nullptr == m_pShareData->m_sHandles.m_hwndDebug
|| !IsSakuraMainWindow( m_pShareData->m_sHandles.m_hwndDebug )
){
// 2007.06.26 ryoji
Expand All @@ -1049,7 +1049,7 @@ bool CShareData::OpenDebugWindow( HWND hwnd, bool bAllwaysActive )
// CODE_UNICODE->CODE_NONE 2010.05.11 Moca デフォルト文字コードで設定できるように無指定に変更
sLoadInfo.eCharCode = CODE_NONE;
sLoadInfo.bViewMode = false;
ret = CControlTray::OpenNewEditor( NULL, hwnd, sLoadInfo, L"-DEBUGMODE", true );
ret = CControlTray::OpenNewEditor( nullptr, hwnd, sLoadInfo, L"-DEBUGMODE", true );
// 2001/06/23 N.Nakatani 窓が出るまでウエイトをかけるように修正
//アウトプットウインドウが出来るまで5秒ぐらい待つ。
// Jun. 25, 2001 genta OpenNewEditorの同期機能を利用するように変更
Expand Down Expand Up @@ -1112,7 +1112,7 @@ int CShareData::GetMacroFilename( int idx, WCHAR *pszPath, int nBufLen )
pszFile = m_pShareData->m_Common.m_sMacro.m_MacroTable[idx].m_szFile;
}
if( pszFile[0] == L'\0' ){ // ファイル名が無い
if( pszPath != NULL ){
if( pszPath != nullptr ){
pszPath[0] = L'\0';
}
return 0;
Expand All @@ -1122,7 +1122,7 @@ int CShareData::GetMacroFilename( int idx, WCHAR *pszPath, int nBufLen )

if( !_IS_REL_PATH( pszFile ) // 絶対パス
|| m_pShareData->m_Common.m_sMacro.m_szMACROFOLDER[0] == L'\0' ){ // フォルダー指定なし
if( pszPath == NULL || nBufLen <= nLen ){
if( pszPath == nullptr || nBufLen <= nLen ){
return -nLen;
}
wcscpy( pszPath, pszFile );
Expand All @@ -1146,7 +1146,7 @@ int CShareData::GetMacroFilename( int idx, WCHAR *pszPath, int nBufLen )

int nDirLen = wcslen( pszDir );
nAllLen = nDirLen + nLen + ( -1 == nFolderSep ? 1 : 0 );
if( pszPath == NULL || nBufLen <= nAllLen ){
if( pszPath == nullptr || nBufLen <= nAllLen ){
return -nAllLen;
}

Expand Down Expand Up @@ -1453,7 +1453,7 @@ void CShareData::RefreshString()

void CShareData::CreateTypeSettings()
{
if( NULL == m_pvTypeSettings ){
if( nullptr == m_pvTypeSettings ){
m_pvTypeSettings = new std::vector<STypeConfig*>();
}
}
Expand Down
23 changes: 7 additions & 16 deletions tests/unittests/test-cnative.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,6 @@
#include "mem/CNativeW.h"
#include "mem/CNativeA.h"

// NULL定義補正
// 詳細はStdAfx.hを参照
#ifdef __cplusplus
# pragma warning( push )
# pragma warning( disable : 4005 )
# define NULL nullptr
# pragma warning( pop )
#endif // end of #ifdef __cplusplus

/*!
CStringRefのテスト
*/
Expand Down Expand Up @@ -259,7 +250,7 @@ TEST(CNativeW, AssignString)
TEST(CNativeW, AssignStringNullPointer)
{
CNativeW value(L"test");
value = NULL;
value = nullptr; // NULLではなくnullptrを使うよう修正
EXPECT_EQ(0, value.GetStringLength());
EXPECT_EQ(NULL, value.GetStringPtr());
}
Expand All @@ -272,7 +263,7 @@ TEST(CNativeW, AssignStringNullPointer)
TEST(CNativeW, AssignStringNullLiteral)
{
CNativeW value(L"test");
value = NULL;
value = nullptr; // NULLではなくnullptrを使うよう修正
ASSERT_EQ(NULL, value.GetStringPtr());
EXPECT_EQ(0, value.GetStringLength());
}
Expand Down Expand Up @@ -319,7 +310,7 @@ TEST(CNativeW, AppendStringNullPointer)
{
CNativeW org(L"orz");
CNativeW value(org);
value += NULL;
value += nullptr; // NULLではなくnullptrを使うよう修正
EXPECT_EQ(value.GetStringLength(), org.GetStringLength());
EXPECT_EQ(org, value);
}
Expand Down Expand Up @@ -352,7 +343,7 @@ TEST(CNativeW, AppendStringWithFormatting)
ASSERT_EQ(L"いちご25%", value);

// 未確保状態からの書式化をテストする
value = NULL; //テスト前の初期値(未確保
value = nullptr; //テスト前の初期値(未確保

Copilot AI Jul 3, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The comment is missing a closing parenthesis—consider changing to // テスト前の初期値 (未確保).

Copilot uses AI. Check for mistakes.

@berryzplus berryzplus Jul 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントに対する指摘なのでスルーします。
（修正しても振る舞いに影響がないため。）

value.AppendStringF( L"KEY[%03d]", 12 );
ASSERT_EQ( L"KEY[012]", value );

Expand All @@ -367,7 +358,7 @@ TEST(CNativeW, AppendStringWithFormatting)
// フォーマット出力長2047字を超える条件をテストする
{
std::wstring longText( 2048, L'=' );
value = NULL; //テスト前の初期値(未確保
value = nullptr; //テスト前の初期値(未確保

Copilot AI Jul 3, 2025

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] Add the missing closing parenthesis in the comment: // テスト前の初期値 (未確保).

Suggested change
value = nullptr; //テスト前の初期値(未確保
value = nullptr; //テスト前の初期値(未確保)

Copilot uses AI. Check for mistakes.

@berryzplus berryzplus Jul 10, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コメントに対する指摘なのでスルーします。
（修正しても振る舞いに影響がないため。）

value.AppendStringF( L"%s", longText.c_str() );
ASSERT_EQ( longText.c_str(), value );
}
Expand All @@ -390,7 +381,7 @@ TEST(CNativeW, operatorEqualNull)
TEST(CNativeW, operatorEqualNullptr)
{
CNativeW value;
ASSERT_EQ(value, NULL);
ASSERT_EQ(value, nullptr); // NULLではなくnullptrを使うよう修正
}

/*!
Expand Down Expand Up @@ -533,7 +524,7 @@ TEST(CNativeW, operatorNotEqualNullptr)
{
constexpr const wchar_t text[] = L"おっす！オラ(ry";
CNativeW value(text);
ASSERT_NE(value, NULL);
ASSERT_NE(value, nullptr); // NULLではなくnullptrを使うよう修正
}

/*!
Expand Down