I agree with you. The SGVector has a T* member that could turn out
problematic if not initialized.
On 6 Aug 2013 20:26, "Shell Hu" <notifications@github.com> wrote:

> In src/shogun/structure/FactorGraph.cpp:
>
> > @@ -12,17 +12,156 @@
> >
> >  using namespace shogun;
> >
> > +CDisjointSet::CDisjointSet()
> > +	: m_num_elements(-1)
> > +{
> > +	SG_UNSTABLE("CDisjointSet::CDisjointSet()", "\n");
> > +
> > +	register_parameters();
>
> Luckily, no problem for this one, since no pointers here, but it's better
> to initialize SGVector as well.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/shogun-toolbox/shogun/pull/1369/files#r5612037>
> .
>