CDropSource::DoDragDropを追加する

Author: berryzplus

sakura_core/_os/CDropTarget.cpp

@@ -171,6 +171,15 @@ STDMETHODIMP CDropSource::GiveFeedback( [[maybe_unused]] DWORD dropEffect )
 	return DRAGDROP_S_USEDEFAULTCURSORS;
 }
 
+DWORD CDropSource::DoDragDrop(LPDATAOBJECT pDataObject, DWORD dwDesiredEffects)
+{
+	if (DWORD dwEffects; SUCCEEDED(::DoDragDrop(pDataObject, this, dwDesiredEffects, &dwEffects))) {
+		return dwEffects;
+	}
+
+	return DROPEFFECT_NONE;
+}
+
 /** 転送対象の文字列を設定する
 	@param lpszText [in] 文字列
 	@param nTextLen [in] pszTextの長さ
@@ -224,15 +233,6 @@ void CDataObject::SetText( LPCWSTR lpszText, size_t nTextLen, BOOL bColumnSelect
 	}
 }
 
-DWORD CDataObject::DragDrop( BOOL bLeft, DWORD dwEffects )
-{
-	DWORD dwEffect;
-	CDropSource drop( bLeft );
-	if( SUCCEEDED( ::DoDragDrop( this, &drop, dwEffects, &dwEffect ) ) )
-		return dwEffect;
-	return DROPEFFECT_NONE;
-}
-
 /** IDataObject::GetData
 	@date 2008.03.26 ryoji 複数フォーマット対応
 */

sakura_core/_os/CDropTarget.h

@@ -122,6 +122,8 @@ class CDropSource : public CYbInterfaceImpl<IDropSource> {
 
 	STDMETHOD( QueryContinueDrag )( BOOL bEscapePressed, DWORD dwKeyState );
 	STDMETHOD( GiveFeedback )( DWORD dropEffect );
+
+	DWORD	DoDragDrop(LPDATAOBJECT pDataObject, DWORD dwDesiredEffects);
 };
 
 class CDataObject : public CYbInterfaceImpl<IDataObject> {
@@ -148,7 +150,6 @@ class CDataObject : public CYbInterfaceImpl<IDataObject> {
 	}
 	~CDataObject(){SetText( nullptr, 0, FALSE );}
 	void	SetText( LPCWSTR lpszText, size_t nTextLen, BOOL bColumnSelect );
-	DWORD	DragDrop( BOOL bLeft, DWORD dwEffects );
 
 	STDMETHOD( GetData )( LPFORMATETC, LPSTGMEDIUM );
 	STDMETHOD( GetDataHere )( LPFORMATETC, LPSTGMEDIUM );

sakura_core/view/CEditView_Mouse.cpp

@@ -2234,6 +2234,8 @@ void CEditView::DragSelection()
 	// ドラッグ開始前のドキュメント操作番号を記憶する
 	const auto nOpe = GetDocument()->m_cDocEditor.m_cOpeBuf.GetCurrentPointer();
 
+	CDropSource drop(true);
+
 	// ドラッグ元ビューを設定する
 	GetEditWnd().SetDragSourceView(this);
 
@@ -2244,7 +2246,7 @@ void CEditView::DragSelection()
 	}
 
 	// ドラッグ＆ドロップを開始する
-	const auto actualEffects = data->DragDrop(TRUE, desiredEffects);
+	const auto actualEffects = drop.DoDragDrop(data.get(), desiredEffects);
 
 	// ドラッグ元ビューの設定を解除する
 	GetEditWnd().SetDragSourceView(nullptr);

sakura_core/window/CEditWnd.cpp

@@ -3389,10 +3389,8 @@ LRESULT CEditWnd::OnMouseMove( WPARAM wParam, LPARAM lParam )
 			if (cxx::com_pointer<IDataObject> pDataObject; SUCCEEDED(GetDocument()->GetDataObject(&pDataObject))) {
 				CDropSource drop(true);
 
-				DWORD dwEffect = 0;
-
 				//移動禁止なので、戻り値を見ない
-				::DoDragDrop(pDataObject, &drop, DROPEFFECT_COPY | DROPEFFECT_LINK, &dwEffect);
+				drop.DoDragDrop(pDataObject, DROPEFFECT_COPY | DROPEFFECT_LINK);
 			}
 		}
 		return 0;

src/test/cpp/tests1/test-cclipboard.cpp

@@ -761,6 +761,18 @@ TEST(CDropSource, GiveFeedback001)
 	EXPECT_HRESULT_EQ(target->GiveFeedback(DROPEFFECT_MOVE), DRAGDROP_S_USEDEFAULTCURSORS);
 }
 
+/*!
+ * DoDragDrop のテスト。
+ */
+TEST(CDropSource, DoDragDrop101)
+{
+	const auto target = std::make_unique<CDropSource>(true);
+
+	LPDATAOBJECT pDataObject = nullptr;
+	DWORD dwEffects = 0;
+	EXPECT_HRESULT_EQ(target->DoDragDrop(pDataObject, dwEffects), DROPEFFECT_NONE);
+}
+
 /*!
  * 意図的にサポートしないメソッド のテスト。
  */
@@ -979,18 +991,6 @@ TEST(CopiedTextData, GetDataHere101)
 	EXPECT_HRESULT_EQ(target->GetDataHere(&format, &smallMedium), STG_E_MEDIUMFULL);
 }
 
-/*!
- * DragDrop のテスト。
- */
-TEST(CopiedTextData, DragDrop101)
-{
-	const auto target = std::make_unique<CDataObject>(L"abc", 3, FALSE);
-
-	BOOL bLeft = FALSE;
-	DWORD dwEffects = 0;
-	EXPECT_THAT(target->DragDrop(bLeft, dwEffects), DROPEFFECT_NONE);
-}
-
 /*!
  * CEnumFORMATETC のテスト。
  */