diff --git a/samples/dataview/dataview.cpp b/samples/dataview/dataview.cpp
index 0f5fc5de16..c55e399eb5 100644
--- a/samples/dataview/dataview.cpp
+++ b/samples/dataview/dataview.cpp
@@ -299,11 +299,12 @@ public:
         MyMusicModelNode *node = (MyMusicModelNode*) item.GetID();
         switch (col)
         {
-            case 0: node->m_title = variant.GetString(); break;
-            case 1: node->m_artist  = variant.GetString(); break;
-            case 2: node->m_year  = variant.GetLong(); break;
+            case 0: node->m_title = variant.GetString(); return true;
+            case 1: node->m_artist  = variant.GetString(); return true;
+            case 2: node->m_year  = variant.GetLong(); return true;
             default: wxLogError( "MyMusicModel::SetValue: wrong column" );
         }
+        return false;
     }
 
     virtual wxDataViewItem GetParent( const wxDataViewItem &item ) const