+ helper function to obtain MulticlassLabels from Labels, - previous …

…constructor
shogun-toolbox · May 21, 2012 · 89a638e · 89a638e
1 parent 5f02a4a
commit 89a638e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/shogun/labels/MulticlassLabels.cpp b/src/shogun/labels/MulticlassLabels.cpp
@@ -21,9 +21,12 @@ CMulticlassLabels::CMulticlassLabels(CFile* loader) : CDenseLabels(loader)
 {
 }
 
-CMulticlassLabels::CMulticlassLabels(CLabels* const base_labels) : CDenseLabels()
+CMulticlassLabels* CMulticlassLabels::obtain_from_generic(CLabels* base_labels)
 {
-	m_labels = ((CMulticlassLabels*) base_labels)->m_labels; 
+	if ( base_labels->get_label_type() == LT_MULTICLASS )
+		return (CMulticlassLabels*) base_labels;
+	else
+		SG_ERROR("base_labels must be of dynamic type CMulticlassLabels");
 }
 
 bool CMulticlassLabels::is_valid()

diff --git a/src/shogun/labels/MulticlassLabels.h b/src/shogun/labels/MulticlassLabels.h
@@ -54,11 +54,11 @@ class CMulticlassLabels : public CDenseLabels
 		 */
 		CMulticlassLabels(CFile* loader);
 
-		/** constructor (can be used to downcast CLabels) 
+		/** helper method used to specialize a base class instance 
 		 *
-		 * @param labs its dynamic type must be CMulticlassLabels
+		 * @param base_labels its dynamic type must be CMulticlassLabels
 		 */
-		CMulticlassLabels(CLabels* const base_labels);
+		CMulticlassLabels* obtain_from_generic(CLabels* base_labels);
 
 		/** is_valid checks if labeling is a multi-class labeling
 		 *