Yes, because another constructor may be used.
On 14 Jan 2014 10:08, "mazumdarparijat" <notifications@github.com> wrote:

> In src/shogun/clustering/KMeans.cpp:
>
> > @@ -30,12 +30,13 @@
> >  	init();
> >  }
> >
> > -CKMeans::CKMeans(int32_t k_, CDistance* d)
> > +CKMeans::CKMeans(int32_t k_, CDistance* d, bool use_kmpp))
> >  : CDistanceMachine()
> >  {
> >  	init();
>
> I have passed on default false to use_kmpp in the .h file . Is it still
> required to initialize in init() ?
> There is error in Travis build here because of an extra ')' which might
> have got added during my final touches. I have fixed it.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/shogun-toolbox/shogun/pull/1814/files#r8853561>
> .
>