Cleaning Up CFile

Incorporating the changes in PR #2596 
Maybe I am getting something wrong here. I am talking about a few methods
whose implementation before this patch was empty, but for which you have
now added a dummy implementation.

On 21 November 2014 09:01, Abhinav Agarwalla <notifications@github.com>
wrote:

> In src/shogun/io/File.cpp:
>
> > @@ -193,6 +426,64 @@ char* CFile::get_variable_name()
> >  	return strdup(variable_name);
> >  }
> >
> > +#define STRING_LIST_GETTER(type)									\
> > +void CFile::get_string_list(										\			
>
> I tried that approach, and it also passed all the tests. I just kept the
> dummy ones considering that whoever wrote them, must have written them for
> some reason.
> IMO, It may have been written for debugging the the class, maybe.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shogun-toolbox/shogun/pull/2616/files#r20702511>.
>