Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Undo the now unnecessary change to the sample

Browse Source 
The previously working code should continue to work, so keep the
original version now that it does work after the changes of the previous
commit.

Just mention in the comment that te data object can also be retrieved
directly.
This commit is contained in:
Vadim Zeitlin
2021-12-10 19:34:54 +00:00
parent 33018ea7a9
commit 5ed5803deb
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
samples/dataview/dataview.cpp
View File

@@ -1430,20 +1430,26 @@ void MyFrame::OnDrop( wxDataViewEvent &event )
return;
}
const wxTextDataObject* const obj = static_cast<wxTextDataObject*>(event.GetDataObject());
// Note that instead of recreating a new data object here we could also
// retrieve the data object from the event, using its GetDataObject()
// method. This would be more efficient as it would avoid copying the text
// one more time, but would require a cast in the code and we don't really
// care about efficiency here.
wxTextDataObject obj;
obj.SetData( wxDF_UNICODETEXT, event.GetDataSize(), event.GetDataBuffer() );
if ( item.IsOk() )
{
if (m_music_model->IsContainer(item))
{
wxLogMessage("Text '%s' dropped in container '%s' (proposed index = %i)",
obj->GetText(), m_music_model->GetTitle(item), event.GetProposedDropIndex());
obj.GetText(), m_music_model->GetTitle(item), event.GetProposedDropIndex());
}
else
wxLogMessage("Text '%s' dropped on item '%s'", obj->GetText(), m_music_model->GetTitle(item));
wxLogMessage("Text '%s' dropped on item '%s'", obj.GetText(), m_music_model->GetTitle(item));
}
else
wxLogMessage("Text '%s' dropped on background (proposed index = %i)", obj->GetText(), event.GetProposedDropIndex());
wxLogMessage("Text '%s' dropped on background (proposed index = %i)", obj.GetText(), event.GetProposedDropIndex());
}
#endif // wxUSE_DRAG_AND_DROP